On Thu, 9 Dec 2021, Niklas Haas wrote:

On Thu, 09 Dec 2021 17:21:58 +0000 Derek Buitenhuis 
<derek.buitenh...@gmail.com> wrote:
On 12/8/2021 10:12 AM, Niklas Haas wrote:
+/* based on guesswork, see mkvtoolnix and dovi_tool */
+int av_dovi_profile(const AVDOVIRpuDataHeader *hdr)
+{

The correct way to find the profile is from the stream level
DOVI configuration record side data, if available.

So, I changed the API to accept the DOVI configuration record, but
actually getting that record into hevcdec isimpossible given the current
design of FFmpeg where stream-level properties are not mode available to
the decoder. Possibilities:

1. Add patch to propagate stream-level properties to
  AVCodecContext.coded_side_data automatically
2. Add patch to automatically propagate stream-level properties to each
  AVPacket somehow?

Don't we already have av_format_inject_global_side_data() for something like this? It is not enabled by default however, only for ffplay, but I am not sure why.

Regards,
Marton

3. Have the code read the profile from the AVPacket even though the side
  data doesn't exist for them, and let this just be an open bug.
4. Something else?

Thoughts?
_______________________________________________
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 "unsubscribe".

_______________________________________________
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 "unsubscribe".

Reply via email to