On Mon, Jun 8, 2020 at 7:19 AM Tobias Rapp wrote:
> On 30.05.2020 12:41, Kieran O Leary wrote:
> > Hi,
> >
> > On Fri 29 May 2020, 22:47 Neil Birkbeck,
> wrote:
> >
> >> [...]
> >> I've changed the side data to be PixelCrop (instead of CleanA
On Mon, May 11, 2020 at 9:37 PM Neil Birkbeck
wrote:
>
>
> On Wed, May 6, 2020 at 8:45 AM James Almer wrote:
>
>> On 5/6/2020 12:22 PM, Neil Birkbeck wrote:
>> > On Tue, May 5, 2020 at 5:11 AM Kieran O Leary > >
>> > wrote:
>> >
>> >&
On Wed, May 6, 2020 at 8:45 AM James Almer wrote:
> On 5/6/2020 12:22 PM, Neil Birkbeck wrote:
> > On Tue, May 5, 2020 at 5:11 AM Kieran O Leary
> > wrote:
> >
> >> Hi,
> >>
> >> I broke the threading with my last reply, i apologise. Here goe
On Thu, May 7, 2020 at 10:21 AM Andreas Rheinhardt <
andreas.rheinha...@gmail.com> wrote:
> Neil Birkbeck:
> > On Tue, Apr 28, 2020 at 3:18 AM Nicolas George wrote:
> >
> >> Andreas Rheinhardt (12020-04-28):
> >>> That's expected. The patch
On Tue, May 5, 2020 at 5:11 AM Kieran O Leary
wrote:
> Hi,
>
> I broke the threading with my last reply, i apologise. Here goes another
> attempt:
>
> On Tue, Apr 28, 2020 at 6:23 PM Neil Birkbeck
> wrote:
>
> > On Tue, Apr 28, 2020 at 3:18 AM Nicolas George wrote
-A1 "Side"
Side data:
Clean aperture:[width 704/1 height:576/1 h_offset:0/1 v_offset:0/1]
From d92371cc2de6a671b44d9f6dd9d28544d05b5287 Mon Sep 17 00:00:00 2001
From: Neil Birkbeck
Date: Sun, 26 Apr 2020 20:42:59 -0700
Subject: [PATCH 3/3] libavformat: Adding MOV/MKV muxer supp
On Mon, Apr 27, 2020 at 4:38 AM Lynne wrote:
> Apr 27, 2020, 05:27 by neil.birkb...@gmail.com:
>
> > The clean aperature represents a cropping of the stored image data used
> to
> > relate the image data to a canonical video system and exists as container
> > metadata (see 'clap' section in
> htt
mpeg-devel/2016-March/192302.html
Transmuxing the side data can then be added (MOV to/from MKV), and
auto-application could optionally be enabled like autorotate in ffmpeg_filter.c.
Signed-off-by: Neil Birkbeck
---
libavcodec/avpacket.c | 1 +
libavcodec/packet.h| 12
l
DaVinci Resolve).
Signed-off-by: Neil Birkbeck
---
libavcodec/dnxhddata.c | 10 +-
tests/ref/fate/dnxhr-12bit | 2 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/libavcodec/dnxhddata.c b/libavcodec/dnxhddata.c
index 154be89860..73260eadba 100644
--- a/libavcodec
On Fri, Apr 15, 2016 at 4:06 AM wm4 wrote:
> On Wed, 6 Apr 2016 17:09:21 -0600
> Nic Wolfe wrote:
>
> > Thanks for elaborating wm4. Out of curiosity when you say "generic"
> > and "normal" metadata what do you mean? Are you talking about using
> > generic key names or is there another way to sto
On Fri, Jan 18, 2019 at 7:48 AM Vittorio Giovara
wrote:
> On Fri, Jan 18, 2019 at 6:43 AM Carl Eugen Hoyos
> wrote:
>
> > 2019-01-18 4:48 GMT+01:00, Neil Birkbeck :
> > > On Thu, Jan 17, 2019 at 7:43 PM Neil Birkbeck >
> > > wrote:
> > >
> > &
On Thu, Jan 17, 2019 at 7:43 PM Neil Birkbeck
wrote:
> This allows preservation of color values set from the container,
> while still letting the bitstream take precedent when its values
> are specified to some actual value (e.g., not *UNSPECIFIED).
>
> Signed-off-by
This allows preservation of color values set from the container,
while still letting the bitstream take precedent when its values
are specified to some actual value (e.g., not *UNSPECIFIED).
Signed-off-by: Neil Birkbeck
---
libavcodec/proresdec2.c | 9 ++---
1 file changed, 6 insertions
On Sat, Dec 2, 2017 at 5:25 PM, Carl Eugen Hoyos wrote:
> 2017-12-01 20:08 GMT+01:00 Neil Birkbeck :
> > On Thu, Nov 30, 2017 at 9:52 AM, Michael Niedermayer wrote:
>
> >> > For that sample, I feel like it may be incorrectly tagged as pc/full.
> >>
> >&
On Thu, Nov 30, 2017 at 9:52 AM, Michael Niedermayer wrote:
> > > Perfect, thanks Michael. Let me check those samples out.
>
> there are 2 more in 2939 which change:
> https://trac.ffmpeg.org/ticket/2939
>
It seems the swscale_unscaled code paths do not get reconfigured when
sws_setColorSpaceDet
On Wed, Nov 29, 2017 at 7:40 PM, Neil Birkbeck
wrote:
>
>>
>> If you are searching for a case where the patch makes a difference
>> one is:
>> ./ffmpeg -i ~/tickets/4493/AVCI100.mov out.nut
>> file should be here:
>> http://samples.ffmpeg.org/ffmpeg-bugs
>
>
>
> If you are searching for a case where the patch makes a difference
> one is:
> ./ffmpeg -i ~/tickets/4493/AVCI100.mov out.nut
> file should be here:
> http://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket524/
>
> if you want more cases that change, ill see if i can find more
>
Perfect, thanks
On Tue, Nov 28, 2017 at 5:14 PM, Neil Birkbeck
wrote:
> -//The srcBpc check is possibly wrong but we seem to lack a definitive
> reference to test this
> -//and what we have in ticket 2939 looks better with this check
> -if (need_reinit && (c->srcBpc == 8
me other edge cases where range conversion doesn't work
unless explicitly set (e.g., when no scale is happening)
Signed-off-by: Neil Birkbeck
---
libswscale/utils.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/libswscale/utils.c b/libswscale/utils.c
index 4df09306d
On Sat, Nov 19, 2016 at 3:28 PM, James Almer wrote:
> On 11/19/2016 7:19 PM, Michael Niedermayer wrote:
> > On Sat, Nov 19, 2016 at 03:09:15PM -0300, James Almer wrote:
> >> This makes av_stream_add_side_data() consistent with
> av_packet_add_side_data().
> >>
> >> Signed-off-by: James Almer
> >
On Wed, Jun 8, 2016 at 11:27 PM, Dave Rice wrote:
>
> > On Jun 8, 2016, at 11:03 PM, Neil Birkbeck
> wrote:
> >
> > The most recent patch should still apply cleanly (unless we have a
> > better way to set these elements). Thanks
>
> There are specific mas
>
>
> This seems to have been made official now, as usual under version 4
> like every other new element.
> https://github.com/Matroska-Org/libmatroska/commit/
> cd05bc14b42bd88218c61837208fa2b6c79b1de4
>
> Could you confirm there were no changes to the draft and remove the
> experimental checks fo
On Sun, Oct 9, 2016 at 6:45 PM, James Almer wrote:
> On 10/9/2016 10:13 PM, Neil Birkbeck wrote:
> > For non-seekable output files, larger elements written in write_header
> > (like larger attachments, or possibly many tags) can go over
> > IO_BUFFER_SIZE meaning seeking bac
data
mimetype: test/data
Stream #0:3: Attachment: none
Metadata:
filename: data
mimetype: test/data
Signed-off-by: Neil Birkbeck
---
libavformat/matroskaenc.c| 101 +++
tests/fate/matroska.mak |
The most recent patch should still apply cleanly (unless we have a
better way to set these elements). Thanks
On Tue, Apr 19, 2016 at 8:12 AM, Neil Birkbeck wrote:
> Updated patch attached.
>
> On Sat, Apr 16, 2016 at 7:08 PM, Michael Niedermayer
> wrote:
>> On Sun, Apr 03,
Any other thoughts on we should move forward with the _ARIB_STD name
or to use something like HYBRID_LOG_GAMMA?
On Thu, Apr 21, 2016 at 5:04 PM, Neil Birkbeck wrote:
> Thanks Hendrik.
>
> For now, I've updated the patch with a better comment and commit
> message, and will change
often referred to as Perceptual Quantizer or PQ).
On Thu, Apr 21, 2016 at 2:03 PM, Hendrik Leppkes wrote:
> On Thu, Apr 21, 2016 at 10:57 PM, Neil Birkbeck
> wrote:
>> The standard:
>> http://www.arib.or.jp/english/html/overview/doc/2-STD-B67v1_0.pdf
>>
>> The choice of
/arch/search/?email_list=cellar&gbt=1&q=Colour+Format+proposal
Signed-off-by: Neil Birkbeck
---
libavutil/color_utils.c| 16
libavutil/pixdesc.c| 2 +-
libavutil/pixfmt.h | 1 +
tests/ref/fate/color_utils | 19 +++
4 files changed, 37 in
If you use "spherical-video" and put it in the stream's metadata, it
will more naturally agree with location in stream-level tag in mkv:
https://github.com/google/spatial-media/blob/master/docs/spherical-video-rfc.md
On Wed, Apr 20, 2016 at 6:24 PM, Colin McFadden wrote:
> ---
> libavformat/mov.
Updated patch attached.
On Sat, Apr 16, 2016 at 7:08 PM, Michael Niedermayer
wrote:
> On Sun, Apr 03, 2016 at 03:38:33PM -0700, Neil Birkbeck wrote:
>> Use "master_display" key/value pair to specify mastering metadata in a
>> similar formatting as accepted by libx265 (u
The value is not written in the matroska tags, only to the colour elements.
I'm happy to consider alternatives, but wasn't sure there was a
precedent for instantiating the packet side data from command line.
On Sun, Apr 3, 2016 at 5:13 PM, Dave Rice wrote:
>
>> On Apr 3, 2016,
transmuxed (mkv -> mkv). Perhaps we want to also
use this same metadata key in matroskadec to also allow for the
metadata to propagate during transcoding.
From b30d80f6ba4b09811039f64af3e7f709d86df5fe Mon Sep 17 00:00:00 2001
From: Neil Birkbeck
Date: Fri, 1 Apr 2016 17:02:42 -0700
Subject:
e still seems to be some discussion around these.
The new elements are exposed under strict experimental mode.
Signed-off-by: Neil Birkbeck
---
libavformat/matroskaenc.c | 63 ++-
1 file changed, 62 insertions(+), 1 deletion(-)
diff --git
And yet another revision, where the syntax lists are actually terminated...
On Thu, Mar 3, 2016 at 12:09 PM, Neil Birkbeck wrote:
> I've attached a slightly cleaner (more isolated) patch that moves the
> parsing into a helper function.
>
> On Mon, Feb 29, 2016 at 5:15 PM, Neil B
I've attached a slightly cleaner (more isolated) patch that moves the
parsing into a helper function.
On Mon, Feb 29, 2016 at 5:15 PM, Neil Birkbeck wrote:
> Adding early support for a subset of the proposed colour elements
> according to the latest version of sp
Signed-off-by: Neil Birkbeck
---
libavformat/dump.c | 21 +
1 file changed, 21 insertions(+)
diff --git a/libavformat/dump.c b/libavformat/dump.c
index 56c285d..9e7c12b 100644
--- a/libavformat/dump.c
+++ b/libavformat/dump.c
@@ -26,6 +26,7 @@
#include "libavutil/disp
ms to be some discussion around these, and the max_cll/max_fall
are currently not propagated as there is not yet side data for them.
The new elements are exposed under strict experimental mode.
Signed-off-by: Neil Birkbeck
---
libavformat/matroska.h| 28 ++
libavformat/matros
Some containers, like webm/mkv, will contain this mastering metadata.
This is analogous to the way 3D fpa data is handled (in frame and
packet side data).
Signed-off-by: Neil Birkbeck
---
libavcodec/avcodec.h | 7 +++
libavcodec/avpacket.c | 33
st if some frames were dropped in some part of the filter
chain.
On Thu, Feb 4, 2016 at 11:04 PM, Hendrik Leppkes wrote:
> On Fri, Feb 5, 2016 at 3:05 AM, Neil Birkbeck wrote:
>> According to the ITU-T H.265 v3, in Table D - 1, the persistence scope
>> of the mastering display met
SEI comes before the IDR, I don't see a great way to do
this aside from initializing the flag to 2 and decrementing (when > 0)
when we see an IDR.
On Tue, Feb 2, 2016 at 7:13 AM, Michael Niedermayer
wrote:
> On Mon, Feb 01, 2016 at 12:11:10PM -0800, Neil Birkbeck wrote:
>> Please see
Please see updated patch.
On Mon, Jan 25, 2016 at 11:39 PM, Neil Birkbeck wrote:
> I guess png is another example; 10 is the denominator for the 32-bit
> integer:
> https://github.com/FFmpeg/FFmpeg/blob/b58cfa616c169c90166938608e7135cdab5820e0/libavcodec/pngenc.c#L290
>
> I
Patch with corrected "From:" is attached.
On Tue, Jan 26, 2016 at 2:28 PM, Vittorio Gambaletta (VittGam)
wrote:
> On 26/01/2016 23:05:22 CET, Neil Birkbeck wrote:
>>
>> Some sort of squash fail. Apologies.
>
>
> No problem!
>
> __
Some sort of squash fail. Apologies.
On Tue, Jan 26, 2016 at 1:13 PM, Vittorio Gambaletta (VittGam) <
ffmpeg-...@vittgam.net> wrote:
> Hi,
>
> On 26/01/2016 22:09:02 CET, Neil Birkbeck wrote:
>
>> From: "Vittorio Gambaletta (VittGam)"
>>
>
>
From: "Vittorio Gambaletta (VittGam)"
(this structure is not referenced anywhere yet)
Signed-off-by: Neil Birkbeck
---
libavutil/mastering_display_metadata.h | 10 ++
libavutil/version.h| 3 +--
2 files changed, 7 insertions(+), 6 deletions(-)
di
7;ll change the avutil
struct float internal fields to rational.
On Mon, Jan 25, 2016 at 1:43 PM, Hendrik Leppkes
wrote:
> On Mon, Jan 25, 2016 at 10:37 PM, Michael Niedermayer
> wrote:
> > On Fri, Jan 22, 2016 at 02:54:21PM -0800, Neil Birkbeck wrote:
> >> Hmm. I don
Also, the mkv adjustments are here:
https://mailarchive.ietf.org/arch/search/?email_list=cellar&gbt=1&index=sZyfPTM-QY69P-0omfOIiTN622o
On Fri, Jan 22, 2016 at 2:54 PM, Neil Birkbeck
wrote:
> Hmm. I don't have a good idea of how likely it is for this conversion to
> f
et a float/double for
mkv.
On Fri, Jan 22, 2016 at 1:01 AM, wm4 wrote:
> On Thu, 21 Jan 2016 15:57:38 -0800
> Neil Birkbeck wrote:
>
> > Thanks for the quick review, Michael.
> >
> > The display primaries are encoded as integers (rational with fixed
> > den
Add support for parsing SEI_TYPE_MASTERING_DISPLAY_INFO and propagate contents
into
the AVMasteringDisplayMetadata side data. Primaries are ordered in RGB order and
the values are converted to the natural ranges ([0,1] for CEI 1931 Chroma
coords,
and cd/m^2 for luma).
Signed-off-by: Neil
; Chroma coords,
> > and cd/m^2 for luma).
> >
> > Signed-off-by: Neil Birkbeck
> > ---
> > libavcodec/hevc.c | 38 ++
> > libavcodec/hevc.h | 7 +++
> > libavcodec/hevc_sei.c | 22 ++
&g
Add support for parsing SEI_TYPE_MASTERING_DISPLAY_INFO and propagate contents
into
the AVMasteringDisplayMetadata side data. Primaries are ordered in RGB order and
the values are converted to the natural ranges ([0,1] for CEI 1931 Chroma
coords,
and cd/m^2 for luma).
Signed-off-by: Neil
Add names for recently added enums to av_frame_side_data_name.
Signed-off-by: Neil Birkbeck
---
libavutil/frame.c | 4
1 file changed, 4 insertions(+)
diff --git a/libavutil/frame.c b/libavutil/frame.c
index f2097e9..92bbe09 100644
--- a/libavutil/frame.c
+++ b/libavutil/frame.c
@@ -728,7
:02:25 +0100
> > Michael Niedermayer wrote:
> >
> > > On Tue, Jan 19, 2016 at 10:18:16AM +0100, wm4 wrote:
> > > > On Tue, 19 Jan 2016 00:32:43 +0100
> > > > Michael Niedermayer wrote:
> > > >
> > > > > On Sat, Jan 16, 2016 a
/arch/search/?email_list=cellar&gbt=1&index=sZyfPTM-QY69P-0omfOIiTN622o
so it is similar to SEI FPA / stereo_mode in MKV and as such this patch follows
how AVStereo3D is implemented.
I'll add support to HEVC in a follow-up (and MKV when spec is approved).
Signed-off-by:
QuickTime metadata can come after trak data. Add indicator for which trak is
being parsed (-1 if none) so that global metadata after the trak can be parsed.
Signed-off-by: Neil Birkbeck
---
libavformat/isom.h | 1 +
libavformat/mov.c | 6 +-
2 files changed, 6 insertions(+), 1 deletion
Fixing small leaks that can occur when mkv_write_tracks fails in
mkv_write_header
(e.g., if video track has unknown codec). Also changing mkv_write_seekhead to
take
the MatroskaMuxContext to avoid having dangling pointers.
Signed-off-by: Neil Birkbeck
---
libavformat/matroskaenc.c | 67
an:[126] stdev:[46.0]
[Parsed_showinfo_1 @ 0x1e307d0] n:1 pts:2 pts_time:0.04 pos:-1 fmt:gray sar:0/1
s:352x432 i:P iskey:1 type:I checksum:4E674BC7 plane_checksum:[4E674BC7]
mean:[125] stdev:[46.0]
(Outputs 50 frames)
Signed-off-by: Neil Birkbeck
---
libavfilter/vf_yadif.c | 6 +++---
1 file changed
:0 pos:-1 fmt:gray sar:0/1
s:352x432 ...
[Parsed_idet_0 @ 0x18f9bb0] Repeated Fields: Neither: 1 Top: 0
Bottom: 0
Fate looks good.
Signed-off-by: Neil Birkbeck
---
libavfilter/vf_idet.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/libavfilter
0 BFF:0
Progressive:0 Undetermined:0
On Wed, Oct 22, 2014 at 11:18 AM, Thierry Foucu wrote:
> On Wed, Oct 22, 2014 at 7:43 AM, Pascal Massimino <
> pascal.massim...@gmail.com> wrote:
>
> > Hi,
> >
> > On Tue, Oct 21, 2014 at 11:40 PM, Neil Birkbeck >
> > wro
ms this filter will also need a subsequent patch for single
frame input)
Signed-off-by: Neil Birkbeck
---
libavfilter/vf_idet.c | 31 +++
libavfilter/vf_idet.h | 1 +
tests/ref/fate/filter-idet | 2 +-
3 files changed, 33 insertions(+), 1 deletion(-)
59 matches
Mail list logo