[FFmpeg-devel] [PATCH] lavc/hevc Parse SEI_TYPE_MASTERING_DISPLAY_INFO and propagate contents into the AVMasteringDisplayMetadata side data.

[Michael Niedermayer]

On Mon, Feb 08, 2016 at 09:36:42AM -0800, Neil Birkbeck wrote:
> I'm sending an updated patch that persists the data within the CVS
> (between IRAP access units with no_rasl_output_flag=1). If this seems
> like overkill, we can fallback to Hendrik's suggestion of just sending
> it once. I did consider this alternative but figured the side data
> could get lost if some frames were dropped in some part of the filter
> chain.
> 
> 
> On Thu, Feb 4, 2016 at 11:04 PM, Hendrik Leppkes <h.leppkes at gmail.com> wrote:
> > On Fri, Feb 5, 2016 at 3:05 AM, Neil Birkbeck <neil.birkbeck at gmail.com> wrote:
> >> According to the ITU-T H.265 v3, in Table D - 1, the persistence scope
> >> of the mastering display metadata is "The Coded Video Sequence (CVS)
> >> containing the SEI message". So I guess we want to clear when we start
> >> the next CVS, so I guess when we see the next IDR frame.
> >>
> >> Given that the SEI comes before the IDR, I don't see a great way to do
> >> this aside from initializing the flag to 2 and decrementing (when > 0)
> >> when we see an IDR.
> >>
> >
> > The usual way I would handle such SEI metadata is just to send the
> > sidedata once when it appears in the bitstream and not repeat it with
> > every single frame, and let the user worry about its lifetime.
> >
> > - Hendrik
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

>  hevc.c     |   52 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>  hevc.h     |    7 +++++++
>  hevc_sei.c |   26 ++++++++++++++++++++++++++
>  version.h  |    2 +-
>  4 files changed, 86 insertions(+), 1 deletion(-)
> 6b6743db49b2982e684f6ef2df94d336bae76947  0001-lavc-hevc-Parse-SEI_TYPE_MASTERING_DISPLAY_INFO-and-.patch
> From e6602d1191a9166e5a38c4c27b430fbe0b84aeca Mon Sep 17 00:00:00 2001
> From: Neil Birkbeck <neil.birkbeck at gmail.com>
> Date: Thu, 21 Jan 2016 10:56:50 -0800
> Subject: [PATCH] lavc/hevc Parse SEI_TYPE_MASTERING_DISPLAY_INFO and propagate
>  content into the AVMasteringDisplayMetadata side data.
> 
> Add support for parsing SEI_TYPE_MASTERING_DISPLAY_INFO and propagate contents into
> the AVMasteringDisplayMetadata side data. Primaries are ordered in RGB order and
> the values are converted to rationals ([0,1] for CEI 1931 Chroma coords,
> and cd/m^2 for luma).
> 
> Signed-off-by: Neil Birkbeck <neil.birkbeck at gmail.com>
> ---
>  libavcodec/hevc.c     | 52 +++++++++++++++++++++++++++++++++++++++++++++++++++
>  libavcodec/hevc.h     |  7 +++++++
>  libavcodec/hevc_sei.c | 26 ++++++++++++++++++++++++++
>  libavcodec/version.h  |  2 +-
>  4 files changed, 86 insertions(+), 1 deletion(-)

applied

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160214/a9f1908f/attachment.sig>