Skip to content

add test data #5

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 1 commit into
base: main
Choose a base branch
from
Open

add test data #5

wants to merge 1 commit into from

Conversation

atlv24
Copy link
Contributor

@atlv24 atlv24 commented Jul 1, 2025

this might be too much test data, open to suggestions.

the most important test here is doomcone.obj, apparently: it is 22kb and currently fails. if i merge vertices in blender it no longer fails, suggesting something is wrong with the weld_vertices rewrite.

@Elabajaba
Copy link

Elabajaba commented Jul 1, 2025

What about adding a dev-dependency on eg. zstd and compressing the models?

edit: Quick test using the zstd cli at a few different compression levels for all the models combined (none is uncompressed):

none:     3,968,622 bytes
level 3:  1,157,252 bytes
level 12: 1,039,909 bytes
level 19: 856,175 bytes

@bushrat011899 bushrat011899 mentioned this pull request Jul 17, 2025
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