> -----Original Message----- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Mark Thompson > Sent: Tuesday, May 22, 2018 8:41 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter. > > On 22/05/18 09:48, Song, Ruiling wrote: > >> -----Original Message----- > >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of > >> Mark Thompson > >> Sent: Tuesday, May 22, 2018 8:19 AM > >> To: ffmpeg-devel@ffmpeg.org > >> Subject: Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter. > >> > >> On 21/05/18 07:50, Ruiling Song wrote: > >>> This filter does HDR(HDR10/HLG) to SDR conversion with tone-mapping. > >>> > >>> An example command to use this filter with vaapi codecs: > >>> FFMPEG -init_hw_device vaapi=va:/dev/dri/renderD128 -init_hw_device \ > >>> opencl=ocl@va -hwaccel vaapi -hwaccel_device va - > hwaccel_output_format \ > >>> vaapi -i INPUT -filter_hw_device ocl -filter_complex \ > >>> '[0:v]hwmap,tonemap_opencl=t=bt2020:tonemap=linear:format=p010[x1]; > \ > >>> [x1]hwmap=derive_device=vaapi:reverse=1' -c:v hevc_vaapi -profile 2 > >> OUTPUT > >>> > >>> Signed-off-by: Ruiling Song <ruiling.s...@intel.com> > >>> ---
> >>> + > >>> + err = av_frame_copy_props(output, input); > >>> + if (err < 0) > >>> + goto fail; > > av_frame_copy_props() copies the side-data which will include the > mastering/light-level information, but that's no longer valid after > tonemapping? I think so, but I am not sure how to update this information correctly. Using result peak and result color-space primaries to replace original metadata? Sounds ok? Ruiling _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
