-
Notifications
You must be signed in to change notification settings - Fork 136
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
Optimisations #21
Comments
@jimmyff Thanks for the suggestion, I am planning to implement the same in the very next release. If you already worked out something, please send a PR. Thanks |
hello...any update on this? thanks |
@Nemesis19 Did you find anything on this? |
I Found the issue with the code, its due to spawning multiple isolate using the compute function but not all the isolates get killed |
@Mangielijah Thanks for finding out the issue. From the comments of the other issue, I assume you fixed it by creating a worker for the compute operation and kill it later. It will be very helpful if you can create a pull request with the changes. |
Eagerly waiting for this to be fixed!!! |
Any solution here? @skkallayath |
Hey, this is a great package however it regularly crashes my development enviroment and also sometimes crashed on my pixel 3 device. I think it's because it's applying each filter to the full sized image every time. I notice in your example you resize the image to 600 however i need a higher quality version for my app.
Could it generate thumbnails using a much smaller version of the source image and then when the user selects a filter apply that one on the full sized image? I think it would make the whole thing much more stable and feel more responsive.
The text was updated successfully, but these errors were encountered: