#7109: [with patch, needs review] polyhedra bugs with linearities, rewrite
proposal
---------------------------+------------------------------------------------
Reporter: vbraun | Owner: mhampton
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-feature
Component: geometry | Keywords: polyhedra
Work_issues: | Author:
Reviewer: | Merged:
---------------------------+------------------------------------------------
Comment(by vbraun):
I agree with you that compatibily should be preserved if possible.
But since the returned vertices/rays/... are now their own objects instead
of just lists of numbers any old code will break even if we return lists
instead of generators. On a more fundamental level, I think one should
always return generators instead of lists if possible (unless one is just
exposing some internal datastructure). That avoids unnecessary list
creation and enables parallelism. This is why I always favoured generators
over returning lists when I had the choice.
The alternative is to rewrite the code to always return lists instead of
generators. I don't think anyone is going to use vertex_generator() if one
can get a list out of the much shorter vertices() method.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7109#comment:11>
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 post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---