#13312: Polyhedral bugfixes and improvements
-------------------------------------+-------------------------------------
       Reporter:  mhampton           |        Owner:  mhampton
           Type:  defect             |       Status:  positive_review
       Priority:  major              |    Milestone:  sage-6.1
      Component:  geometry           |   Resolution:
       Keywords:  polyhedra, polar   |    Merged in:
        Authors:  Marshall Hampton,  |    Reviewers:  Nathann Cohen
  Volker Braun                       |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  9e2fe957e16c53fe39df2d58cb739b0dc416bfd6
  u/vbraun/polyhedral_neg_polar      |     Stopgaps:
   Dependencies:  #11763             |
-------------------------------------+-------------------------------------
Changes (by ncohen):

 * status:  needs_review => positive_review
 * reviewer:   => Nathann Cohen


Comment:

 Oh sorry. I thought that you just changed the result of this `.polar()`
 thing so that the two constructors would be equal, and not just
 isomorphic. I was saying that instead of changing the behaviour of this
 function to get this result, you should just multiply one of the two
 constructors by `-1` which would appear more correct `:-P`

 Okayokay, then if this def of `polar` is the good one, this patch is good
 to go as all other additions seem good too `;-)`

 Nathann

--
Ticket URL: <http://trac.sagemath.org/ticket/13312#comment:20>
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/groups/opt_out.

Reply via email to