rust-av / ssimulacra2_bin

Binary interface for the ssimulacra2 Rust port
BSD 2-Clause "Simplified" License
40 stars 10 forks source link

Remove conversion to Xyb before passing to ssimulacra2 #8

Closed FreezyLemon closed 1 year ago

FreezyLemon commented 1 year ago

This not only reduces the amount of code, it also eliminates a redundant conversion: compute_frame_ssimulacra2 currently converts back to LRGB from the XYB that is passed to it.

This makes the code a bit faster, ~10% from simple testing.

shssoichiro commented 1 year ago

Thanks, this must have been left in here from when the algorithm got updated--it used to require Xyb only, then it changed upstream to require Linear RGB. :+1: