As found in HEVC.
I put the code in mastering_display_metadata as they usually go together in
Blu-Ray UHD sources.
--
update the previous patch:
- rename CEA 861.3 to CTA-861.3
- use MaxCLL and MaxFALL names that are more commonly found
- use unsigned integer rather than rational numbers as (supp
These data are necessary when transmitting HDR over HDMI.
--
update the previous patch:
- rename CEA 861.3 to CTA-861.3
- update with MaxCLL/MaxFALL changes in avutil
update the previous patch:
- rebased and updated version bump
---
libavcodec/avcodec.h | 7 +++
libavcodec/avpacket.c | 1 +
Hi. Thanks for the patch.
Le quartidi 14 germinal, an CCXXV, Timothy Lee a écrit :
> Capture is an input stream capture protocol that dumps the input stream to a
> file. The default name of the output file is "capture.dat", but it can be
> changed using the "capture_file" option.
>
> capture.c
On 04/03/2017 06:35 PM, Nicolas George wrote:
Hi. Thanks for the patch.
Le quartidi 14 germinal, an CCXXV, Timothy Lee a écrit :
Capture is an input stream capture protocol that dumps the input stream to a
file. The default name of the output file is "capture.dat", but it can be
changed using
On Fri, Mar 31, 2017 at 7:15 PM, James Almer wrote:
> On 3/31/2017 5:42 AM, Vittorio Giovara wrote:
>> On Wed, Mar 29, 2017 at 4:55 AM, James Almer wrote:
>>> Signed-off-by: James Almer
>>> ---
>>> doc/APIchanges| 3 +++
>>> libavutil/spherical.c | 15 +++
>>> libavutil/sph
On Mon, Apr 03, 2017 at 07:10:42PM +1000, Timothy Lee wrote:
> On 04/03/2017 06:35 PM, Nicolas George wrote:
> >Hi. Thanks for the patch.
> >
> >Le quartidi 14 germinal, an CCXXV, Timothy Lee a écrit :
> >>Capture is an input stream capture protocol that dumps the input stream to a
> >>file. The d
Ping
From: Eran Kornblau
Sent: Tuesday, March 28, 2017 3:13 PM
To: FFmpeg development discussions and patches
Subject: Match source video timestamp
Hi all,
I'm trying to transcode some video file (MP4/h264 baseline vfr/AAC) with
ffmpeg, and I would like to have the
frame timestamps in the tra
> For both positive and negative numbers.
>
> 3 / 2 = 1
> -3 / 2 = -1
You’re right. I was confusing integer division by a power of two, with
right-shifting. The bug was in my code all along.
I apologise for wasting your time.
Christina
___
ffmpeg-de
Le quartidi 14 germinal, an CCXXV, Timothy Lee a écrit :
> Thanks for your quick reply. Regarding the almost direct copy of code from
> cache.c, I previously submitted a patch on 31 March that adds a "cache_file"
> option to the cache protocol. It was intended to allow a specifically named
> cach
On Sun, Apr 02, 2017 at 10:45:41PM -0300, James Almer wrote:
> Signed-off-by: James Almer
> ---
> libavcodec/hevc_parse.c | 32 +---
> 1 file changed, 25 insertions(+), 7 deletions(-)
>
> diff --git a/libavcodec/hevc_parse.c b/libavcodec/hevc_parse.c
> index 6c1138e01
2017-04-01 16:37 GMT+02:00 Ricardo Constantino :
> On 1 April 2017 at 14:03, Hendrik Leppkes wrote:
>
>>
>> This is the Boost Software license, very similar to BSD/MIT, so if
>> that license is all that matters, its obviously fine.
>>
>>
> Just downloaded the latest 10.8.5 SDK and checked every fi
On Sun, Apr 02, 2017 at 10:45:43PM -0300, James Almer wrote:
> While they shouldn't be present, they are harmless if they are.
>
> Signed-off-by: James Almer
> ---
> libavcodec/hevc_parse.c | 21 ++---
> 1 file changed, 2 insertions(+), 19 deletions(-)
should be ok but maybe wai
2017-04-02 21:15 GMT+02:00 Michael Niedermayer :
> i intend to make 3.3 from this branch within a few days
Please wait for a resolution for the decklink license issue:
Either the library is free (as explained here) or non-free
as was claimed on the Debian bugtracker.
Carl Eugen
_
On Mon, 3 Apr 2017 11:25:39 +0200
Michael Niedermayer wrote:
> On Mon, Apr 03, 2017 at 07:10:42PM +1000, Timothy Lee wrote:
> > On 04/03/2017 06:35 PM, Nicolas George wrote:
> > >Hi. Thanks for the patch.
> > >
> > >Le quartidi 14 germinal, an CCXXV, Timothy Lee a écrit :
> > >>Capture is an
On Mon, 3 Apr 2017 10:31:15 +0100
Christina Brien wrote:
> > For both positive and negative numbers.
> >
> > 3 / 2 = 1
> > -3 / 2 = -1
>
> You’re right. I was confusing integer division by a power of two, with
> right-shifting. The bug was in my code all along.
>
> I apologise for wasting y
Signed-off-by: Nicolas George
---
doc/muxers.texi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
In principle I would rather fix the default value, but that may break a lot
of users' workflow.
diff --git a/doc/muxers.texi b/doc/muxers.texi
index 166c929369..844bbce990 100644
--- a/doc/
2017-04-03 18:12 GMT+08:00 wm4 :
> On Mon, 3 Apr 2017 11:25:39 +0200
> Michael Niedermayer wrote:
>
> > On Mon, Apr 03, 2017 at 07:10:42PM +1000, Timothy Lee wrote:
> > > On 04/03/2017 06:35 PM, Nicolas George wrote:
> > > >Hi. Thanks for the patch.
> > > >
> > > >Le quartidi 14 germinal, an CCXX
On Mon, 3 Apr 2017 18:20:35 +0800
Steven Liu wrote:
> 2017-04-03 18:12 GMT+08:00 wm4 :
>
> > On Mon, 3 Apr 2017 11:25:39 +0200
> > Michael Niedermayer wrote:
> >
> > > On Mon, Apr 03, 2017 at 07:10:42PM +1000, Timothy Lee wrote:
> > > > On 04/03/2017 06:35 PM, Nicolas George wrote:
> > >
On 3 April 2017 at 10:56, Carl Eugen Hoyos wrote:
> 2017-04-01 16:37 GMT+02:00 Ricardo Constantino :
> > On 1 April 2017 at 14:03, Hendrik Leppkes wrote:
> >
> >>
> >> This is the Boost Software license, very similar to BSD/MIT, so if
> >> that license is all that matters, its obviously fine.
>
Le quartidi 14 germinal, an CCXXV, Ricardo Constantino a écrit :
> I was talking about the headers, which is the only part that matters to
> FFmpeg.
I do not know the specifics, so just for references:
The license that matter is the license of ALL the files that are needed
at build time.
> The S
Le quartidi 14 germinal, an CCXXV, Steven Liu a écrit :
> change name from av_strreplace to av_strireplace
> Use AVBprint to implement av_strireplace
> add av_strireplace test case TEST_STRIREPLACE
>
> Signed-off-by: Steven Liu
> ---
> libavutil/avstring.c | 76
> +++--
---
doc/examples/extract_mvs.c | 56 +++---
1 file changed, 28 insertions(+), 28 deletions(-)
diff --git a/doc/examples/extract_mvs.c b/doc/examples/extract_mvs.c
index 552a7334cf..7ae934ead3 100644
--- a/doc/examples/extract_mvs.c
+++ b/doc/examples/extrac
---
doc/examples/extract_mvs.c | 72 ++
1 file changed, 35 insertions(+), 37 deletions(-)
diff --git a/doc/examples/extract_mvs.c b/doc/examples/extract_mvs.c
index 8b22b40c11..d6730db3a2 100644
--- a/doc/examples/extract_mvs.c
+++ b/doc/examples/extrac
---
doc/examples/extract_mvs.c | 18 ++
1 file changed, 6 insertions(+), 12 deletions(-)
diff --git a/doc/examples/extract_mvs.c b/doc/examples/extract_mvs.c
index d6730db3a2..552a7334cf 100644
--- a/doc/examples/extract_mvs.c
+++ b/doc/examples/extract_mvs.c
@@ -31,13 +31,11 @@ s
Makes the reason of the "FIXME" comment more obvious.
Avoid name conflicts for the next commit.
Signed-off-by: Nicolas George
---
ffmpeg.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/ffmpeg.c b/ffmpeg.c
index 11faf0d4a8..02ff073615 100644
--- a/ffmpeg.c
+++ b
Signed-off-by: Nicolas George
---
ffmpeg.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
Not sure if this is correct, the timestamp handling is awfully complicated.
But FATE succeeds and the result is a correct value of next_pts after the
last frame with a VFR file.
diff --git
On Mon, 3 Apr 2017 14:52:15 +0200
Nicolas George wrote:
> Signed-off-by: Nicolas George
> ---
> ffmpeg.c | 8 +---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
>
> Not sure if this is correct, the timestamp handling is awfully complicated.
> But FATE succeeds and the result is a c
On 4/3/2017 7:00 AM, Michael Niedermayer wrote:
> On Sun, Apr 02, 2017 at 10:45:41PM -0300, James Almer wrote:
>> Signed-off-by: James Almer
>> ---
>> libavcodec/hevc_parse.c | 32 +---
>> 1 file changed, 25 insertions(+), 7 deletions(-)
>>
>> diff --git a/libavcodec/h
Fixes a reported (but false) race condition in tsan for fate-apng:
WARNING: ThreadSanitizer: data race (pid=6274)
Read of size 4 at 0x7d680001ec78 by main thread (mutexes: write M1338):
#0 update_thread_context src/libavcodec/pngdec.c:1456
(ffmpeg+0x00dacf0c)
[..]
Previous write of si
This tries to handle cases where separate invocations of decode_frame()
(each running in separate threads) write to respective fields in the
same AVFrame->data[]. Having per-field owners makes interaction between
readers (the referencing thread) and writers (the decoding thread)
slightly more optim
A new filter ANSNR is added. libavfilter/Makefile is changed.
Run 'ffmpeg -i input1 -i input2 -lavfi ansnr -f null -' to get an overall score
while per-frame value is stored but not printed.
This implementation is for constructing the vmaf filter later since ANSNR is
one of individual tools used
2017-04-03 23:00 GMT+08:00 Betty Wu :
> A new filter ANSNR is added. libavfilter/Makefile is changed.
> Run 'ffmpeg -i input1 -i input2 -lavfi ansnr -f null -' to get an overall
> score while per-frame value is stored but not printed.
> This implementation is for constructing the vmaf filter later
On Mon, Apr 03, 2017 at 10:09:30AM -0400, Ronald S. Bultje wrote:
> Fixes a reported (but false) race condition in tsan for fate-apng:
>
> WARNING: ThreadSanitizer: data race (pid=6274)
> Read of size 4 at 0x7d680001ec78 by main thread (mutexes: write M1338):
> #0 update_thread_context src/l
2017-04-03 13:34 GMT+02:00 Ricardo Constantino :
> On 3 April 2017 at 10:56, Carl Eugen Hoyos wrote:
>
>> 2017-04-01 16:37 GMT+02:00 Ricardo Constantino :
>> > On 1 April 2017 at 14:03, Hendrik Leppkes wrote:
>> >
>> >>
>> >> This is the Boost Software license, very similar to BSD/MIT, so if
>> >
On Mon, Apr 3, 2017, at 02:14 AM, Nicolas George wrote:
> Signed-off-by: Nicolas George
> ---
> doc/muxers.texi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
LGTM
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman
Hi!
Attached patch fixes setting the language for the sample from vlc ticket
#10830: http://streams.videolan.org/issues/10830/vlc-audio-lang.ts
Please comment, Carl Eugen
From d8b30253cc6b1c2156f1e8898e0a0c2ff3c3f900 Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos
Date: Mon, 3 Apr 2017 19:03:29
On 4/3/17, Carl Eugen Hoyos wrote:
> Hi!
>
> Attached patch fixes setting the language for the sample from vlc ticket
> #10830: http://streams.videolan.org/issues/10830/vlc-audio-lang.ts
>
> Please comment, Carl Eugen
>
Breaks indentation, nak.
___
ffmp
On 2 April 2017 at 22:13, Paul B Mahol wrote:
> On 4/2/17, Paul B Mahol wrote:
> > Signed-off-by: Paul B Mahol
> > ---
> > doc/filters.texi | 3 +++
> > libavfilter/vf_pad.c | 14 ++
> > 2 files changed, 17 insertions(+)
> >
> > +if (adjusted_aspect.num && adjusted_aspect.
On 4/3/17, Ricardo Constantino wrote:
> On 2 April 2017 at 22:13, Paul B Mahol wrote:
>
>> On 4/2/17, Paul B Mahol wrote:
>> > Signed-off-by: Paul B Mahol
>> > ---
>> > doc/filters.texi | 3 +++
>> > libavfilter/vf_pad.c | 14 ++
>> > 2 files changed, 17 insertions(+)
>> >
>>
or if x/y go beyond padded area.
This is mostly useful when paired with the aspect option.
Defaults aren't changed.
Idea for this was taken from mpv's soon-to-be-removed expand vf.
---
doc/filters.texi | 3 +++
libavfilter/vf_pad.c | 12 +++-
2 files changed, 10 insertions(+), 5 del
Fixes tsan warnings in fate-apng:
WARNING: ThreadSanitizer: data race (pid=51230)
Read of size 4 at 0x7d5042fc by main thread (mutexes: write M1000):
#0 frame_copy_props frame.c:302 (ffmpeg:x86_64+0x1019a35d6)
[..]
Previous write of size 4 at 0x7d5042fc by thread T1 (mutexes: write
On 3 April 2017 at 19:18, Paul B Mahol wrote:
> On 4/3/17, Ricardo Constantino wrote:
> > On 2 April 2017 at 22:13, Paul B Mahol wrote:
> >
> >> On 4/2/17, Paul B Mahol wrote:
> >> > Signed-off-by: Paul B Mahol
> >> > ---
> >> > doc/filters.texi | 3 +++
> >> > libavfilter/vf_pad.c | 14
On 4/3/17, Ronald S. Bultje wrote:
> Fixes tsan warnings in fate-apng:
>
> WARNING: ThreadSanitizer: data race (pid=51230)
> Read of size 4 at 0x7d5042fc by main thread (mutexes: write M1000):
> #0 frame_copy_props frame.c:302 (ffmpeg:x86_64+0x1019a35d6)
> [..]
> Previous write of size
On 4/3/17, Ricardo Constantino wrote:
> On 3 April 2017 at 19:18, Paul B Mahol wrote:
>
>> On 4/3/17, Ricardo Constantino wrote:
>> > On 2 April 2017 at 22:13, Paul B Mahol wrote:
>> >
>> >> On 4/2/17, Paul B Mahol wrote:
>> >> > Signed-off-by: Paul B Mahol
>> >> > ---
>> >> > doc/filters.te
On 3 April 2017 at 19:58, Paul B Mahol wrote:
>
> Yes, but that conflict with expand behaviour.
>
>
How so? Still works fine with squared pixels:
$ ffmpeg -f lavfi -i "color=s=hd720" -vf pad=aspect=4/3 -vframes 1 -f null
- -v verbose 2>&1 | grep Parsed_pad
[Parsed_pad_0 @ 01c953343940] w:1280
On 4/3/17, Ricardo Constantino wrote:
> On 3 April 2017 at 19:58, Paul B Mahol wrote:
>
>>
>> Yes, but that conflict with expand behaviour.
>>
>>
> How so? Still works fine with squared pixels:
> $ ffmpeg -f lavfi -i "color=s=hd720" -vf pad=aspect=4/3 -vframes 1 -f null
> - -v verbose 2>&1 | grep
On 3 April 2017 at 20:04, Paul B Mahol wrote:
> On 4/3/17, Ricardo Constantino wrote:
> > On 3 April 2017 at 19:58, Paul B Mahol wrote:
> >
> >>
> >> Yes, but that conflict with expand behaviour.
> >>
> >>
> > How so? Still works fine with squared pixels:
> > $ ffmpeg -f lavfi -i "color=s=hd720
Hi Betty,
On Mon, Apr 3, 2017 at 11:00 AM, Betty Wu wrote:
> +typedef double number_t;
>
Why?
+static int ansnr_filter(const uint8_t* src_image, number_t* dst_image, int
> img_row, int img_col, const double *filter, int stride)
>
[..]
> +number_t **imme_image;
> +if(!(imme_image = (nu
2017-04-03 20:13 GMT+02:00 Paul B Mahol :
> On 4/3/17, Carl Eugen Hoyos wrote:
>> Hi!
>>
>> Attached patch fixes setting the language for the sample from vlc ticket
>> #10830: http://streams.videolan.org/issues/10830/vlc-audio-lang.ts
>>
>> Please comment, Carl Eugen
>
> Breaks indentation, nak.
Hi,
This patch adds copy of HDR VUI info from decode ctx to encode ctx. Currently,
information under colour_description_present_flag (eg. colour primaries,
transfer_characteristics, matrix_coeffs) do not get copied to output stream
when trancode happens.
Testing performed:
ffmpeg.exe -y -hwac
On 4/3/17, Carl Eugen Hoyos wrote:
> 2017-04-03 20:13 GMT+02:00 Paul B Mahol :
>> On 4/3/17, Carl Eugen Hoyos wrote:
>>> Hi!
>>>
>>> Attached patch fixes setting the language for the sample from vlc ticket
>>> #10830: http://streams.videolan.org/issues/10830/vlc-audio-lang.ts
>>>
>>> Please comme
Isn't supported.
Signed-off-by: Rostislav Pehlivanov
---
libavcodec/mpegvideo_enc.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c
index 71a858fc72..8cc964be23 100644
--- a/libavcodec/mpegvideo_enc.c
+++ b/libavcodec/mpegvideo_enc
When coding lossless jpeg the priv context will be pointing to LJpegEncContext
rather than MpegEncContext, which the function expects.
Signed-off-by: Rostislav Pehlivanov
---
libavcodec/mjpegenc_common.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/libavcodec/mj
As it gives excellent encoding gains at an insignificant speed increase
and passes fate without problems, it should now be safe to enable by
default.
Signed-off-by: Rostislav Pehlivanov
---
libavcodec/mjpegenc.c| 2 +-
tests/ref/vsynth/vsynth1-mjpeg | 4 ++--
tests/
On 4/3/17, Ricardo Constantino wrote:
> On 3 April 2017 at 20:04, Paul B Mahol wrote:
>
>> On 4/3/17, Ricardo Constantino wrote:
>> > On 3 April 2017 at 19:58, Paul B Mahol wrote:
>> >
>> >>
>> >> Yes, but that conflict with expand behaviour.
>> >>
>> >>
>> > How so? Still works fine with squar
On Mon, 3 Apr 2017, Carl Eugen Hoyos wrote:
Hi!
Attached patch fixes setting the language for the sample from vlc ticket
#10830: http://streams.videolan.org/issues/10830/vlc-audio-lang.ts
Please comment, Carl Eugen
From d8b30253cc6b1c2156f1e8898e0a0c2ff3c3f900 Mon Sep 17 00:00:00 2001
From:
On Sun, Apr 02, 2017 at 07:30:00AM -0400, Ronald S. Bultje wrote:
> Hi,
>
> On Sat, Apr 1, 2017 at 8:19 PM, Michael Niedermayer
> wrote:
>
> > Supporting changing delay and number of threads at runtime was a
> > big factor in the design of the frame thread encoder. Its something
> > i cared abou
As it gives excellent encoding gains at an insignificant speed increase
and passes fate without problems, it should now be safe to enable by
default.
Signed-off-by: Rostislav Pehlivanov
---
Had to add some missing tests
---
libavcodec/mjpegenc.c| 2 +-
tests/ref/lavf/jpg
On Apr 2, 2017 5:21 PM, "Michael Niedermayer"
wrote:
On Sun, Apr 02, 2017 at 03:27:57PM -0700, Thomas Turner wrote:
> Reference found at: http://bit.ly/2ogQygb
>
> Signed-off-by: Thomas Turner
> ---
> tests/fate/filter-video.mak | 7 +++
> 1 file changed, 7 insertions(+)
reference file upl
On Mon, Apr 03, 2017 at 02:32:15PM -0700, Thomas Turner wrote:
> On Apr 2, 2017 5:21 PM, "Michael Niedermayer"
> wrote:
>
> On Sun, Apr 02, 2017 at 03:27:57PM -0700, Thomas Turner wrote:
> > Reference found at: http://bit.ly/2ogQygb
> >
> > Signed-off-by: Thomas Turner
> > ---
> > tests/fate/fi
On Sat, Apr 01, 2017 at 07:18:36PM +0200, Michael Niedermayer wrote:
> Fixes integer overflow
> Fixes: 701640
>
> Found-by: Found-by: Thomas Guilbert
> Signed-off-by: Michael Niedermayer
> ---
> libavformat/mov.c | 6 ++
> 1 file changed, 6 insertions(+)
applied
[...]
--
Michael GnuP
2017-04-03 22:13 GMT+02:00 Rostislav Pehlivanov :
> +if (s->huffman && avctx->codec_id == AV_CODEC_ID_AMV) {
> +av_log(avctx, AV_LOG_WARNING, "AMV doesn't support optimized huffman
> tables, disabling\n");
Iiuc, together with 3/3 this prints a warning by default when encoding
AMV, th
On 4 April 2017 at 00:02, Carl Eugen Hoyos wrote:
> 2017-04-03 22:13 GMT+02:00 Rostislav Pehlivanov :
>
> > +if (s->huffman && avctx->codec_id == AV_CODEC_ID_AMV) {
> > +av_log(avctx, AV_LOG_WARNING, "AMV doesn't support optimized
> huffman tables, disabling\n");
>
> Iiuc, together wi
On Mon, Apr 03, 2017 at 10:31:27PM +0100, Rostislav Pehlivanov wrote:
> As it gives excellent encoding gains at an insignificant speed increase
> and passes fate without problems, it should now be safe to enable by
> default.
Still breaks slice threading
both
Assertion n <= 31 && value < (1U << n
Fixes the following tsan warning when running fate-vsynth_lena-ffvhuff:
WARNING: ThreadSanitizer: data race (pid=6484)
Write of size 8 at 0x7d64000154b8 by main thread (mutexes: write M1331):
#0 update_context_from_user src/libavcodec/pthread_frame.c:331
(ffmpeg+0x00dca887)
[..]
Previ
On Mon, 3 Apr 2017 19:50:07 +
Ben Chang wrote:
> From c8092bc538ba7ae32357ca166e67809eb3705f60 Mon Sep 17 00:00:00 2001
> From: Ben Chang
> Date: Mon, 27 Mar 2017 17:00:19 -0700
> Subject: [PATCH] Allow transfer of HDR-required VUI info from dec ctx to enc
> ctx
>
> ---
> ffmpeg.c | 3 +++
Can do audio decoding, audio encoding, video decoding, video encoding,
video HW encoding. I also had video HW decoding, but removed it for now,
as thw hwframes integration wasn't very sane.
Some of the MS codecs aren't well tested, and might not work properly.
---
Didn't actually test compilation
On Fri, 31 Mar 2017 03:49:16 +0200
Michael Niedermayer wrote:
> On Thu, Mar 30, 2017 at 05:05:36PM +0200, wm4 wrote:
> > Not sure if it behaves ideally in presence of decoding errors.
> > ---
> > The original version of this patch is actually 1 year old.
> > ---
> > ffprobe.c | 42 ++
68 matches
Mail list logo
