Thanks for the test case. I briefly glanced at the code yesterday, and it's
structured as if it expects subimages to work in the DPX code. So it's
probably just a bug, not a case of it being written without considering
subimages at all and needs a total rewrite. I'll try to take a quick look and
see if it's something obvious, I'll let you know what I find.
-- lg
On Aug 14, 2013, at 1:34 AM, Mark Boorer wrote:
> Hi Larry,
>
> Thanks for your quick reply and willingness to take a look.
> I've attached an example file (its a test plate so no copyright issues). It
> was written using an internal, quite old library, and is fine for reading
> back by said library (it also seems okay in RV). I'm looking to transition
> new code over to OIIO, so as I mentioned, if you don't have time to tinker
> with it, I'm happy to take a stab :)
>
> File can be downloaded from here:
> https://docs.google.com/file/d/0B7tMukhco9MwVjZWa09ZWW04Vm8/edit?usp=sharing
>
> Thanks!
> Mark
>
>
> On Tue, Aug 13, 2013 at 8:20 PM, Larry Gritz <[email protected]> wrote:
> Nobody is working on it as far as I know. We use the DPX support extensively
> here, but I think our convention is to save L & R as separate images, so it
> hadn't been brought to my attention that our DPX subimage support was
> lacking. A fix for that would be very welcome.
>
> But before you dive in too deep... if you want to send me (privately is ok, I
> won't share) one of these stereo DPX files, I would be happy to take a
> *quick* look at the DPX code -- which I didn't write, but am at least
> passingly familiar with -- just to see if it's an extremely minor alteration
> (or even an obvious bug) to fix subimages, and if so I can just bang it out.
> But if it's more extensive, I'll let you know because if I don't know
> immediately how to do it, you'll probably get a fix faster if you are able to
> do it yourself rather than wait for me to free up a bigger block of time.
>
> -- lg
>
>
> On Aug 13, 2013, at 11:50 AM, Mark Boorer wrote:
>
> > Hi,
> >
> > I'm looking to read/write some stereo DPX files I have, and I noticed
> > that the DPXOutput class explicitly does not support subimages.
> > I've also tried reading some files that contain subimages (written
> > from a custom library), but I'm unable to get the data out of them
> > after calling seek_subimage() (though the seek call works as
> > expected).
> >
> > Is anyone presently working on supporting subimages in the DPX writer?
> > I'm not too familiar with the guts of the format, but I do have access
> > to the spec and may be motivated to add support.
> >
> > Cheers,
> > Mark
> >
>
> --
> Larry Gritz
> [email protected]
>
>
> _______________________________________________
> Oiio-dev mailing list
> [email protected]
> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org
>
> _______________________________________________
> 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
