Because this works for me and passes the CI tests, I'm going to speculatively merge it. If it turns out that your difficulties are a bug and not something weird with your setup, we can always patch it.
> On Sep 13, 2016, at 2:28 PM, Larry Gritz <[email protected]> wrote: > > No match? Wha? > > Which version of OIIO? > > Just for fun, try enclosing the "-attrib:type=float[8]" in double quotes, in > case the shell is getting confused by the brackets. > > It works for me, BTW. > > >> On Sep 13, 2016, at 2:09 PM, Andrew Gartner <[email protected] >> <mailto:[email protected]>> wrote: >> >> Sure thing, here's what i get in my shell with this command. I'm guessing >> it's an argument syntax issue to be honest. (sorry I can't include the >> images for legal reasons). >> >> oiiotool input.exr -attrib:type=float[8] chromaticities >> "0.680,0.320,0.265,0.690,0.150,0.060,0.3127,0.3290" -o output.exr >> oiiotool: No match. >> >> >> >> On Tue, Sep 13, 2016 at 1:59 PM, Larry Gritz <[email protected] >> <mailto:[email protected]>> wrote: >> Not finding a match? Can you clarify? (Like, with a command line and exactly >> what the error says?) >> >> You're right, OpenEXR says the name should be lowercase, I will fix that in >> the patch. >> >> >> >>> On Sep 13, 2016, at 1:28 PM, Andrew Gartner <[email protected] >>> <mailto:[email protected]>> wrote: >>> >>> Hey Larry, >>> >>> Thanks for the super fast response. >>> >>> The patch worked from python like a charm once I set the type desc to the >>> attribute call to float[8]. I did notice it automatically capitalizes the >>> attribute name in the header on writing, is that a concern at all? >>> >>> Also oiiotool still complained about not finding a match when I tried your >>> above syntax but like I said, python worked fine. >>> >>> Cheers and thanks again, >>> >>> ~Andrew >>> >>> On Tue, Sep 13, 2016 at 12:23 PM, Larry Gritz <[email protected] >>> <mailto:[email protected]>> wrote: >>> Yep, sorry, this was subtly broken for OpenEXR. >>> >>> This patch should fix it: https://github.com/OpenImageIO/oiio/pull/1487 >>> <https://github.com/OpenImageIO/oiio/pull/1487> >>> >>> Let me know if you need that backported to a release branch. >>> >>> >>> >>> > On Sep 13, 2016, at 11:49 AM, Larry Gritz <[email protected] >>> > <mailto:[email protected]>> wrote: >>> > >>> > I think the correct syntax is: >>> > >>> > oiiotool input.exr -attrib:type=float[8] chromaticities "0,1,0,0,0,0,0,0" >>> > -o output.exr >>> > >>> > But upon trying it, I see that it's not quite working. Hang on, let me >>> > poke around a bit. OpenEXR may be particular about how the chromaticities >>> > are declared. >>> > >>> > >>> >> On Sep 13, 2016, at 11:35 AM, Andrew Gartner <[email protected] >>> >> <mailto:[email protected]>> wrote: >>> >> >>> >> Hey all, >>> >> >>> >> Has anyone tried to add chromaticies to an EXR by hand via oiiotool (or >>> >> python)? >>> >> >>> >> We're starting to expand our color support so for testing I'm trying to >>> >> add it by hand. I believe the attribute is a special case in the EXR >>> >> spec that needs 8 floats (in an array?). however trying this on the >>> >> command line didn't work (I had to wrap in quotes in which case it >>> >> becomes a string) >>> >> >>> >> For example: >>> >> >>> >> oiiotool input.exr -attrib [0,1,0,0,0,0,0,0] -o output.exr >>> >> >>> >> Any help would be welcome. >>> >> >>> >> Cheers, >>> >> >>> >> ~Andrew >>> >> >>> >> >>> >> >>> >> >>> >> _______________________________________________ >>> >> Oiio-dev mailing list >>> >> [email protected] <mailto:[email protected]> >>> >> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org >>> >> <http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org> >>> > >>> > -- >>> > Larry Gritz >>> > [email protected] <mailto:[email protected]> >>> > >>> > >>> > _______________________________________________ >>> > Oiio-dev mailing list >>> > [email protected] <mailto:[email protected]> >>> > http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org >>> > <http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org> >>> >>> -- >>> Larry Gritz >>> [email protected] <mailto:[email protected]> >>> >>> >>> _______________________________________________ >>> Oiio-dev mailing list >>> [email protected] <mailto:[email protected]> >>> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org >>> <http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org> >>> >>> _______________________________________________ >>> Oiio-dev mailing list >>> [email protected] <mailto:[email protected]> >>> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org >>> <http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org> >> >> -- >> Larry Gritz >> [email protected] <mailto:[email protected]> >> >> >> >> _______________________________________________ >> Oiio-dev mailing list >> [email protected] <mailto:[email protected]> >> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org >> <http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org> >> >> >> _______________________________________________ >> Oiio-dev mailing list >> [email protected] <mailto:[email protected]> >> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org > > -- > Larry Gritz > [email protected] <mailto:[email protected]> > > > _______________________________________________ > Oiio-dev mailing list > [email protected] > http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org -- Larry Gritz [email protected]
_______________________________________________ Oiio-dev mailing list [email protected] http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org
