Scott Dyer

Results 29 comments of Scott Dyer

Because this change would (subtly) alter the output of the HDR Output Transforms, I'd really like to hold off on merging this in as a part of v1.2. I know...

I can't exactly recall why I left this duplication in there, but there was a reason. I believe it had to do with the annoying requirement to keep the commented...

Good thing to note, @nick-shaw 1.1 was released in June of 2018. ITU-R BT.2100-2 was not published until July of 2018. We can add this clarification in the 1.2 release.

@nick-shaw what sort of a comment were you thinking should be added here? After reviewing this, I'm not sure if it's necessary, but I could certainly add something if you...

@scoopxyz this looks good to me. I think adding the transform in a v4 directory for now makes sense - at least until (and if) you ever want to remove...

@remia good catch! Nobody else caught this is the beta testing phase but it should definitely be done in a consistent space. Like the "surround" module, the "white point sim"...

@miaoqi , I am having issues writing ADX files after your latest changes, though at one point earlier in the commit chain it seemed to be writing files just fine....

https://github.com/ampas/CTL/tree/ctl_1.5_rc

In lieu of a default of 16-bits and automatic scaling regardless of the bit depth of the input file, the error message “EXR files only support 16 or 32 bps...

This is also sort of related to issue #87 In general, any of the tool's limitations as well as expectations for user imported data should be well specified.