#8709: polyhedra level set doctest failure
----------------------------------+-----------------------------------------
   Reporter:  jhpalmieri          |       Owner:  mhampton       
       Type:  defect              |      Status:  positive_review
   Priority:  major               |   Milestone:  sage-4.4       
  Component:  geometry            |    Keywords:                 
     Author:  Marshall Hampton    |    Upstream:  N/A            
   Reviewer:  Andrey Novoseltsev  |      Merged:                 
Work_issues:                      |  
----------------------------------+-----------------------------------------
Changes (by newvalueoldvalue):

  * status:  needs_review => positive_review
  * reviewer:  => Andrey Novoseltsev
  * author:  => Marshall Hampton


Comment:

 The code in the patch is the same as in #8650, therefore it passes all
 other doctests just as well as before. The sorting issue is not an issue
 anymore with the modified doctest for these changes. So positive review,
 but it still would be nice to add some sorting in the future, especially
 for dim=0 and codim=1 faces.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8709#comment:3>
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.

Reply via email to