-
Notifications
You must be signed in to change notification settings - Fork 587
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
Benchmark #99
Comments
It is really slow. The algorithms are not optimized for speed at all. They are more or less textbook implementations. |
@eddyb has a very fast png decoder, rivaling libpng. On Wed, Sep 10, 2014 at 1:28 PM, nwin notifications@github.com wrote:
|
That isn't in a public repos, is it? Haven't found anything. |
https://github.com/cmr/TEMP-rust-png is the last I know of it, he might On Wed, Sep 10, 2014 at 3:29 PM, nwin notifications@github.com wrote:
|
No, I do not. And I need to remove some unsafety now that LLVM understands some interesting boolean predicates, including those having to do with modulo operations. |
I just tested loading a png with this library compared to C++ using lodepng, and it was 550ms vs 24ms. Definitely not fast by any means. |
Closing in favor of #257. |
I know, I know, 💃 early stage.
Although, I'm very very interested to know how it performs with jpg (or png).
I did some image manipulations (most resize things) in the past few days (see: https://github.com/daddye/vips) and I had to give up and use straight c with libturbojpeg.
Would be very nice to know that now is possible to use rust for that!
Anyway, not a rustler, but the code seems very clean, simple and concise. For what is worth, Congrats!
The text was updated successfully, but these errors were encountered: