[FFmpeg-devel] [PATCH] Add a gamma flag to exr loader to avoid banding

Clément Bœsch u at pkh.me
Tue Apr 29 10:30:04 CEST 2014


On Tue, Apr 29, 2014 at 05:05:10AM -0300, Gonzalo Garramuno wrote:
> On 28/04/14 21:01, Michael Niedermayer wrote:
> >On Mon, Apr 28, 2014 at 08:21:03PM -0300, Gonzalo Garramuno wrote:
> >>On 28/04/14 17:10, Michael Niedermayer wrote:
> >>>On Mon, Apr 28, 2014 at 09:19:07PM +0200, Reimar Döffinger wrote:
> >>>
> >>>>>patience
> >>I apologize for being so impatient.  Sorry.
> >>>>>First it seems to still have a bug
> >>>>>it fails fate
> >>>>>--- ./tests/ref/fate/exr-slice-raw      2014-04-28 04:14:37.849375679 +0200
> >>>>>+++ tests/data/fate/exr-slice-raw       2014-04-28 19:50:27.430558597 +0200
> >>>>>@@ -1,2 +1,2 @@
> >>>>>#tb 0: 1/25
> >>>>>-0,          0,          0,        1,  3169800, 0x6a356d0d
> >>>>>+0,          0,          0,        1,  3169800, 0x2cfefd25
> >>I am stuck with this bug.  Not sure exactly what the numbers mean in
> >>the first place, except that it seems +0 and -0 are misplaced.
> >the data outputed by the decoder changes, which changes the checksum
> >
> >Either the decoder should not change in output by default
> >or
> >the checksum should be updated
> >or
> >the command line options for the tests would need to be changed so
> >as to maintain the same output in the test
> >
> >
> >[...]
> >
> Thanks Michael.  I understand how the checksum works.  What I don't
> understand is where is the  framecrc command and what does its 5 numbers of
> output mean.

make fate-exr-slice-raw V=1 will give you the command
see libavformat/framecrcenc.c for column values

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140429/30be5ecd/attachment.asc>


More information about the ffmpeg-devel mailing list