Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-06-04 Thread Song, Ruiling
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Mark Thompson > Sent: Monday, June 4, 2018 7:20 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter. > > On 29/0

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-06-03 Thread Mark Thompson
On 29/05/18 06:54, 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

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-31 Thread Song, Ruiling
> -Original Message- > From: Song, Ruiling > Sent: Tuesday, May 29, 2018 4:47 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: RE: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter. > > > > > -Original Me

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-29 Thread Niklas Haas
I see no obvious issues with the algorithm. (Though I haven't tested it) So "LGTM" On Tue, 29 May 2018 13:54:27 +0800, 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

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-29 Thread Song, Ruiling
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of myp...@gmail.com > Sent: Tuesday, May 29, 2018 3:40 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH] lavf

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-29 Thread myp...@gmail.com
2018-05-29 13:54 GMT+08:00 Ruiling Song : > 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

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-24 Thread Niklas Haas
On Thu, 24 May 2018 08:58:22 +, "Song, Ruiling" wrote: > Where comes the "1000 cd/m² is the reference display peak"? seems no clear > statement in BT2100? The concept of there being a "standard" 1000 cd/m² display is introduced in multiple places. Refer to Table 5

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-24 Thread Song, Ruiling
cussions and > patches <ffmpeg-devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter. > > > Excellent explanation. I think I get your idea. Will refine the code per > > your > suggestion. > > But still some question, will peop

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-24 Thread Song, Ruiling
> -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. > >

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-23 Thread Niklas Haas
On Mon, 21 May 2018 14:50:17 +0800, Ruiling Song wrote: > +float3 map_one_pixel_rgb(float3 rgb, float peak, float average) { > +float sig = max(max(rgb.x, max(rgb.y, rgb.z)), 1e-6f); > +// de-saturate > +if (desat_param > 0.0f) { > +float luma =

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-23 Thread Niklas Haas
> Excellent explanation. I think I get your idea. Will refine the code per your > suggestion. > But still some question, will people/tools tend to fill in the mastering > information for HLG video? > I currently see no document that recommend to fill the mastering display for > HLG. > I only

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-22 Thread Song, Ruiling
cussions and > patches <ffmpeg-devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter. > > On Tue, 22 May 2018 08:56:37 +, "Song, Ruiling" <ruiling.s...@intel.com> > wrote: > > Yes, your idea sounds reasonable. But it ma

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-22 Thread Niklas Haas
On Tue, 22 May 2018 08:56:37 +, "Song, Ruiling" wrote: > Yes, your idea sounds reasonable. But it may need much effort to re-structure > the code to make it (that would launch two kernels, and we may need a wait > between them) and evaluate the performance.

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-22 Thread Mark Thompson
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-deve

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-22 Thread Song, Ruiling
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Niklas Haas > Sent: Tuesday, May 22, 2018 10:28 AM > To: ffmpeg-devel@ffmpeg.org > Cc: Mark Thompson <s...@jkqxz.net> > Subject: Re: [FFmpeg-devel] [PATCH] lavfi

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-22 Thread Song, Ruiling
> -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/0

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-21 Thread Niklas Haas
On Tue, 22 May 2018 01:18:30 +0100, Mark Thompson wrote: > 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

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-21 Thread Song, Ruiling
> > Subject: Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter. > > 2018-05-21 14:50 GMT+08:00 Ruiling Song <ruiling.s...@intel.com>: > > This filter does HDR(HDR10/HLG) to SDR conversion with tone-mapping. > > > > An example command to use this filter wit

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-21 Thread Mark Thompson
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

Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

2018-05-21 Thread myp...@gmail.com
2018-05-21 14:50 GMT+08:00 Ruiling Song : > 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