johannesvollmer / exrs

100% Safe Rust OpenEXR file library
Other
149 stars 22 forks source link

byte algorithm in compression module: test coverage not satisfying #190

Open johannesvollmer opened 1 year ago

johannesvollmer commented 1 year ago

https://github.com/johannesvollmer/exrs/blob/56f1d0a2cf6eb6cafd35c478f96f7e0707a5f3c7/src/compression/mod.rs#L630

this should test at least for 16 elements, because the algorithm is specialized for 16 elements

also it would be good to have a ground truth comparison instead of only a roundtrip test