Sergey "Shnatsel" Davidoff

Results 942 comments of Sergey "Shnatsel" Davidoff

jpeg-encoder seems to be a fork of the image-rs encoder. I have already reached out to the author and they are on board with the integration, and have the capacity...

We could in theory copy over the jpeg-encoder code into `image`, but then we'd have two copies of the code and it's not entirely clear how to maintain them. Also,...

I've added utilities for premultiplying by alpha: https://github.com/image-rs/image/blob/ee7c5d979cb503ed0a78b984542af990dd7ecd87/src/imageops/resize.rs#L156-L197 However, they currently only work for `DynamicImage` because `Subpixel` trait is not expressive enough and does not provide the necessary bounds.

I like the idea! The main drawback of `wide` is the lack of runtime CPU feature detection. But since that's not relevant to ARM or WASM, and this crate already...

There was also a prototype of RGB-YCbCr conversion using `std::simd` in #13, I think it can be translated to `wide` quite easily. I've translated it to NEON intrinsics in #24...

MSRV bumps are no longer a concern now that our MSRV is above 1.85. Cargo has shipped the [Rust-version aware resolver](https://doc.rust-lang.org/edition-guide/rust-2024/cargo-resolver.html) and made it the default [in Rust 1.85](https://blog.rust-lang.org/2025/02/20/Rust-1.85.0/). Anyone...

People who stay on older Rust compiler (e.g. Linux distributions) are already backporting critical bugfixes by themselves. They already have to do it for the Rust compiler and standard library,...

`i16x8` should be a zero-cost conversion as long as the data is guaranteed to be aligned in memory, so I don't think it will be of any benefit compared to...

This looks very promising! If we could also generate versions with `#[target_feature]` attributes for SSE 4.2 and AVX2 and dispatch to those, that'd be great! The [`multiversion`](https://crates.io/crates/multiversion) crate makes that...

I can confirm the performance gain compared to explicit AVX on desktop Zen 4 as well, built with `-C target-cpu=x86-64-v3` as opposed to `native` that would tune for my CPU...