#5117: remove (or enhance an rename) the Polyhedron.union()) method
-------------------------+--------------------------------------------------
 Reporter:  sbarthelemy  |       Owner:  mhampton
     Type:  defect       |      Status:  new     
 Priority:  major        |   Milestone:          
Component:  geometry     |    Keywords:          
-------------------------+--------------------------------------------------
 The Polyhedron class (in the polyhedra module) has a union method
 {{{
 def union(self, other):
     """
     Returns a polyhedron whose vertices are the union of the vertices
     of the two polyhedra.
     ....
 }}}
 The name is misleading as the method does not return the union of
 {{{self}}} and {{{other}}} (which would not be a convex polyhedron).

 The method should then be removed or renamed. As the method itself
 consists in one single line of code (and as I have no idea of a proper
 name), I would tend to remove it.

 The attached patch removes it.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5117>
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to