imageworks / OpenColorIO-Configs

Color Configurations for OpenColorIO
opencolorio.org
441 stars 804 forks source link

ACES 0.7.1 #5

Closed alexfry closed 10 years ago

alexfry commented 10 years ago

First commit of ACES 0.7.1 config

jeremyselan commented 10 years ago

Thanks for this! I'm excited about getting an updated ACES profile in there.

I note that the size of the luts dir is 202MB. Does this concern you? If there's a way to do a more compact profile, it would probably be preferable. The size of the prior ACES profile luts dir is 5.3 MB, for comparison. I'm not sure what counts as reasonable in this case, but files of this size do make it harder to manage. Thoughts?

alexfry commented 10 years ago

Yeah, it's way bigger.

There are 3 contributing factors:

1: There are now way more ODT variations, for example there used to be only 1 for Rec709, there are now 4 to cover smpte vs full levels + different white points. You could potentially cut down here by implementing part of the ODTs in OCIO directly rather than baking each ODT variation into it's own LUT. (although thats probably easier in the forward direction than reverse)

2: Almost all RRT/ODT transforms now have a forward and a reverse direction, almost doubling the total.

3: To help with situations where you will be need to do things like go ..... Display Rec709 -> Inv Rec709 ODT/RRT -> Do work -> RRT/ODT -> Display ..... I thought it was worth moving up from 32x32 cubes to 64x64. I also want to move to a broader range 1D shaper soon, which I think will make the move to 64x64 worth it. I've also run into real world situations where 32x32 was causing banding with the old config, I spent a lot of time telling people in reviews to expose up and down to check if the banding was in the image or the display lut.

..

One easy win might be making some of the less useful ones an optional download (Do we really need 70MB of Rec2020 ODT variations) Although there are a few more that should really be in there like the 0.2.2 LMT, so the bundle size may actually need to go up.

Another might be to do a 32x32 version and just let people choose.

-Alex

On Sat, May 24, 2014 at 5:45 AM, Jeremy Selan notifications@github.comwrote:

Thanks for this! I'm excited about getting an updated ACES profile in there.

I note that the size of the luts dir is 202MB. Does this concern you? If there's a way to do a more compact profile, it would probably be preferable. The size of the prior ACES profile luts dir is 5.3 MB, for comparison. I'm not sure what counts as reasonable in this case, but files of this size do make it harder to manage. Thoughts?

— Reply to this email directly or view it on GitHubhttps://github.com/imageworks/OpenColorIO-Configs/pull/5#issuecomment-44040256 .

jeremyselan commented 10 years ago

Closing this commit. Ended up doing a squashed commit 2335988 with 32x32x32, for now, to keep the luts from exploding on us. We're open to investigating higher fidelity (binary?) 3d lut options in the future, but this is a great step 1. Thanks!