[FFmpeg-devel] Fwd: Fate test for mxfdec.c: prefer metadata from Footer -

2023-03-13 Thread emcodem
uot;, > > > > Exception one Op1b self contained file, where the "old" version > > did > > not spit out a "duration" value at all, so it was not even > > calculated > > from bitrate, it was just missing in the format section and set >

Re: [FFmpeg-devel] Fwd: Re: [PATCH] mxfdec.c: prefer metadata from Footer

2021-07-23 Thread emcodem
...@ffmpeg.org with subject "unsubscribe". Anything left that needs investigation/correction on this one? ...it looks to be a very nice addition so far. Thanks -emcodem ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailma

Re: [FFmpeg-devel] Fwd: Re: [PATCH] mxfdec.c: prefer metadata from Footer

2021-07-04 Thread emcodem
Am 2021-07-04 20:28, schrieb emco...@ffastrans.com: Am 2021-07-04 17:28, schrieb Marton Balint: On Sat, 3 Jul 2021, Tomas Härdin wrote: lör 2021-07-03 klockan 15:13 +0200 skrev emco...@ffastrans.com: Unfortunately the wetransfer link for the fate samples expired, so i thought it might be a

Re: [FFmpeg-devel] Fwd: Re: [PATCH] mxfdec.c: prefer metadata from Footer

2021-07-04 Thread emcodem
Am 2021-07-04 17:28, schrieb Marton Balint: On Sat, 3 Jul 2021, Tomas Härdin wrote: lör 2021-07-03 klockan 15:13 +0200 skrev emco...@ffastrans.com: Unfortunately the wetransfer link for the fate samples expired, so i thought it might be a good idea to resend it as link to gdrive:

[FFmpeg-devel] Fwd: Re: [PATCH] mxfdec.c: prefer metadata from Footer

2021-07-03 Thread emcodem
Also, the FATE samples i shared will still work and can be used for this patch. Attached a patch for adding only the fate samples. Note that these Fate tests of course fail when the patch for mxfdec.c is not applied. https://we.tl/t-MVmyG2mZHq Thanks a lot! -emcodem

Re: [FFmpeg-devel] [PATCH] mxfdec.c: prefer metadata from Footer

2021-06-28 Thread emcodem
still work and can be used for this patch. Attached a patch for adding only the fate samples. Note that these Fate tests of course fail when the patch for mxfdec.c is not applied. https://we.tl/t-MVmyG2mZHq Thanks a lot! -emcodemFrom 499a41f0db6cfda746fd8751d4e55e910f495687 Mon Sep 17 00:00:00

Re: [FFmpeg-devel] [PATCH] mxfdec.c: prefer metadata from Footer

2021-06-27 Thread emcodem
Am 2021-06-27 20:12, schrieb Marton Balint: Why? I though it is enough if you store the partition number in the metadata set, that way you should be able to compare if the existing metadata set is better than the current one when adding a new metadata set. Or am I missing something? OK, i just

Re: [FFmpeg-devel] [PATCH] mxfdec.c: prefer metadata from Footer

2021-06-27 Thread emcodem
Am 2021-06-27 20:12, schrieb Marton Balint: On Sun, 27 Jun 2021, Tomas Härdin wrote: Order of parsing depends on the file, we usually do header, footer and then backwards the partitions. So after your patch metadata in the second partition will have priority over footer, which is also

[FFmpeg-devel] [PATCH] mxfdec.c: prefer metadata from Footer

2021-06-26 Thread emcodem
In case there is a Footer, regarding to SMPTE 377 all versions, the metadata in Footer shall be correct (where in Header it can be incomplete).. If there is no footer (stream, truncated...) it will still work as usual. Tested with a huge set of files and compared old/new ffprobes, it will not

Re: [FFmpeg-devel] [PATCH] avisynth.c corrected interlace detection

2021-05-31 Thread emcodem
mmit i sent is included , alter it and commit again - or you want me to start from scratch and send a totally new patch?) Thanks a lot Steve -emcodem ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] mxfdec.c: Try TC from Footer if Header TC was < 1

2021-05-24 Thread emcodem
Am 2021-05-24 17:31, schrieb Tomas Härdin: mån 2021-05-24 klockan 12:30 +0200 skrev emcodem: Added support for reading Start Timecode from Footer (if any). Specifically targets Omneon 6.4.3.0 but also works on other Versions and Vendors, e.g. when Header is OpenIncomplete. Function

[FFmpeg-devel] [PATCH] avisynth.c corrected interlace detection

2021-05-24 Thread emcodem
Sorry for the delay on this, should have corrected it much earlier. There was some confusion in the interlaced analysis. From 3rdparty decoders perspective, a clip can only be interlaced when it is NOT field_based. This is because in a field_based clip, the fields are separate images, so it is

[FFmpeg-devel] [PATCH] mxfdec.c: Try TC from Footer if Header TC was < 1

