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

Custom progressive steps for MozJPEG #1004

Draft
wants to merge 3 commits into
base: dev
Choose a base branch
from
Draft

Conversation

@jakearchibald
Copy link
Collaborator

@jakearchibald jakearchibald commented May 5, 2021

These progressive steps appear to play better with the DCT smoothing currently in Firefox and Chrome. However, I think there's a decoder bug right now, so it's possible we might not need this.

Or, maybe it should be upstreamed to MozJPEG itself?

@google-cla google-cla bot added the CLA: Yes label May 5, 2021
@jakearchibald
Copy link
Collaborator Author

@jakearchibald jakearchibald commented May 7, 2021

I'm going to let this rest until the libjpeg-turbo decoder bugs are gone, but I still think there'll be a benefit to this.

Btw, https://static-misc-3.glitch.me/slow-img-load/ lets you slow-load images (you can select multiple)

scanptr->Se = 63;
scanptr->Ah = 1;
scanptr->Al = 0;
scanptr++;
Copy link
Collaborator Author

@jakearchibald jakearchibald May 25, 2021

I'll extract these into a little helper function once we've got the output looking right

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant