#5117: remove (or enhance an rename) the Polyhedron.union()) method
-------------------------+--------------------------------------------------
Reporter: sbarthelemy | Owner: mhampton
Type: defect | Status: new
Priority: major | Milestone:
Component: geometry | Resolution:
Keywords: |
-------------------------+--------------------------------------------------
Comment (by sbarthelemy):
Replying to [comment:1 mhampton]:
> I disagree that this should be removed. I created it because I use it!
using
{{{
p = Polyhedron(p1.vertices() + p2.vertices())
}}}
instead of
{{{
p = p1.union(p2)
}}}
does not make a big difference ;)
> I think the docstring makes it pretty clear what it does,
agreed
> but I do not object to renaming it. Perhaps union_of_vertices? Or
union_by_vertices?
what about extending ot to handle unbounded polyhedra as well in this way:
{{{
p = Polyhedron(p1.vertices() + p2.vertices(), p1.rays() + p2.rays())
}}}
We could then name it convex_hull() or something like that?
--
Sebastien
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5117#comment:2>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
