compress json data arrays#62
Merged
joshday merged 3 commits intoJuliaComputing:masterfrom Jan 12, 2026
Merged
Conversation
Collaborator
|
Very cool! Before merging, I think I'd want this to be implemented as an extension package and make it opt-in. Maybe add |
Contributor
Author
|
Thanks yea I can do the optional. Would you consider non-extension though? It just that its another step for the user and CodecZlib seems pretty light, the only leaf dependencies added are TranscodingStreams and CodecZlib_jll. Ok either way just putting out there. |
Contributor
Author
|
Made it opt-in. Also added to readme, as well as removed the note about JSON3.write which sounded contradictory to the the compression (and I think was no longer true anyway). Lmk about the extension thing. |
joshday
approved these changes
Jan 12, 2026
Collaborator
Yeah, that's cool. LGTM |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This compresses Float, Int, and String arrays inside the plot data to massively reduce html file size and improve performance.
Recent Plotly versions have the ability to feed in base64 strings directly, but I chose to do it this way to allow for the Zlib compression and because we can also compress string arrays, which Plotly doesn't do. Zlib compressed string arrays are nice because in this domain its common for eg
mesh.facecolor = [...]with hex strings and many repeated entries, so you get huge compression factors. Python Plotly uses the base64 feature but doesn't have compression or string arrays, so won't be as good as this package.Open to feedback on this, lmk, I know there's some opionated choices. Anecdotally this code has been in production at @Atomic-Industries for ~1yr and is working great.
Here's an example code snippet that results in a 7.4Mb file on master and 1.2Mb file on this PR (and 3.4Mb with Python, I think the difference is Zlib)