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

Alternative textureResize() implementations #647

Closed
donmccurdy opened this issue Aug 2, 2022 · 0 comments · Fixed by #775
Closed

Alternative textureResize() implementations #647

donmccurdy opened this issue Aug 2, 2022 · 0 comments · Fixed by #775
Labels
feature New enhancement or request help wanted Extra attention is needed package:functions
Milestone

Comments

@donmccurdy
Copy link
Owner

It may be beneficial for this project to consider alternative libraries for resizing textures. The project currently uses ndarray-lanczos, solely for the reason that it works both in Node.js and Web. But it's not particularly fast or memory efficient, leading to issues like #344, and may have other bugs like #562.

I'd been hoping that @squoosh/lib would eventually work on the web (GoogleChromeLabs/squoosh#1084), and I could switch to its resize implementation. But it's unclear if that will happen.

The best Node.js resizing implementation is probably sharp.js, so supporting this as an alternative might be the way to go.

A few other related issues:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New enhancement or request help wanted Extra attention is needed package:functions
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant