[FFmpeg-devel] [PATCH] lavfi/qsv: Copy metadata fields from the given input

2024-04-24 Thread Xiang, Haihao
From: Haihao Xiang Currently it always copies the metadata fields from the last input when there are multiple inputs for the filter. For example, the metadata fields from input1 are copied to the output for overlay_qsv filter, however for regular overlay filters, the metadata fields from input0

Re: [FFmpeg-devel] [PATCH v2] avcodec/jpeg2000dec: support of 2 fields in 1 AVPacket

2024-04-24 Thread Michael Niedermayer
On Wed, Apr 24, 2024 at 01:20:02PM +0200, Jerome Martinez wrote: > Hi, I'm bumping this patch proposal for avoiding a situation where FFmpeg > skips half the visual content when 2 jpeg2000 codestreams are present in one > frame. I re-reviewed this discussion and think I answered all concerns. I'm

Re: [FFmpeg-devel] [PATCH v2] opusdsp: add ability to modify deemphasis constant

2024-04-24 Thread Michael Niedermayer
On Wed, Apr 24, 2024 at 12:58:54PM +0200, Lynne wrote: > Apr 24, 2024, 10:54 by d...@lynne.ee: > > > xHE-AAC relies on the same postfilter mechanism > > that Opus uses to improve clarity (albeit with a steeper > > deemphasis filter). > > > > The code to apply it is identical, it's still just a >

Re: [FFmpeg-devel] [RFC] 5 year plan & Inovation

2024-04-24 Thread Michael Niedermayer
On Thu, Apr 25, 2024 at 12:50:02AM +0200, Tomas Härdin wrote: > ons 2024-04-17 klockan 15:58 +0200 skrev Michael Niedermayer: > > > * ffchat > >     (expand into realtime chat / zoom) this would > >     bring in more users and developers, and we basically have almost > >     all parts for it

Re: [FFmpeg-devel] [RFC] 5 year plan & Inovation

2024-04-24 Thread Diederick C. Niehorster
On Thu, Apr 25, 2024 at 12:50 AM Tomas Härdin wrote: > A large long term project that would help immensely with security is > moving to a proper parsing framework, rather than the present shotgun > parsing approach. But this might be such a large undertaking that it's > better to start from

Re: [FFmpeg-devel] [PATCH v2] avcodec/jpeg2000dec: support of 2 fields in 1 AVPacket

2024-04-24 Thread Tomas Härdin
ons 2024-04-24 klockan 13:20 +0200 skrev Jerome Martinez: > Hi, I'm bumping this patch proposal for avoiding a situation where > FFmpeg skips half the visual content when 2 jpeg2000 codestreams are > present in one frame. I re-reviewed this discussion and think I > answered > all concerns. I'm

Re: [FFmpeg-devel] [RFC] 5 year plan & Inovation

2024-04-24 Thread Tomas Härdin
ons 2024-04-17 klockan 15:58 +0200 skrev Michael Niedermayer: > * ffchat >     (expand into realtime chat / zoom) this would >     bring in more users and developers, and we basically have almost >     all parts for it already but some people where against it You mean inventing a new chat

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: High bit depth encoding for HEVC

2024-04-24 Thread Timo Rothenpieler
On 19.04.2024 10:38, Diego Felix de Souza via ffmpeg-devel wrote: From: Diego Felix de Souza Adding 10-bit encoding support for HEVC if the input is 8-bit. In case of 8-bit input content, NVENC performs an internal CUDA 8 to 10-bit conversion of the input prior to encoding. Currently, only AV1

Re: [FFmpeg-devel] Inconsistent usage of AVFieldOrder values

2024-04-24 Thread Marton Balint
On Mon, 22 Apr 2024, Devin Heitmueller wrote: Hello, I suspect this topic has been visited a number of times over the years, but I figured I should re-raise it. In the compressed domain, field ordering is represented by the AVFieldOrder enumeration. Among the interlaced possibilities,

