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

Current array indexing assumes low integer IDs #22

Closed
cmutel opened this issue Oct 31, 2024 · 0 comments
Closed

Current array indexing assumes low integer IDs #22

cmutel opened this issue Oct 31, 2024 · 0 comments
Assignees

Comments

@cmutel
Copy link
Member

cmutel commented Oct 31, 2024

This breaks use of Snowflake IDs.

See comment at

# TODO
# Sparse matrices could be used if the number of values present is much less
# than the number of possible values, given the (min, max) range.
# The default code will generate a complete mapping for the (min, max)
# interval, which can use too much memory in certain cases.
# self.use_sparse = len(self.keys) / self.keys.max() <= sparse_cutoff:

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

No branches or pull requests

1 participant