Re: [FFmpeg-devel] [RFC PATCH] libavcodec/libopenjpeg: pix fmt selection change

2020-07-19 Thread Gautam Ramakrishnan
On Sun, Jul 19, 2020 at 12:09 AM Carl Eugen Hoyos wrote: > > Am Di., 9. Juni 2020 um 20:25 Uhr schrieb Carl Eugen Hoyos > : > > > > Am Di., 9. Juni 2020 um 19:12 Uhr schrieb Gautam Ramakrishnan > > : > > > > > > On Tue, Jun 9, 2020 at 10:24 PM Carl Eugen Hoyos > > > wrote: > > > > > > > > Am

Re: [FFmpeg-devel] [RFC PATCH] libavcodec/libopenjpeg: pix fmt selection change

2020-07-18 Thread Carl Eugen Hoyos
Am Di., 9. Juni 2020 um 20:25 Uhr schrieb Carl Eugen Hoyos : > > Am Di., 9. Juni 2020 um 19:12 Uhr schrieb Gautam Ramakrishnan > : > > > > On Tue, Jun 9, 2020 at 10:24 PM Carl Eugen Hoyos wrote: > > > > > > Am Di., 9. Juni 2020 um 14:07 Uhr schrieb : > > > > > > > > From: Gautam Ramakrishnan > >

Re: [FFmpeg-devel] [RFC PATCH] libavcodec/libopenjpeg: pix fmt selection change

2020-06-12 Thread Gautam Ramakrishnan
On Thu, Jun 11, 2020 at 9:41 PM Michael Bradshaw wrote: > > On Thu, Jun 11, 2020 at 9:42 AM Gautam Ramakrishnan > wrote: > > > Got it. In that case we can safely ignore the patch to fix libopenjpeg. > > However, p1_03.j2k is one of the 2 files to have ppm marker. How could I > > validate a patch

Re: [FFmpeg-devel] [RFC PATCH] libavcodec/libopenjpeg: pix fmt selection change

2020-06-11 Thread Michael Bradshaw
On Thu, Jun 11, 2020 at 9:42 AM Gautam Ramakrishnan wrote: > Got it. In that case we can safely ignore the patch to fix libopenjpeg. > However, p1_03.j2k is one of the 2 files to have ppm marker. How could I > validate a patch to add ppm marker? I need something to cross validate. > Any

Re: [FFmpeg-devel] [RFC PATCH] libavcodec/libopenjpeg: pix fmt selection change

2020-06-11 Thread Gautam Ramakrishnan
On Thu, Jun 11, 2020 at 11:42 AM Michael Bradshaw wrote: > > On Wed, Jun 10, 2020 at 9:35 PM Gautam Ramakrishnan > wrote: > > > The reference file has 4 components, Whereas all the Bayer formats > > have 3 components. Are we missing any Bayer pixel format in ffmpeg? > > Also, any other ideas on

Re: [FFmpeg-devel] [RFC PATCH] libavcodec/libopenjpeg: pix fmt selection change

2020-06-11 Thread Michael Bradshaw
On Wed, Jun 10, 2020 at 9:35 PM Gautam Ramakrishnan wrote: > The reference file has 4 components, Whereas all the Bayer formats > have 3 components. Are we missing any Bayer pixel format in ffmpeg? > Also, any other ideas on what has to be done for the 2 reference files > mentioned? If this

Re: [FFmpeg-devel] [RFC PATCH] libavcodec/libopenjpeg: pix fmt selection change

2020-06-10 Thread Gautam Ramakrishnan
On Tue, Jun 9, 2020 at 11:55 PM Carl Eugen Hoyos wrote: > > Am Di., 9. Juni 2020 um 19:12 Uhr schrieb Gautam Ramakrishnan > : > > > > On Tue, Jun 9, 2020 at 10:24 PM Carl Eugen Hoyos wrote: > > > > > > Am Di., 9. Juni 2020 um 14:07 Uhr schrieb : > > > > > > > > From: Gautam Ramakrishnan > > > >

Re: [FFmpeg-devel] [RFC PATCH] libavcodec/libopenjpeg: pix fmt selection change

2020-06-09 Thread Carl Eugen Hoyos
Am Di., 9. Juni 2020 um 19:12 Uhr schrieb Gautam Ramakrishnan : > > On Tue, Jun 9, 2020 at 10:24 PM Carl Eugen Hoyos wrote: > > > > Am Di., 9. Juni 2020 um 14:07 Uhr schrieb : > > > > > > From: Gautam Ramakrishnan > > > > > > This patch makes selection of pix_fmt similar to > > > that in the

Re: [FFmpeg-devel] [RFC PATCH] libavcodec/libopenjpeg: pix fmt selection change

2020-06-09 Thread Gautam Ramakrishnan
On Tue, Jun 9, 2020 at 10:24 PM Carl Eugen Hoyos wrote: > > Am Di., 9. Juni 2020 um 14:07 Uhr schrieb : > > > > From: Gautam Ramakrishnan > > > > This patch makes selection of pix_fmt similar to > > that in the native decoder. This makes samples such > > as p0_05.j2k and p1_03.j2k decodable by

Re: [FFmpeg-devel] [RFC PATCH] libavcodec/libopenjpeg: pix fmt selection change

2020-06-09 Thread Carl Eugen Hoyos
Am Di., 9. Juni 2020 um 14:07 Uhr schrieb : > > From: Gautam Ramakrishnan > > This patch makes selection of pix_fmt similar to > that in the native decoder. This makes samples such > as p0_05.j2k and p1_03.j2k decodable by libopenjpeg. Since both files are not YUVA420P, I am not sure if this

Re: [FFmpeg-devel] [RFC PATCH] libavcodec/libopenjpeg: pix fmt selection change

2020-06-09 Thread Michael Bradshaw
On Tue, Jun 9, 2020 at 6:07 AM wrote: > From: Gautam Ramakrishnan > > This patch makes selection of pix_fmt similar to > that in the native decoder. This makes samples such > as p0_05.j2k and p1_03.j2k decodable by libopenjpeg. > --- > libavcodec/libopenjpegdec.c | 21 +++-- >

[FFmpeg-devel] [RFC PATCH] libavcodec/libopenjpeg: pix fmt selection change

2020-06-09 Thread gautamramk
From: Gautam Ramakrishnan This patch makes selection of pix_fmt similar to that in the native decoder. This makes samples such as p0_05.j2k and p1_03.j2k decodable by libopenjpeg. --- libavcodec/libopenjpegdec.c | 21 +++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff