Skip to content

UUID caching - #114

Open
gizatt wants to merge 2 commits into
meshcat-dev:masterfrom
gizatt:uuid_caching
Open

UUID caching#114
gizatt wants to merge 2 commits into
meshcat-dev:masterfrom
gizatt:uuid_caching

Conversation

@gizatt

@gizattgizatt commented Apr 21, 2022

Copy link
Copy Markdown
Collaborator

This pairs with meshcat js PR #115 to avoid re-sending geometry or materials with duplicate UUIDs while achieving the same visualization result.

This hopefully saves a ton of bandwidth, and also dramatically reduces static_html export size for scenes with many repeated heavy elements. For example, the attached scene with 1000 Stanford bunnies (generated by this script) serializes to ~1MB with this caching, but >100MB without.

save_with_caching.zip

Note that for this to be useful, you do have to make sure your geometries / materials are actually have the same UUID. (E.g. if you create a meshcat geometry from the same mesh data twice, that might not be true by default.)

@codecov-commenter

codecov-commenter commented Apr 21, 2022

Copy link
Copy Markdown

Codecov Report

Merging #114 (3f7fc1a) into master (0f25add) will increase coverage by 0.58%.
The diff coverage is 100.00%.

@@ Coverage Diff @@## master #114 +/- ##
==========================================
+ Coverage 73.39% 73.97% +0.58% 
==========================================
Files 11 11 Lines 1075 1099 +24 ==========================================
+ Hits 789 813 +24 
Misses 286 286 
Impacted FilesCoverage Δ
src/meshcat/geometry.py89.18% <100.00%> (+0.21%)⬆️
src/meshcat/tests/test_drawing.py98.00% <100.00%> (+0.19%)⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f25add...3f7fc1a. Read the comment docs.

Sign up for freeto 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

@gizatt@codecov-commenter