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.

Sign up for GitHub

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

Jump to bottom

Defaults to 32 bit functions on 64 bit installation of Python #5

Closed
thdb-theo opened this issue Mar 29, 2023 · 1 comment · May be fixed by #6
Closed

Defaults to 32 bit functions on 64 bit installation of Python #5

thdb-theo opened this issue Mar 29, 2023 · 1 comment · May be fixed by #6

Comments

@thdb-theo
Copy link

Because sys.maxint no longer exists, the code uses the 32 bit version of the functions. interleaving 0xffff and 0xffff yields the same result as 0xfffff and 0xffff since it's using the 32 bit function and the result should be 33 bits long.

Screenshot from 2023-03-29 17-15-04

The issue lies here:

pymorton/pymorton/pymorton.py

Line 92 in f248683

if getattr(sys, 'maxint', 0) and sys.maxint <= 2 ** 31 - 1:

@jeremyh3
Copy link

jeremyh3 commented May 18, 2023 •
edited

I also have this issue. Just have some very large numbers (64bit) that need to get the morton index for (geospatial mapping stuff), and it fails to function if the numbers are too large.

@Tobias-Fischer Tobias-Fischer mentioned this issue Aug 6, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use 32-bit check only if Python2 Tobias-Fischer/pymorton
2 participants
@jeremyh3 @thdb-theo