Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: "J. Dekker" <jdek@itanimul.li>
To: ffmpeg-devel@ffmpeg.org
Subject: Re: [FFmpeg-devel] [PATCH 1/7] lavu: add ecinfo sidedata
Date: Tue, 25 Jul 2023 14:05:43 +0200
Message-ID: <m21qgwdvt0.fsf@itanimul.li> (raw)
In-Reply-To: <CAHGibzHiYSyw9CPtUQm3Nf4P+STDLJv2cPpWXM-8THVUV=gzOw@mail.gmail.com>


Hi Devin,

Devin Heitmueller <devin.heitmueller@ltnglobal.com> writes:
> On Fri, Jul 21, 2023 at 9:38 AM J. Dekker <jdek@itanimul.li> wrote:
>
> I appreciate the value of stats so I can tell that the stream had
> errors, but how is this side data "helpful to a renderer attempting to
> filter or conceal video decoding errors and artifacts" if there is no
> information relating to the region of the picture where the errors are
> present?  Is the assumption that an application will simply have some
> threshold at which it decides to duplicate the previous frame rather
> than showing the current one?

Yes, this is primarily the usecase here initially. As some point a
renderer would rather duplicate previous frame than rely on
reconstruction. This could be done within EC itself but letting the
decoder make this choice rather than exporting the information to a
renderer and doing it there seems incorrect to me.

It is the intention to expand the metadata to provide information about
specific regions in the frame in the future but mainly just providing
the framework for providing this sort of information in the first place
at the moment.

-- 
jd
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

  reply	other threads:[~2023-07-25 12:23 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-21 13:37 J. Dekker
2023-07-21 13:37 ` [FFmpeg-devel] [PATCH 2/7] lavc: add AV_CODEC_EXPORT_DATA_ERROR J. Dekker
2023-07-21 13:37 ` [FFmpeg-devel] [PATCH 3/7] lavc/error_resilience: fill ecinfo J. Dekker
2023-07-21 13:37 ` [FFmpeg-devel] [PATCH 4/7] lavc: set decode_error_flags when ec active J. Dekker
2023-07-21 13:37 ` [FFmpeg-devel] [PATCH 5/7] lavc/h264: export ecinfo J. Dekker
2023-07-21 13:37 ` [FFmpeg-devel] [PATCH 6/7] tools/ffprobe: add ecinfo frame side data J. Dekker
2023-07-21 13:47   ` [FFmpeg-devel] [PATCH 6/7 v1.1] " J. Dekker
2023-07-21 13:37 ` [FFmpeg-devel] [PATCH 7/7] fate: add ecinfo sidedata test J. Dekker
2023-07-21 14:44 ` [FFmpeg-devel] [PATCH 1/7] lavu: add ecinfo sidedata Devin Heitmueller
2023-07-25 12:05   ` J. Dekker [this message]
2023-07-25 22:00     ` Michael Niedermayer
2023-07-21 23:49 ` Michael Niedermayer
     [not found] ` <20230721133746.33335-1-jdek@itanimul.li-N_sf260--B-9>
2023-07-22  0:12   ` Lynne

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=m21qgwdvt0.fsf@itanimul.li \
    --to=jdek@itanimul.li \
    --cc=ffmpeg-devel@ffmpeg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git