Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add YUV color space metadata to AVCodec

2024-04-03 Thread Niklas Haas
On Sun, 24 Mar 2024 14:04:37 +0100 Andreas Rheinhardt wrote: > Niklas Haas: > > On Mon, 05 Feb 2024 19:04:30 +0100 Andreas Rheinhardt > > wrote: > >> This presumes the relevant states to be a cartesian product. Which need > >> not be true. A callback would be better; this would also allow to

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add YUV color space metadata to AVCodec

2024-03-23 Thread Niklas Haas
On Fri, 09 Feb 2024 13:11:38 +0100 Niklas Haas wrote: > On Mon, 05 Feb 2024 19:04:30 +0100 Andreas Rheinhardt > wrote: > > This presumes the relevant states to be a cartesian product. Which need > > not be true. A callback would be better; this would also allow to base > > the list on other

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add YUV color space metadata to AVCodec

2024-02-09 Thread Niklas Haas
On Mon, 05 Feb 2024 19:04:30 +0100 Andreas Rheinhardt wrote: > This presumes the relevant states to be a cartesian product. Which need > not be true. A callback would be better; this would also allow to base > the list on other values already set in an AVCodecContext. And if this > were

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add YUV color space metadata to AVCodec

2024-02-08 Thread Niklas Haas
On Thu, 08 Feb 2024 13:33:51 +0100 Andreas Rheinhardt wrote: > Sorry for not answering earlier. > My intention is to allow users who only want to deal with the common > case of a cartesian product to continue to do so, but to also support > other usecases. > The public function would look like >

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add YUV color space metadata to AVCodec

2024-02-08 Thread Niklas Haas
On Mon, 05 Feb 2024 19:37:48 +0100 Niklas Haas wrote: > On Mon, 05 Feb 2024 19:04:30 +0100 Andreas Rheinhardt > wrote: > > This presumes the relevant states to be a cartesian product. Which need > > not be true. A callback would be better; this would also allow to base > > the list on other

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add YUV color space metadata to AVCodec

2024-02-05 Thread Niklas Haas
On Mon, 05 Feb 2024 19:04:30 +0100 Andreas Rheinhardt wrote: > This presumes the relevant states to be a cartesian product. Which need > not be true. A callback would be better; this would also allow to base > the list on other values already set in an AVCodecContext. And if this > were

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add YUV color space metadata to AVCodec

2024-02-05 Thread Niklas Haas
It would make me happy if we could get this API addition in before 7.0, even though it's non-functional on its own, as it sets the groundwork to be able to start properly deprecating YUVJ. After this series, the next steps are: 1. Make decoders stop outputting YUVJ frames 2. Mark the actual YUVJ

[FFmpeg-devel] [PATCH 1/2] avcodec: add YUV color space metadata to AVCodec

2024-02-05 Thread Niklas Haas
From: Niklas Haas This is motivated primarily by a desire for YUVJ removal, which will require signalling the supported color ranges as part of the codec capabilities. But since we're adding YUV range, we might as well add the YUV color matrix as well - since some codecs (e.g. VP8, JPEG) only