#16180: Subdivide fans using PPL
-------------------------------------+-------------------------------------
Reporter: novoselt | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.3
Component: geometry | Resolution:
Keywords: toric | Merged in:
Authors: Andrey | Reviewers: Jan Keitel
Novoseltsev | Work issues:
Report Upstream: N/A | Commit:
Branch: | f7f0dd54706d14d4d909715903581604ad133dc9
u/jkeitel/subdivide_fans_using_ppl | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by novoselt):
Well don't hate it please, there is nothing wrong with the class, only
with its backend which was designed for a quite specific purpose ;-) And
of course bounded polytopes are not the best way to represent unbounded
cones.
As for caching - why should we cache `polyhedron()`? It is a complicated
data structure which will take memory and can cause issues in pickling
(this was the main reason for the change here, if I recall correctly - I
was trying to simplify pickling code), it is not used extensively in
internal computations, and I don't think there are many reasons for users
to construct them. On the other hand it definitely can be handy if you
want to do some generic operations like shifting the cone from the origin.
Were there any conflicts during merge? (I never liked what is shown when
you click on "merge commit.")
--
Ticket URL: <http://trac.sagemath.org/ticket/16180#comment:6>
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.
