Re: [FFmpeg-devel] [PATCH] avformat/movenc: Add support for more colorspaces

2017-12-14 Thread Steven Robertson
either. I presume the same behavior applies to very old versions of QuickTime looking at these new codepoints. On Dec 14, 2017 6:51 PM, "Carl Eugen Hoyos" <ceffm...@gmail.com> wrote: > 2017-12-14 20:52 GMT+01:00 Steven Robertson <ste...@strobe.cc>: > > With FCPX 1

[FFmpeg-devel] [PATCH] avformat/movenc: Add support for more colorspaces

2017-12-14 Thread Steven Robertson
With FCPX 10.4, Apple has expanded the set of colorspace, primaries, and trc flags officially supported in QuickTime files. The expanded set matches the codepoints used in ffmpeg and many other specs. --- libavformat/movenc.c | 25 - 1 file changed, 16 insertions(+), 9

[FFmpeg-devel] [PATCH] avcodec/dnxhddec: Do not overwrite colorspace if the container has set it.

2017-11-28 Thread Steven Robertson
information. This patch maintains the existing behavior of choosing the 709 over the 601 matrix when container-level information is missing, and allows container-level information to win if present. Signed-off-by: Steven Robertson <ste...@strobe.cc> --- libavcodec/dnxhddec.c | 4 +++-

Re: [FFmpeg-devel] [PATCH] [dnxhddec] Do not overwrite colorspace if the container has set it.

2017-11-27 Thread Steven Robertson
2020NCL for HDR files work properly. On Mon, Nov 27, 2017 at 4:35 PM, Michael Niedermayer <mich...@niedermayer.cc > wrote: > On Mon, Nov 27, 2017 at 12:36:48AM -0800, Steven Robertson wrote: > > Signed-off-by: Steven Robertson <ste...@strobe.cc> > > --- > > liba

[FFmpeg-devel] [PATCH] [dnxhddec] Do not overwrite colorspace if the container has set it.

2017-11-27 Thread Steven Robertson
Signed-off-by: Steven Robertson <ste...@strobe.cc> --- libavcodec/dnxhddec.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c index f46e41a456..6f8c716412 100644 --- a/libavcodec/dnxhddec.c +++ b/libavcodec/dnxhddec.c @@

[FFmpeg-devel] [PATCH 2/2] [avformat/mxf] Add support for ST 2084 / ST 2067-21 mastering metadata.

2016-09-23 Thread Steven Robertson
Signed-off-by: Steven Robertson <ste...@strobe.cc> --- libavformat/mxfdec.c | 132 --- libavformat/mxfenc.c | 43 - 2 files changed, 145 insertions(+), 30 deletions(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c

[FFmpeg-devel] [PATCH 1/2] [avformat/mxf] Add core colorimetry support to MXF.

2016-09-23 Thread Steven Robertson
Signed-off-by: Steven Robertson <ste...@strobe.cc> --- libavformat/mxf.c| 33 + libavformat/mxf.h| 3 +++ libavformat/mxfdec.c | 29 + libavformat/mxfenc.c | 36 +++- 4 files chang

Re: [FFmpeg-devel] [PATCH] libavformat/mov: Accept known codepoints in 'colr'

2016-08-20 Thread Steven Robertson
. 2020 color in MOV, even though TN 2162 hasn't been updated yet.) Also it'd be nice to land this for the bugfix in SD. Anything I can do to help with the review? Thanks! Steve On Wed, Aug 17, 2016 at 12:25 AM, Steven Robertson <ste...@strobe.cc> wrote: > This change relaxes the whitelist o

[FFmpeg-devel] [PATCH] libavformat/mov: Accept known codepoints in 'colr'

2016-08-17 Thread Steven Robertson
ist will still kick in, so this won't result in out-of-spec files being created. [1] https://developer.apple.com/library/mac/technotes/tn2162/_index.html Signed-off-by: Steven Robertson <ste...@strobe.cc> --- libavformat/mov.c | 40 +--- 1 file changed, 9 in

[FFmpeg-devel] [PATCH] fate: add DNxHR 12-bit example.

2016-08-16 Thread Steven Robertson
Signed-off-by: Steven Robertson <ste...@strobe.cc> --- tests/fate/dnxhd.mak | 2 ++ tests/ref/fate/dnxhr-12bit | 6 ++ 2 files changed, 8 insertions(+) create mode 100644 tests/ref/fate/dnxhr-12bit diff --git a/tests/fate/dnxhd.mak b/tests/fate/dnxhd.mak index 4008e6c..60bfd34

Re: [FFmpeg-devel] [PATCH] fate: add DNxHR 12-bit example.

2016-08-16 Thread Steven Robertson
Yes, thank you. I'll resend this. On Wed, Aug 3, 2016 at 2:01 PM, Mark Reid <mindm...@gmail.com> wrote: > On Tue, Aug 2, 2016 at 11:51 PM, Steven Robertson <ste...@strobe.cc> > wrote: > > Test file available at http://tinyurl.com/fate-dnxhd-12bit . > > > >

[FFmpeg-devel] fate: add DNxHR 12-bit example.

2016-08-03 Thread Steven Robertson
Test file available at http://tinyurl.com/fate-dnxhd-12bit . ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] fate: add DNxHR 12-bit example.

2016-08-03 Thread Steven Robertson
Test file available at http://tinyurl.com/fate-dnxhd-12bit . Signed-off-by: Steven Robertson <ste...@strobe.cc> --- tests/fate/dnxhd.mak | 2 ++ tests/ref/fate/dnxhr-12bit | 6 ++ 2 files changed, 8 insertions(+) create mode 100644 tests/ref/fate/dnxhr-12bit diff --git a/test

[FFmpeg-devel] [PATCH] libavcodec/dnxhd: Enable 12-bit DNxHR support.

2016-08-02 Thread Steven Robertson
A sample file will be uploaded to show that with this patch, things decode correctly: dnxhr_hqx_12bit_1080p_smpte_colorbars_davinci_resolve.mov Signed-off-by: Steven Robertson <ste...@strobe.cc> --- libavcodec/dnxhddec.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --

[FFmpeg-devel] [PATCH] libavcodec/dnxhd: Enable 12-bit DNxHR support.

2016-08-02 Thread Steven Robertson
and will upload a reference file (Google Drive no-sign-in-required link below) as soon as the upload server's back up. https://drive.google.com/folderview?id=0B1w4_XzCPf1jRWhzU1lINl96d2s=sharing Thanks, Steve Steven Robertson (1): libavcodec/dnxhd: Enable 12-bit DNxHR support. libavcodec/dnxhddec.c

[FFmpeg-devel] [PATCH] libavcodec/dnxhd: Enable 12-bit DNxHR support.

2016-08-01 Thread Steven Robertson
Hey there, DNxHR support for 12 bits wasn't waiting on a VLC table after all - there was just an improper dependence on the value of bitdepth. I've attached a patch and will upload a reference file (Google Drive no-sign-in-required link below) as soon as the upload server's back up.

Re: [FFmpeg-devel] [PATCH] vf_lut: Add support for RGB48 and RGBA64

2015-10-25 Thread Steven Robertson
Done, thanks. On Tue, Oct 13, 2015 at 12:51 AM, Paul B Mahol <one...@gmail.com> wrote: > On 10/11/15, Steven Robertson <ste...@strobe.cc> wrote: > > Thanks for taking a look! > > > > Steve > > > > lgtm, do you need to update fate? >