DPX writing should support endian switch

[jeremyselan]

DPX files come in two endian flavors, and all systems are expected to read both. However, for hardware compatibility purposes it's often preferable to force Big Endian. The dpx writer currently does not allow the writing endian to be toggle at runtime. We should add writer support for an optional hint such as oiio:Endian, to control the output.

See this previous email chain on oiio-dev, "DPX Endian", 12/08/10.

[sanjay-nambiar]

Hi, I would like to work on this issue. I want to particiate in GSOC 2011 OpenImageIO projects so fixing this issue will give me the prerequisite patch. I have good C/C++ knowledge and a decent knowledge on Image processing and Computer Graphics. Im new to contributing to oen source software, please help me on how to do this.

[inequation]

So was there any agreement on the attribute name? If there is a consensus, I can code it in.

[jeremyselan]

Let's go with oiio:Endian for now.
It can have values: "big", "little", or "native".

The writer will obey the value, if set. And I think DPX's writer should default to 'Big' if not specified. (Which is the default used in many other linux packages).

We can always adjust the defaults in the future, I think 99% of the work will just be getting the code path in there.

Thanks!

[inequation]

Jeremy, please see #69. As far as I can see it should do the trick.

[jeremyselan]

thanks - ill check it out

On Wed, Apr 20, 2011 at 1:52 AM, inequation
reply@reply.github.com
wrote:

Jeremy, please see #69. As far as I can see it should do the trick.

Reply to this email directly or view it on GitHub:
#36 (comment)