Re: Review: FileTransform supports .cc and .ccc files


Jeremy Selan <jeremy...@...>
 

LGTM, committed.
Thanks!

-- Jeremy

On Thu, Jan 27, 2011 at 2:52 AM, dbr/Ben <b...@...> wrote:
Oh,
https://github.com/dbr/OpenColorIO/commit/0247f16ae202eeb5316f695c1310dac523272a0e
On 23/01/2011, at 4:41 AM, Jeremy Selan wrote:

Nice job on the docs.  By all means, check them in!  (Even partial
docs are better than nothing). :)

I wonder, could the cccid's be presented in a drop-down, similar to the

colour-space listings?

I like this, but I'm not sure that there's a nuke UI widget rich
enough for this behavior.   What we want is a popdown that lets you
choose from a bunch of pre-defined options, but also lets you type in
new values (such as envvars).  Im not sure that nuke popdowns can
allow text entry capability.  The other difficulty is that in nuke if
you use the popdowns, the underlying value is an int index rather than
a string.  This sucks because if you were to update the array, change
the underlying value, etc, the only precious data for nuke is what
order it was in the list. (Whereas in our case the value is precious).
So I think it's safer to keep it a string for now.   But... what if
we added a button that when pressed provided a popup for the user?
(Calling out to OCIO to enumerate the options).  That would make it
convenient + keep data value 'native'.

I like the use of the invisible flag. I'll investigate setting it up.

-- Jeremy

Join ocio-dev@lists.aswf.io to automatically receive all group messages.