#4747: [with patch; with review, needs work] add custom hash function for cusps.
---------------------------+------------------------------------------------
Reporter: was | Owner: was
Type: enhancement | Status: new
Priority: major | Milestone: sage-3.2.2
Component: number theory | Resolution:
Keywords: |
---------------------------+------------------------------------------------
Changes (by cremona):
* summary: [with patch; needs review] add custom hash function for
cusps. => [with patch; with review, needs work]
add custom hash function for cusps.
Comment:
Patch applies fine and test ok on 32-bit but on 64-bit I get this:
{{{
sage -t "sage-3.2.1.rc0/devel/sage-
hash/sage/modular/cusps.py"**********************************************************************
File "/home/jec/sage-3.2.1.rc0/devel/sage-hash/sage/modular/cusps.py",
line 319:
sage: hash(Cusp(1/3))
Expected:
3713081631933328131
Got:
163512108431620420
**********************************************************************
File "/home/jec/sage-3.2.1.rc0/devel/sage-hash/sage/modular/cusps.py",
line 322:
sage: hash(Cusp(oo))
Expected:
3713081631936575706
Got:
6982220252595711780
**********************************************************************
}}}
I am amazed at the effect this has on the speed. Hashing is not something
I ever think about, but clearly I should!
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4747#comment:1>
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
-~----------~----~----~----~------~----~------~--~---
