Skip to content

Compress-selection #207

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

simonbethke
Copy link
Contributor

This is an improvement to shrink the memory footprint of the undo-history.
What this basically does is, that a indices are combined as ranges if they cover more than one subsequent. So this:
1, 2, 3, 4, 5, 6, 10
becomes:
1-6, 10

When selecting only few splats, this compression shows results in a typical range of 50 - 100% of the uncompressed size.

When selecting many splats like a tenth of the scene, it becomes more efficient and the typical compressed size is 2-20%.

When selecting almost all splats, the compressed size is less than 1%.

@simonbethke simonbethke mentioned this pull request Nov 27, 2024
@simonbethke
Copy link
Contributor Author

@slimbuck is this something you have already solved in a different way or is there something fundamental which you don't like on this implementation, but I should give it another shot from a different starting point?
If you don't want this for any reason, please just reject it. Else, what can I do to get this finished?

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

Successfully merging this pull request may close these issues.

1 participant