MrLixm / AgXc

Fork of Troy.S AgX, a display rendering transform available via OCIO and more
92 stars 8 forks source link

fix(ocio): Lifted black log issue #10

Closed MrLixm closed 1 year ago

MrLixm commented 1 year ago

Closes #9

This should fix black values being incorrectly offsetted during the log transform. This still doesn't create a perfect match with the base OCIO v2 AgX implementation. The input negatives are still not clamped. This should be addressed in #11

Changes

dev/

Comparison

Same image, with reference -> AgX Log transform only

test_log before ↳ before

test_log after ↳ now (this PR)

test_log ref ↳ original AgX on OCIOv2