search

microsoft / lepton_jpeg_rust

Port of DropBox Lepton compression to Rust
Apache License 2.0
129 stars 10 forks source link

Fix of calc_sign_index #85

Closed Melirius closed 4 months ago

Melirius commented 4 months ago

To get in line with initial Lepton implementation we need to use sign of i16 truncated prior.

See https://github.com/dropbox/lepton/blob/master/src/vp8/model/model.hh#L1114

Melirius commented 4 months ago

Looks good... can you add some comments about the reasons? Thanks!

Done