2021-05-24 Thread emcodem
Added support for reading Start Timecode from Footer (if any). Specifically targets Omneon 6.4.3.0 but also works on other Versions and Vendors, e.g. when Header is OpenIncomplete. Function mxf_resolve_strong_ref_reverse can potentially be re-used for getting other values like Duration and

Re: [FFmpeg-devel] [PATCH] mov.c log qt ref extenal essence metadata

2021-03-06 Thread emcodem
Am 2021-03-06 10:48, schrieb Jan Ekström: On Sat, Mar 6, 2021 at 10:38 AM emcodem wrote: --- libavformat/mov.c | 8 1 file changed, 8 insertions(+) In quicktime reference files, exposing the parsed info for external essences location can be very handy for users Unfortunately

[FFmpeg-devel] [PATCH] mov.c log qt ref extenal essence metadata

2021-03-06 Thread emcodem
--- libavformat/mov.c | 8 1 file changed, 8 insertions(+) In quicktime reference files, exposing the parsed info for external essences location can be very handy for users diff --git a/libavformat/mov.c b/libavformat/mov.c index 1c07cff6b5..e9625c0cf9 100644 --- a/libavformat/mov.c

Re: [FFmpeg-devel] [PATCH] Populate field order returned by avs script, Trac Ticket 8757

2021-02-08 Thread emcodem
for Pinging again :-( I know it's not much that the patch does but the resulting functionality from a user perspective can be really useful, especially when using programatically calculated avs scripts.From 243bb2cbae9fabc78fe7f48cb90ae1c620c51a51 Mon Sep 17 00:00:00 2001 From: emcodem Date: Thu, 21 Jan

Re: [FFmpeg-devel] [PATCH] Populate field order returned by avs script, Trac Ticket 8757

2021-02-01 Thread emcodem
Am 2021-01-21 19:08, schrieb emco...@ffastrans.com: On 2021-01-21 14:10, Stephen Hutchinson wrote: Yeah, never mind about that. I didn't notice that those are declared AVSC_INLINE, not AVSC_API, so they don't get used through the dynamic API loader. The comment formatting seems to have been

Re: [FFmpeg-devel] [PATCH v4 3/3] avformat/mxfenc: prefer to use the configured metadta

2021-01-28 Thread emcodem
Am 2021-01-28 02:21, schrieb lance.lmw...@gmail.com: I haven't found the s337m freely, so I'm not sure about the Platforma metadata. I think we haven't support the field yet, I guess it's 0x3C08 tag, but I haven't the document in hand so it's not OK to add it by me. Yeah the SMPTE are

Re: [FFmpeg-devel] [PATCH v4 3/3] avformat/mxfenc: prefer to use the configured metadta

2021-01-26 Thread emcodem
Am 2021-01-26 00:34, schrieb Marton Balint: Uhm guys, it is very bad practice: if you just insert a different manufacturer name then you just cheat. SMPTE 377 has a clear statement about what goes to the identification, you cannot just write the infos from a different program there.

Re: [FFmpeg-devel] [PATCH v4 3/3] avformat/mxfenc: prefer to use the configured metadta

2021-01-25 Thread emcodem
Am 2021-01-20 16:41, schrieb Tomas Härdin: ons 2021-01-20 klockan 00:27 +0100 skrev Marton Balint: On Tue, 19 Jan 2021, Tobias Rapp wrote: > On 18.01.2021 23:53, Tomas Härdin wrote: > > lör 2021-01-16 klockan 08:43 +0800 skrev lance.lmw...@gmail.com: > > > On Fri, Jan 15, 2021 at 09:43:58PM

Re: [FFmpeg-devel] [PATCH] Populate field order returned by avs script, Trac Ticket 8757

2021-01-21 Thread emcodem
weaknesses :D Thanks for your patience and also thanks for telling me directly how the formatting is done correctly. New patch with formatted comment attached From 243bb2cbae9fabc78fe7f48cb90ae1c620c51a51 Mon Sep 17 00:00:00 2001 From: emcodem Date: Thu, 21 Jan 2021 18:59:45 +0100 Subject: [PATCH

Re: [FFmpeg-devel] [PATCH] Populate field order returned by avs script, Trac Ticket 8757

2021-01-17 Thread emcodem
Mon Sep 17 00:00:00 2001 From: emcodem Date: Sun, 17 Jan 2021 18:59:41 +0100 Subject: [PATCH] Populate field order returned by avs script, Trac Ticket 8757 --- libavformat/avisynth.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/libavformat/avisynth.c b/libavformat

[FFmpeg-devel] [PATCH] Populate field order returned by avs script, Trac Ticket 8757

2021-01-16 Thread emcodem
. Make FATE passed.From 31a37518afa5f9b635e6c52a8f29513bc756c715 Mon Sep 17 00:00:00 2001 From: emcodem Date: Sun, 17 Jan 2021 01:02:08 +0100 Subject: [PATCH] Populate field order returned by avs script, Trac Ticket 8757 --- libavformat/avisynth.c | 20 1 file changed, 20