Definitely. Can you prepare a real PR with the patch that seems to work? I'll do some digging to try to figure out what broke. Did it used to be like this and we broke something?
> On Sep 3, 2019, at 5:32 PM, Renaud Talon <[email protected]> wrote: > > Hi Michael, > > your patch absolutely works ! My benchmark goes from 60s+ to 700ms !!! > > I was getting corrupted images with my test code at first because I wasn't > passing the image spec.format to the "read_image()" method. (which worked > fine with OIIO 1.8.5 but is clearly needed to use the DPXInput::read_image > patch you provided) > > I changed : > pixels = srcFile.read_image() > > to: > pixels = srcFile.read_image(spec.format) > > and it worked like a charm. > > Larry, if you're reading this. Could we get Michael's patch or a similar fix > merged into OIIO code please ? > Thanks, > > Renaud > _______________________________________________ > 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]
_______________________________________________ Oiio-dev mailing list [email protected] http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org
