#19321: provide better hash functions
-------------------------------------+-------------------------------------
       Reporter:  vdelecroix         |        Owner:
           Type:  enhancement        |       Status:  positive_review
       Priority:  critical           |    Milestone:  sage-6.9
      Component:  misc               |   Resolution:
       Keywords:  hash               |    Merged in:
        Authors:  Nils Bruin,        |    Reviewers:  Nathann Cohen
  Vincent Delecroix                  |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:  public/19321       |  bebb8af5eaa7ff834a03e9c887da69b118a257af
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by ncohen):

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


Comment:

 Yo !

 To me, it seems good to go. I was a bit worried about the hash of
 polyhedra but the elements of your tuple are all integers, and the
 functions that return them are, well, 'not too scary' even though there is
 a bit of useless wrapping going on.

 Nathann

--
Ticket URL: <http://trac.sagemath.org/ticket/19321#comment:5>
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/d/optout.

Reply via email to