The `dec_ctx->time_base` was incorrectly default set by avcodec_open2(), while
`enc_ctx->time_base` was derived from `dec_ctx->framerate`. This mismatch could
cause incorrect video duration in the output.
This patch corrects the issue by adjusting the `enc_ctx->time_base` calculation
to accoun
On 2025-02-02 11:27 pm, Gyan Doshi wrote:
The ref input may have its frame rate unset, which would then lead to
SIGFPE. So fall back to the main link frame rate. If that too is unset,
default to 0.
Related to #11428
Plan to push tomorrow.
Regards,
Gyan
---
libavfilter/vf_xpsnr.c | 5 +
On 2025-02-05 01:17 am, Gyan Doshi wrote:
On 2025-02-04 11:58 pm, Marth64 wrote:
Thank you Timo for removing the flag.
Now users should be able to join correctly.
Patch LGTM. I don't have access to merge to web.
Will push tomorrow morning, if no one else has.
Pushed as e99836bf11abbe38c
To be clear, i want to give an example, i use a 10s duration, 30fps video.
The ifmt_ctx->streams[stream_index]->time_base is same as
ofmt_ctx->streams[stream_index]->time_base after initializing.
The stream->dec_ctx->time_base is default 0,60 but
stream->enc_ctx->time_base is set to av_inv_q(dec_ct
>
> AVCodecContext.time_base is not used for decoding.
Thank you for your reply. I understand that time_base is not used during
decoding, but the transcoding code calls av_packet_rescale_ts twice, once
before decoding and once after encoding, as shown below:
540 if (filter_ctx[stream_
>
> AVCodecContext.time_base is not used for decoding.
Thank you for your reply. I understand that time_base is not used during
decoding, but the transcoding code calls av_packet_rescale_ts twice, once
before decoding and once after encoding, as shown below:
540 if (filter_ctx[stream_
The `dec_ctx->time_base` was incorrectly default set to 0/60, while
`enc_ctx->time_base` was derived from `dec_ctx->framerate`. This
mismatch could cause incorrect video duration in the output.
This patch aligns `enc_ctx->time_base` with `dec_ctx->time_base` to
prevent rescaling issues and ens
From bfd5500a5448ad468d32994816e8a55c0d4a2428 Mon Sep 17 00:00:00 2001
From: Jack Lau
Date: Tue, 4 Feb 2025 21:39:20 +0800
Subject: [PATCH] examples/transcoding: Fix time_base handling
The `dec_ctx->time_base` was incorrectly default set to 0/60, while
`enc_ctx->time_base` was derived from `dec
fix-time-base-handling.patch
Description: Binary data
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubs
--- Begin Message ---
From bfd5500a5448ad468d32994816e8a55c0d4a2428 Mon Sep 17 00:00:00 2001
From: Jack Lau
Date: Tue, 4 Feb 2025 21:39:20 +0800
Subject: [PATCH] examples/transcoding: Fix time_base handling
X-Unsent: 1
To: ffmpeg-devel@ffmpeg.org
The `dec_ctx->time_base` was incorrectly default s
On 03.02.2025 18:38, Zhao Zhili wrote:
Hi Timo,
You have added encoding support for hevc alpha video, may I invite you
to help review the decoding support?
I'm a bit busy with work at the moment, but I'll get to it.
Don't have the most experience with that code, but I can at least give
it a l
On 1/28/25 4:44 PM, Michael Niedermayer wrote:
Hi
On Tue, Jan 28, 2025 at 10:12:30PM +0200, Jan Ekström wrote:
On Tue, Jan 28, 2025 at 4:24 PM Michael Niedermayer
wrote:
Maybe fixes: 11435
Do I understand correctly that the root issue that's being attempted
to be fixed by the initial pa
On Mon, 16 Dec 2024, Dennis Sädtler via ffmpeg-devel wrote:
From: Dennis Sädtler
Added in macOS 15 "Sequoia".
Signed-off-by: Dennis Sädtler
---
Fixed line-endings, otherwise identical to v1.
libavcodec/videotoolboxenc.c | 12
1 file changed, 12 insertions(+)
diff --git a/libavc
It's less confusing to have ff_frame_new_side_data_from_buf() be a drop in
replacement for av_frame_side_data_add(), and the addition of the missing flags
field will make it more versatile, as will be seen in the following commit.
Signed-off-by: James Almer
---
libavcodec/av1dec.c | 6 +++
Signed-off-by: James Almer
---
libavcodec/hevc/hevcdec.c | 11 +++
1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/libavcodec/hevc/hevcdec.c b/libavcodec/hevc/hevcdec.c
index cf4062c75c..24d801c39f 100644
--- a/libavcodec/hevc/hevcdec.c
+++ b/libavcodec/hevc/hevcdec.c
@@ -30
Add a parameter (boolean) r to switch the area from
rectangular (r=0) to circular(r=1) delogo.
Signed-off-by: Jörg Habenicht
---
libavfilter/vf_delogo.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/libavfilter/vf_delogo.c b/libavfilter/vf_delogo.c
index bfbf8e6f4c..
Use the corrected x,y,w,h values with boundaries and clipping to
calculate the points inside the ellipse.
Signed-off-by: Jörg Habenicht
---
libavfilter/vf_delogo.c | 28 ++--
1 file changed, 26 insertions(+), 2 deletions(-)
diff --git a/libavfilter/vf_delogo.c b/libavfil
Dear list,
sorry for the previous posting. I noticed the lines got mangled. So here
is a new try to get the patches right.
Old text:
I made a patch for the delogo filter. With this I can create a circular or
elliptic zone to apply the delogo algorithm. The borders are defined with
x,y,w,h parame
Added a description of the elliptic formed delogo algorithm. Placed it
outside the doxygen function comment.
Signed-off-by: Jörg Habenicht
---
libavfilter/vf_delogo.c | 24
1 file changed, 24 insertions(+)
diff --git a/libavfilter/vf_delogo.c b/libavfilter/vf_delogo.c
i
On 2025-02-04 11:58 pm, Marth64 wrote:
Thank you Timo for removing the flag.
Now users should be able to join correctly.
Patch LGTM. I don't have access to merge to web.
Will push tomorrow morning, if no one else has.
Regards,
Gyan
___
ffmpeg-dev
Thank you Timo for removing the flag.
Now users should be able to join correctly.
Patch LGTM. I don't have access to merge to web.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visi
On 04.02.2025 17:30, James Almer wrote:
Including Mastering Display and Content Light Level.
Requires SDK 13.0, and only supports HEVC and AV1.
Signed-off-by: James Almer
---
libavcodec/nvenc.c | 82 ++
libavcodec/nvenc.h | 2 ++
2 files changed,
When seeking on multi-angle titles, libdvdnav does not lock on
to the correct sectors initially as it seeks to find the right NAV packet.
This manifests itself as two bugs:
(1) When seeking on the first angle in a multi-angle segment,
frames from another angle will appear (for example in intro
or
Including Mastering Display and Content Light Level.
Requires SDK 13.0, and only supports HEVC and AV1.
Signed-off-by: James Almer
---
libavcodec/nvenc.c | 82 ++
libavcodec/nvenc.h | 2 ++
2 files changed, 84 insertions(+)
diff --git a/libavcodec/nv
On 04/02/2025 15:58, Zhao Zhili wrote:
On Feb 4, 2025, at 22:00, Martin Storsjö wrote:
On Tue, 4 Feb 2025, James Almer wrote:
No, they don't need to match afaik. The headers are meant to be ABI backwards
compatible (hence all the reserved fields).
Right, then there's probably less concer
The `dec_ctx->time_base` was incorrectly default set to 0/60, while
`enc_ctx->time_base` was derived from `dec_ctx->framerate`. This mismatch could
cause incorrect video duration in the output.
This patch aligns `enc_ctx->time_base` with `dec_ctx->time_base` to prevent
rescaling issues and ensu
The `dec_ctx->time_base` was incorrectly default set to 0/60, while
`enc_ctx->time_base` was derived from `dec_ctx->framerate`. This mismatch
could cause incorrect video duration in the output.
This patch aligns `enc_ctx->time_base` with `dec_ctx->time_base` to prevent
rescaling issues and ensure
> On Feb 4, 2025, at 22:00, Martin Storsjö wrote:
>
> On Tue, 4 Feb 2025, James Almer wrote:
>
>> No, they don't need to match afaik. The headers are meant to be ABI
>> backwards compatible (hence all the reserved fields).
>
> Right, then there's probably less concern about updating them.
>
On Tue, 4 Feb 2025, James Almer wrote:
No, they don't need to match afaik. The headers are meant to be ABI backwards
compatible (hence all the reserved fields).
Right, then there's probably less concern about updating them.
I was just wondering if the headers were provided by your distro or n
On 2/4/2025 10:52 AM, Martin Storsjö wrote:
On Tue, 4 Feb 2025, James Almer wrote:
Thanks, this does indeed seem to fix compilation in my setup.
Applied.
Why don't don't just update the headers? Is it a system package
provided by your distro?
I guess I could, but as long as I don't have a
On Tue, 4 Feb 2025, James Almer wrote:
Thanks, this does indeed seem to fix compilation in my setup.
Applied.
Why don't don't just update the headers? Is it a system package provided by
your distro?
I guess I could, but as long as I don't have a pressing need to, just
staying with the cur
Hi Rémi,
On Wed, 15 Jan 2025, Rémi Bernon wrote:
Some files keep extra metadata such as 'name' fields within udta, and
it is useful for Wine to access them with the "export_all" option so
they can then be exposed to Windows applications.
Signed-off-by: Rémi Bernon
---
libavformat/mov.c | 10 +
On 2/4/2025 10:38 AM, Martin Storsjö wrote:
On Tue, 4 Feb 2025, James Almer wrote:
Fixes compilation when using API headers older than 12.0
Signed-off-by: James Almer
---
libavcodec/nvenc.c | 2 ++
libavcodec/nvenc.h | 5 +
2 files changed, 7 insertions(+)
diff --git a/libavcodec/nvenc.c b
On Tue, 4 Feb 2025, James Almer wrote:
Fixes compilation when using API headers older than 12.0
Signed-off-by: James Almer
---
libavcodec/nvenc.c | 2 ++
libavcodec/nvenc.h | 5 +
2 files changed, 7 insertions(+)
diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c
index c359c2bc8a..eea3e8d
On Tue, 4 Feb 2025, Timo Rothenpieler wrote:
On 04/02/2025 13:25, Martin Storsjö wrote:
On Mon, 3 Feb 2025, Zhao Zhili wrote:
ffmpeg | branch: master | Zhao Zhili | Thu Jan 23
22:17:29 2025 +0800| [1438f6997db70945173f01aea1768e3b27ce2679] |
committer: Zhao Zhili
avcodec/nvenc: Enable rec
On 2/4/2025 10:23 AM, Martin Storsjö wrote:
On Tue, 4 Feb 2025, Timo Rothenpieler wrote:
On 04/02/2025 13:25, Martin Storsjö wrote:
On Mon, 3 Feb 2025, Zhao Zhili wrote:
ffmpeg | branch: master | Zhao Zhili | Thu
Jan 23 22:17:29 2025 +0800|
[1438f6997db70945173f01aea1768e3b27ce2679] | comm
Fixes compilation when using API headers older than 12.0
Signed-off-by: James Almer
---
libavcodec/nvenc.c | 2 ++
libavcodec/nvenc.h | 5 +
2 files changed, 7 insertions(+)
diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c
index c359c2bc8a..eea3e8d703 100644
--- a/libavcodec/nvenc.c
+++
On 04/02/2025 13:25, Martin Storsjö wrote:
On Mon, 3 Feb 2025, Zhao Zhili wrote:
ffmpeg | branch: master | Zhao Zhili | Thu Jan
23 22:17:29 2025 +0800| [1438f6997db70945173f01aea1768e3b27ce2679] |
committer: Zhao Zhili
avcodec/nvenc: Enable recovery point SEI for intra refresh mode
Otherwi
This is the new FATE test.
Test samples are available here:
https://www.dropbox.com/scl/fo/fxt2edwkyj2mjc9qubku5/AICHxJyxMMAK8MIJqWLcvk4?rlkey=mlt12lsu741ejukz0p5qtn9rq&dl=0
Output prior to the changes is:
Stream ID: 0, codec name: flac, metadata: encoder=Lavc61.19.100
flac:title=First Stream
S
This is a series of 2 patches to allow proper decoding of ogg metadata
in chained ogg/flac streams.
ogg/flac streams are pretty important because there are perhaps the only
combination of lossless audio codec and open-source container that
allows for proper transmittion of lossless audio data accr
libavformat/oggparseflac: Parse ogg/flac comments in new ogg packets,
add them to ogg stream new_metadata.
libavcodec/flacdec: Process AV_PKT_DATA_METADATA_UPDATE on new packets,
add them as new metadata on the new decoded audio frame.
---
libavcodec/flacdec.c | 20 +++-
l
On Mon, 3 Feb 2025, Zhao Zhili wrote:
ffmpeg | branch: master | Zhao Zhili | Thu Jan 23
22:17:29 2025 +0800| [1438f6997db70945173f01aea1768e3b27ce2679] | committer: Zhao
Zhili
avcodec/nvenc: Enable recovery point SEI for intra refresh mode
Otherwise all frames can be dropped after seek with
Le mar. 4 févr. 2025 à 06:15, Romain Beauxis
a écrit :
>
> This is a series of 3 patches to allow proper decoding of ogg metadata
> in chained ogg/flac streams.
>
> ogg/flac streams are pretty important because there are perhaps the only
> combination of lossless audio codec and open-source contai
On Tue, 28 Jan 2025 at 22:44, Michael Niedermayer
wrote:
>
> Hi
>
> On Tue, Jan 28, 2025 at 10:12:30PM +0200, Jan Ekström wrote:
> > On Tue, Jan 28, 2025 at 4:24 PM Michael Niedermayer
> > wrote:
> > >
> > > Maybe fixes: 11435
> > >
> >
> > Do I understand correctly that the root issue that's bei
This is the new FATE test.
Test samples are available here:
https://www.dropbox.com/scl/fo/fxt2edwkyj2mjc9qubku5/AICHxJyxMMAK8MIJqWLcvk4?rlkey=mlt12lsu741ejukz0p5qtn9rq&dl=0
Output prior to the changes is:
Stream ID: 0, codec name: flac, metadata: encoder=Lavc61.19.100
flac:title=First Stream
S
This is the main patch. Parse ogg comments in ogg packets with comment,
keep them in pending_metadata and copy them to the next decoded audio
frame.
---
libavcodec/flacdec.c | 12 +++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/libavcodec/flacdec.c b/libavcodec/flacdec.
Firt argument in these function is only used to pass to av_log. This
makes it possible to re-use them with any type of AVClass struct.
---
libavformat/oggdec.h | 5 -
libavformat/oggparsevorbis.c | 4 ++--
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/libavformat/oggd
This is a series of 3 patches to allow proper decoding of ogg metadata
in chained ogg/flac streams.
ogg/flac streams are pretty important because there are perhaps the only
combination of lossless audio codec and open-source container that
allows for proper transmittion of lossless audio data accr
Le jeu. 30 janv. 2025 à 08:08, Romain Beauxis a
écrit :
>
>
>
>
> Le mer. 29 janv. 2025 à 17:40, Marvin Scholz a écrit
:
> >
> >
> >
> > On 29 Jan 2025, at 15:40, Romain Beauxis wrote:
> >
> > > This patch makes sure that ogg/flac headers are parsed again when
> > > encountering a new logic strea
49 matches
Mail list logo