Re: [FFmpeg-devel] [RFC] 5 year plan & Inovation

2024-04-24 Thread Michael Niedermayer
On Tue, Apr 23, 2024 at 03:12:59PM +0300, Rémi Denis-Courmont wrote: > > > Le 21 avril 2024 23:40:08 GMT+03:00, Michael Niedermayer > a écrit : > >On Sun, Apr 21, 2024 at 05:11:36PM +0800, Rémi Denis-Courmont wrote: > >> > >> > >> Le 17 avril 2024 21:58:32 GMT+08:00, Michael Niedermayer >

Re: [FFmpeg-devel] [PATCH] fate: allow https for git URLs

2024-04-24 Thread Timo Rothenpieler
On 24.04.2024 22:53, Alexander Strasser via ffmpeg-devel wrote: On 2024-04-24 22:26 +0200, Timo Rothenpieler wrote: On 24.04.2024 22:12, Alexander Strasser via ffmpeg-devel wrote: On 2024-04-24 22:01 +0200, Timo Rothenpieler wrote: --- tests/fate.sh | 4 ++-- 1 file changed, 2

Re: [FFmpeg-devel] [PATCH 4/5] doc/examples/qsv_transcode: Simplify str_to_dict() loop

2024-04-24 Thread Michael Niedermayer
On Wed, Apr 24, 2024 at 05:18:55AM +, Xiang, Haihao wrote: > On Wo, 2024-04-24 at 03:45 +0200, Michael Niedermayer wrote: > > Fixes: CID1517022 Logically dead code > > > > Sponsored-by: Sovereign Tech Fund > > Signed-off-by: Michael Niedermayer > > --- > >  doc/examples/qsv_transcode.c | 3

Re: [FFmpeg-devel] [PATCH 3/5] doc/examples/vaapi_transcode: Simplify loop

2024-04-24 Thread Michael Niedermayer
On Wed, Apr 24, 2024 at 02:39:45PM +0800, myp...@gmail.com wrote: > On Wed, Apr 24, 2024 at 9:45 AM Michael Niedermayer > wrote: > > > > Fixes: CID1428858(1/2) Logically dead code > > > > Sponsored-by: Sovereign Tech Fund > > Signed-off-by: Michael Niedermayer > > --- > >

Re: [FFmpeg-devel] [PATCH 2/5] doc/examples/qsv_transcode: Simplify loop

2024-04-24 Thread Michael Niedermayer
On Wed, Apr 24, 2024 at 05:25:11AM +, Xiang, Haihao wrote: > On Wo, 2024-04-24 at 03:45 +0200, Michael Niedermayer wrote: > > Fixes: CID1428858(2/2) Logically dead code > > > > Sponsored-by: Sovereign Tech Fund > > Signed-off-by: Michael Niedermayer > > --- > >  doc/examples/qsv_transcode.c

Re: [FFmpeg-devel] [PATCH] fate: allow https for git URLs

2024-04-24 Thread Alexander Strasser via ffmpeg-devel
On 2024-04-24 22:26 +0200, Timo Rothenpieler wrote: > On 24.04.2024 22:12, Alexander Strasser via ffmpeg-devel wrote: > > On 2024-04-24 22:01 +0200, Timo Rothenpieler wrote: > > > --- > > > tests/fate.sh | 4 ++-- > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > diff --git

Re: [FFmpeg-devel] [PATCH] fate: allow https for git URLs

2024-04-24 Thread Timo Rothenpieler
On 24.04.2024 22:12, Alexander Strasser via ffmpeg-devel wrote: On 2024-04-24 22:01 +0200, Timo Rothenpieler wrote: --- tests/fate.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/fate.sh b/tests/fate.sh index c5ee18de80..4081e865ae 100755 --- a/tests/fate.sh

Re: [FFmpeg-devel] [PATCH] fate: allow https for git URLs

2024-04-24 Thread Alexander Strasser via ffmpeg-devel
On 2024-04-24 22:01 +0200, Timo Rothenpieler wrote: > --- > tests/fate.sh | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tests/fate.sh b/tests/fate.sh > index c5ee18de80..4081e865ae 100755 > --- a/tests/fate.sh > +++ b/tests/fate.sh > @@ -30,14 +30,14 @@ lock(){ >

[FFmpeg-devel] [PATCH] fate: allow https for git URLs

2024-04-24 Thread Timo Rothenpieler
--- tests/fate.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/fate.sh b/tests/fate.sh index c5ee18de80..4081e865ae 100755 --- a/tests/fate.sh +++ b/tests/fate.sh @@ -30,14 +30,14 @@ lock(){ checkout(){ case "$repo" in file:*|/*) src="${repo#file:}"

Re: [FFmpeg-devel] [PATCH v2 0/9] HTTP rate limiting and retry improvements

2024-04-24 Thread Derek Buitenhuis
On 4/24/2024 12:13 PM, Martin Storsjö wrote: > I had a look over this patchset, and I had a handful of minor comments, > but overall, the patchset seems fine to me. Thanks! Applied all your comments. Will wait a day and then push if no others appear. - Derek

Re: [FFmpeg-devel] [PATCH v2 6/9] avformat/http: Add options to set the max number of connection retries

2024-04-24 Thread Derek Buitenhuis
On 4/24/2024 12:08 PM, Martin Storsjö wrote: > Minor inconsistency; the corresponding variable in the other function was > called conn_attempts, as a plural. Renamed to the plural version. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH v2 2/9] avformat/http: Use AVERROR_HTTP_TOO_MANY_REQUESTS

2024-04-24 Thread Derek Buitenhuis
On 4/24/2024 11:58 AM, Martin Storsjö wrote: > This function seems to handle both the literal status codes, like 429, and > also AVERROR style error codes, as when called from handle_http_errors, so > perhaps it would be good for consistency to add the AVERROR here too. Good catch. Added. -

Re: [FFmpeg-devel] [PATCH v2 2/9] avformat/http: Use AVERROR_HTTP_TOO_MANY_REQUESTS

2024-04-24 Thread Derek Buitenhuis
On 4/24/2024 11:53 AM, Martin Storsjö wrote: > Typo in the commit message Fixed locally. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email

Re: [FFmpeg-devel] [PATCH v2 4/9] avformat/http: Add support for Retry-After header

2024-04-24 Thread Derek Buitenhuis
On 4/24/2024 12:06 PM, Martin Storsjö wrote: > Is this feature standardized in a RFC, or is it some other spec somewhere? > I think it would be nice with a link to a spec in the commit message here. It is in the RFC for 429 I noted in the commit I added that: RFC6585. It is also probably in the

[FFmpeg-devel] [PATCH 2/2] checkasm/h264dsp: support checking more idct depths

2024-04-24 Thread J. Dekker
Signed-off-by: J. Dekker --- tests/checkasm/h264dsp.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tests/checkasm/h264dsp.c b/tests/checkasm/h264dsp.c index 0f484e3f43..5cb646ae49 100644 --- a/tests/checkasm/h264dsp.c +++ b/tests/checkasm/h264dsp.c @@ -173,6 +173,7

[FFmpeg-devel] [PATCH 1/2] avfilter/riscv: build afir only if required

2024-04-24 Thread J. Dekker
Signed-off-by: J. Dekker --- libavfilter/riscv/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/riscv/Makefile b/libavfilter/riscv/Makefile index 0b968a9c0d..277dde2aed 100644 --- a/libavfilter/riscv/Makefile +++ b/libavfilter/riscv/Makefile @@ -1,2

Re: [FFmpeg-devel] [PATCH v3 1/4] lavu: Remove libva 1.x support

2024-04-24 Thread Xiang, Haihao
On Ma, 2024-04-22 at 22:41 +0100, Mark Thompson wrote: > libva 2.0 was released in 2017 and the 2.x versions are included in all > supported distributions nowadays. > --- > Rebased. > > I think we can also drop the other quirks?  They are for the proprietary media > SDK driver (which I think is

Re: [FFmpeg-devel] [PATCH 1/2] lavc/h265_profile_level: Expand profile compatibility checking

2024-04-24 Thread Xiang, Haihao
On Ma, 2024-04-22 at 22:22 +0100, Mark Thompson wrote: > Replace existing get_profile() with find_profile(), which finds the > lowest compatible profile rather than requiring an exact match. > --- >  libavcodec/h265_profile_level.c | 73 + >  

Re: [FFmpeg-devel] [PATCH 2/2] lavc/vaapi_hevc: Don't require exact profiles

2024-04-24 Thread Xiang, Haihao
On Ma, 2024-04-22 at 22:23 +0100, Mark Thompson wrote: > Rather than turning the constraint flags into a single profile and then > searching for that profile (and failing if it doesn't match any profile > exactly), instead search all supported profiles and use the first one > which supports the

Re: [FFmpeg-devel] [PATCH 0/5] replace scale2ref by scale=rw:rh

2024-04-24 Thread Niklas Haas
On Wed, 24 Apr 2024 16:48:54 +0530 Gyan Doshi wrote: > > > On 2024-04-24 04:21 pm, Niklas Haas wrote: > > As discussed in my previous series for fixing scale2ref[1], this filter > > is fundamentally broken, and the only real fix would be to switch to > > activate(), or ideally FFFrameSync. > >

Re: [FFmpeg-devel] [PATCH 0/5] replace scale2ref by scale=rw:rh

2024-04-24 Thread Timo Rothenpieler
On 24/04/2024 13:18, Gyan Doshi wrote: On 2024-04-24 04:21 pm, Niklas Haas wrote: As discussed in my previous series for fixing scale2ref[1], this filter is fundamentally broken, and the only real fix would be to switch to activate(), or ideally FFFrameSync. [1]

Re: [FFmpeg-devel] [PATCH v2] avcodec/jpeg2000dec: support of 2 fields in 1 AVPacket

2024-04-24 Thread Jerome Martinez
Hi, I'm bumping this patch proposal for avoiding a situation where FFmpeg skips half the visual content when 2 jpeg2000 codestreams are present in one frame. I re-reviewed this discussion and think I answered all concerns. I'm hesitant with patch v3 because I consider that touching

Re: [FFmpeg-devel] [PATCH 0/5] replace scale2ref by scale=rw:rh

2024-04-24 Thread Gyan Doshi
On 2024-04-24 04:21 pm, Niklas Haas wrote: As discussed in my previous series for fixing scale2ref[1], this filter is fundamentally broken, and the only real fix would be to switch to activate(), or ideally FFFrameSync. [1] https://ffmpeg.org//pipermail/ffmpeg-devel/2024-March/323382.html

Re: [FFmpeg-devel] [PATCH v2 0/9] HTTP rate limiting and retry improvements

2024-04-24 Thread Martin Storsjö
On Mon, 22 Apr 2024, Derek Buitenhuis wrote: This patch set adds support for properly handling HTTP 429 codes, and their rate limiting, which is widely used and is standardized. Changes since first set: * Added AVERROR_HTTP_TOO_MANY_REQUESTS top error_entries in error.c, per Andreas' review.

Re: [FFmpeg-devel] [PATCH v2 6/9] avformat/http: Add options to set the max number of connection retries

2024-04-24 Thread Martin Storsjö
On Mon, 22 Apr 2024, Derek Buitenhuis wrote: Not every use case benefits from setting retries in terms of the backoff. Signed-off-by: Derek Buitenhuis --- libavformat/http.c| 12 +--- libavformat/version.h | 2 +- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH v2 4/9] avformat/http: Add support for Retry-After header

2024-04-24 Thread Martin Storsjö
On Mon, 22 Apr 2024, Derek Buitenhuis wrote: 429 and 503 codes can, and often do (e.g. all Google Cloud Storage URLs can), return a Retry-After header with the error, indicating how long to wait, in seconds, before retrying again. If it is not respected by, for example, using our default

[FFmpeg-devel] [PATCH 5/5] avfilter/scale2ref: deprecate in favor of scale=rw:rh

2024-04-24 Thread Niklas Haas
From: Niklas Haas And remove it from the documentation. --- Changelog | 1 + doc/filters.texi | 73 -- libavfilter/vf_scale.c | 3 ++ 3 files changed, 4 insertions(+), 73 deletions(-) diff --git a/Changelog b/Changelog index

[FFmpeg-devel] [PATCH] aacdec: avoid generating unused code when either implementation is disabled

2024-04-24 Thread Lynne
Minor optimization to remove extra branches. We need to include the header for xHE anyway, which is float-only. >From 54f53fb1c0694bf2777a1eefb1f585a804d56978 Mon Sep 17 00:00:00 2001 From: Lynne Date: Wed, 24 Apr 2024 13:01:14 +0200 Subject: [PATCH] aacdec: avoid generating unused code when

[FFmpeg-devel] [PATCH 4/5] fate/scale2ref_keep_aspect: switch to vf_scale ref_*

2024-04-24 Thread Niklas Haas
From: Niklas Haas --- tests/filtergraphs/scale2ref_keep_aspect | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/tests/filtergraphs/scale2ref_keep_aspect b/tests/filtergraphs/scale2ref_keep_aspect index f407460ec7..00b04fc3d1 100644 ---

[FFmpeg-devel] [PATCH 3/5] avfilter/vf_scale: add optional "ref" input

2024-04-24 Thread Niklas Haas
From: Niklas Haas This is automatically enabled if the width/height expressions reference any ref_* variable. This will ultimately serve as a more principled replacement for the fundamentally broken scale2ref. See-Also: https://trac.ffmpeg.org/ticket/10795 --- Changelog | 1 +

[FFmpeg-devel] [PATCH 2/5] avfilter/vf_scale: switch to FFFrameSync

2024-04-24 Thread Niklas Haas
From: Niklas Haas Preliminary commit, in anticipation of adding support for multiple inputs (with proper synchronization and activate() callback). --- doc/filters.texi | 4 +-- libavfilter/vf_scale.c | 65 +++--- 2 files changed, 63 insertions(+), 6

[FFmpeg-devel] [PATCH 1/5] avfilter/framesync: make framesync_class un-static

2024-04-24 Thread Niklas Haas
From: Niklas Haas And rename to ff_framesync_class. More convenient for downstream users. --- libavfilter/framesync.c | 6 +++--- libavfilter/framesync.h | 1 + 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/libavfilter/framesync.c b/libavfilter/framesync.c index

[FFmpeg-devel] [PATCH 0/5] replace scale2ref by scale=rw:rh

2024-04-24 Thread Niklas Haas
As discussed in my previous series for fixing scale2ref[1], this filter is fundamentally broken, and the only real fix would be to switch to activate(), or ideally FFFrameSync. [1] https://ffmpeg.org//pipermail/ffmpeg-devel/2024-March/323382.html The main thing making this difficult is the fact

Re: [FFmpeg-devel] [PATCH v2] opusdsp: add ability to modify deemphasis constant

2024-04-24 Thread Lynne
Apr 24, 2024, 10:54 by d...@lynne.ee: > xHE-AAC relies on the same postfilter mechanism > that Opus uses to improve clarity (albeit with a steeper > deemphasis filter). > > The code to apply it is identical, it's still just a > simple IIR low-pass filter. This commit makes it possible > to use

Re: [FFmpeg-devel] [PATCH v2 2/9] avformat/http: Use AVERROR_HTTP_TOO_MANY_REQUESTS

2024-04-24 Thread Martin Storsjö
On Mon, 22 Apr 2024, Derek Buitenhuis wrote: Added in thep previous commit. Signed-off-by: Derek Buitenhuis --- libavformat/http.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/http.c b/libavformat/http.c index ed20359552..bbace2694f 100644 --- a/libavformat/http.c +++

Re: [FFmpeg-devel] [PATCH v2 2/9] avformat/http: Use AVERROR_HTTP_TOO_MANY_REQUESTS

2024-04-24 Thread Martin Storsjö
On Mon, 22 Apr 2024, Derek Buitenhuis wrote: Added in thep previous commit. Typo in the commit message // Martin ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link

Re: [FFmpeg-devel] [PATCH] avdevice/avfoundation: fix macOS/iOS/tvOS SDK conditional checks

2024-04-24 Thread Martin Storsjö
On Wed, 17 Apr 2024, Marvin Scholz wrote: This fixes the checks to properly use runtime feature detection and check the SDK version (*_MAX_ALLOWED) instead of the targeted version for the relevant APIs. As these things are pretty hard to think straight about, it could be good with a more

[FFmpeg-devel] [PATCH v2] opusdsp: add ability to modify deemphasis constant

2024-04-24 Thread Lynne
xHE-AAC relies on the same postfilter mechanism that Opus uses to improve clarity (albeit with a steeper deemphasis filter). The code to apply it is identical, it's still just a simple IIR low-pass filter. This commit makes it possible to use alternative constants. Version 2 changes: Move the

Re: [FFmpeg-devel] Fw: Your GSoC Org has 0 Contributors ranked - deadline is 1800 UTC April 24

2024-04-24 Thread Michael Niedermayer
On Wed, Apr 24, 2024 at 07:59:17AM +0200, Thilo Borgmann via ffmpeg-devel wrote: > Hi, > > > adding people who "want to mentor" to CC > > > > thx > > > > On Wed, Apr 24, 2024 at 01:50:31AM +0200, Michael Niedermayer wrote: > > > Hi all > > > > > > I just got the mail below. I assume the other

[FFmpeg-devel] [PATCH v3] lavu/opt: Clarify the scope of AVOptions

2024-04-24 Thread Andrew Sayers
See discussion on the mailing list: https://ffmpeg.org/pipermail/ffmpeg-devel/2024-April/326054.html --- libavutil/opt.h | 10 ++ 1 file changed, 10 insertions(+) diff --git a/libavutil/opt.h b/libavutil/opt.h index e6013662f6..6cf2b39a63 100644 --- a/libavutil/opt.h +++

Re: [FFmpeg-devel] [PATCH 3/5] doc/examples/vaapi_transcode: Simplify loop

2024-04-24 Thread myp...@gmail.com
On Wed, Apr 24, 2024 at 9:45 AM Michael Niedermayer wrote: > > Fixes: CID1428858(1/2) Logically dead code > > Sponsored-by: Sovereign Tech Fund > Signed-off-by: Michael Niedermayer > --- > doc/examples/vaapi_transcode.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git

[FFmpeg-devel] Cease and desist

2024-04-24 Thread Rémi Denis-Courmont
To whom it may concern. Stop your harassment and withdraw your public insults against my person. In light of prompt action from the Cc, I reserve any and all legal avenues against Thilo Borgmann. Le 24 avril 2024 09:07:43 GMT+03:00, Thilo Borgmann via ffmpeg-devel a écrit : >Hi, > >> Thilo

Re: [FFmpeg-devel] FFmpeg table at NAB

2024-04-24 Thread Thilo Borgmann via ffmpeg-devel
Hi, Thilo is calling Kieran, myself and presumably Derek and JB trolls. This seems to be a pattern against anyone who disagrees with him. please avoid top-posting on this list [1]. -Thilo [1] https://ffmpeg.org/mailing-list-faq.html#What-is-top_002dposting_003f-1 Le 22 avril 2024