Skip to content

Compress-selection#207

Closed
simonbethke wants to merge 5 commits intoplaycanvas:mainfrom
simonbethke:Compress-selection
Closed

Compress-selection#207
simonbethke wants to merge 5 commits intoplaycanvas:mainfrom
simonbethke:Compress-selection

Conversation

@simonbethke
Copy link
Copy Markdown
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
Copy Markdown
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?

@slimbuck
Copy link
Copy Markdown
Member

This idea was eventually rolled into #803.

Closing, but with a little sadness to see it go :D

@slimbuck slimbuck closed this Feb 16, 2026
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.

2 participants