Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pynac hashes are restricted to 32 bits #19310

Closed
jdemeyer opened this issue Sep 29, 2015 · 9 comments
Closed

Pynac hashes are restricted to 32 bits #19310

jdemeyer opened this issue Sep 29, 2015 · 9 comments

Comments

@jdemeyer
Copy link

src/sage/symbolic/expression.pyx claims

        The hash of an object in Python or its coerced version into
        the symbolic ring is the same::

This is unfortunately not true since Pynac uses the 32-bit unsigned data type for hashes:

sage: hash(ZZ(2^32))
4294967296
sage: hash(SR(2^32))
0

See also pynac/pynac#95

Upstream: Fixed upstream, in a later stable release.

CC: @rwst

Component: packages: standard

Reviewer: Jeroen Demeyer

Issue created by migration from https://trac.sagemath.org/ticket/19310

@rwst

This comment has been minimized.

@rwst
Copy link

rwst commented Sep 29, 2015

Dependencies: #19312

@rwst
Copy link

rwst commented Sep 29, 2015

Upstream: Reported upstream. Developers acknowledge bug.

@rwst
Copy link

rwst commented Dec 19, 2015

Changed upstream from Reported upstream. Developers acknowledge bug. to Fixed upstream, in a later stable release.

@rwst
Copy link

rwst commented Dec 19, 2015

Changed dependencies from #19312 to pynac-0.5.4

@rwst
Copy link

rwst commented Dec 19, 2015

comment:2

The fix will be in pynac-0.5.4. This ticket will provide some more doctests after the upgrade.

@jdemeyer
Copy link
Author

jdemeyer commented Jan 3, 2016

Changed dependencies from pynac-0.5.4 to none

@jdemeyer
Copy link
Author

jdemeyer commented Jan 3, 2016

Reviewer: Jeroen Demeyer

@jdemeyer
Copy link
Author

jdemeyer commented Jan 3, 2016

comment:3

"Duplicate" of #19819.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants