#15280: Extensions of PALP normal form, affine normal form and isomorphisms
-------------------------------------+-------------------------------------
Reporter: jkeitel | Owner:
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-6.3
Component: geometry | Resolution:
Keywords: toric | Merged in:
Authors: Jan Keitel | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/jkeitel/normal_form_codimension | 61b0d264008bc64948d8aab6509e0cccf4c0f991
Dependencies: #13525 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by novoselt):
Jan, it would be better also to try to take advantage of the switch rather
than just making things work:
{{{
vertices = matrix(QQ, self.vertices_pc().column_matrix().transpose())
}}}
can be written more naturally as
{{{
vertices = matrix(QQ, self.vertices_pc().matrix())
}}}
which is shorter and does not require double transposition of a cached
matrix. Or perhaps as
{{{
vertices = self.vertices_pc().matrix().change_ring(QQ)
}}}
Sorry for annoying rebasing, but in the long run it should be nice to have
uniform behaviour for polytopes and cones/fans.
--
Ticket URL: <http://trac.sagemath.org/ticket/15280#comment:22>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
