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

Switch spatial registers to a sparse array #110

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

deft-code
Copy link
Contributor

This is the minimal change needed to use a sparse spatial index.

I think this would be better as a Map (the datatype is more idiomatic for a matrix) but this current change is so small that it's very low risk.

@o4kapuk
Copy link
Contributor

o4kapuk commented Dec 19, 2019

@deft-code what's the point of this change?

@o4kapuk o4kapuk self-assigned this Dec 19, 2019
@sparr
Copy link
Contributor

sparr commented Jun 17, 2023

This change decreases memory usage and initial allocation time, at the expense of increased runtime for some future array operations.

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

Successfully merging this pull request may close these issues.

3 participants