Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-11-01 Thread Ronald S. Bultje
Hi, On Tue, Nov 1, 2016 at 8:09 AM, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 31, 2016 at 5:51 PM, Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > >> On Mon, Oct 31, 2016 at 4:08 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Mon, Oct 31, 2016 at 3:31 PM, Vittorio Giovara >>

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-11-01 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 5:51 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > On Mon, Oct 31, 2016 at 4:08 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Mon, Oct 31, 2016 at 3:31 PM, Vittorio Giovara > > wrote: > >> > >> On Mon, Oct 31, 2016 at 2:53 PM, Ronald S. Bultje > >>

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Vittorio Giovara
On Mon, Oct 31, 2016 at 4:08 PM, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 31, 2016 at 3:31 PM, Vittorio Giovara > wrote: >> >> On Mon, Oct 31, 2016 at 2:53 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Mon, Oct 31, 2016 at 11:13 AM, Vittorio Giovara >> > wrote: >> >> >> >> On Mon, O

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 3:31 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > On Mon, Oct 31, 2016 at 2:53 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Mon, Oct 31, 2016 at 11:13 AM, Vittorio Giovara > > wrote: > >> > >> On Mon, Oct 31, 2016 at 7:43 AM, Ronald S. Bultje > >>

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Vittorio Giovara
On Mon, Oct 31, 2016 at 2:53 PM, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 31, 2016 at 11:13 AM, Vittorio Giovara > wrote: >> >> On Mon, Oct 31, 2016 at 7:43 AM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Mon, Oct 31, 2016 at 5:50 AM, Kevin Wheatley >> > >> > wrote: >> >> >> >> On Sun

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 11:13 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > On Mon, Oct 31, 2016 at 7:43 AM, Ronald S. Bultje > wrote: > > Hi, > > > > On Mon, Oct 31, 2016 at 5:50 AM, Kevin Wheatley < > kevin.j.wheat...@gmail.com> > > wrote: > >> > >> On Sun, Oct 30, 2016 at 1:

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Kevin Wheatley
I would really strongly suggest including DCI in the name at least - though nobody else would choose to use it for anything other than the reference calibration - most titles use a creative white different to that of the encoding reference (one that is less green). Kevin __

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Vittorio Giovara
On Mon, Oct 31, 2016 at 7:43 AM, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 31, 2016 at 5:50 AM, Kevin Wheatley > wrote: >> >> On Sun, Oct 30, 2016 at 1:18 PM, Ronald S. Bultje >> wrote: >> > Hmm... So, the wikipedia page https://en.wikipedia.org/wiki/DCI-P3 >> > refers >> > to the two whitep

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 5:50 AM, Kevin Wheatley wrote: > On Sun, Oct 30, 2016 at 1:18 PM, Ronald S. Bultje > wrote: > > Hmm... So, the wikipedia page https://en.wikipedia.org/wiki/DCI-P3 > refers > > to the two whitepoints here as DCI-P3 D65 and DCI-P3 Theater. Calling one > > D65 and the o

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Kevin Wheatley
On Sun, Oct 30, 2016 at 1:18 PM, Ronald S. Bultje wrote: > Hmm... So, the wikipedia page https://en.wikipedia.org/wiki/DCI-P3 refers > to the two whitepoints here as DCI-P3 D65 and DCI-P3 Theater. Calling one > D65 and the other DCI seems confusing in that light (assuming the wikipedia > page is c

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-30 Thread Ronald S. Bultje
Hi Kieran, On Sun, Oct 30, 2016 at 10:06 AM, Kieran O Leary wrote: > Hi, > > On Sun, Oct 30, 2016 at 7:07 AM, Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > > > Signed-off-by: Vittorio Giovara > > --- > > I couldn't find any reference to the name of the whitepoint used for 431, > > s

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-30 Thread Kieran O Leary
Hi, On Sun, Oct 30, 2016 at 7:07 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > Signed-off-by: Vittorio Giovara > --- > I couldn't find any reference to the name of the whitepoint used for 431, > so I came up with DCI, since it looks like it is only used there. > Please CC. > Could

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-30 Thread Ronald S. Bultje
Hi, On Sun, Oct 30, 2016 at 3:07 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > Signed-off-by: Vittorio Giovara > --- > I couldn't find any reference to the name of the whitepoint used for 431, > so I came up with DCI, since it looks like it is only used there. > Please CC. > Vittor

[FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-30 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara --- I couldn't find any reference to the name of the whitepoint used for 431, so I came up with DCI, since it looks like it is only used there. Please CC. Vittorio libavfilter/vf_colorspace.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavfilter/vf