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

Zip-compress .amf file #12845

Merged
merged 1 commit into from
Dec 12, 2017
Merged

Zip-compress .amf file #12845

merged 1 commit into from
Dec 12, 2017

Conversation

takahirox
Copy link
Collaborator

This PR zip-compresses rook.amf file.

An AMF can be stored as zip-compressed text and AMFLoader supports zip-compressed AMF file loading.

https://en.wikipedia.org/wiki/Additive_Manufacturing_File_Format#Compression

Advantage

  • Reduce file size from 403KB to 49KB

Disadvantage

  • The diff of binary file can't be visible on version control system.
  • AMFLoader example needs to load jszip.min.js

@mrdoob mrdoob merged commit 38f68d1 into mrdoob:dev Dec 12, 2017
@mrdoob
Copy link
Owner

mrdoob commented Dec 12, 2017

Thanks!

@takahirox takahirox deleted the CompressAMF branch December 12, 2017 06:50
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.

3 participants