Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-25 Thread Michael Niedermayer
On Mon, Oct 23, 2023 at 05:56:48PM -0700, Vittorio Giovara wrote: > On Sat, Oct 21, 2023 at 4:20 PM Michael Niedermayer > wrote: > > > On Fri, Oct 20, 2023 at 07:13:00PM -0400, Ronald S. Bultje wrote: > > > Hi, > > > > > > On Fri, Oct 20, 2023 at 12:14 PM Vittorio Giovara < > > > vittorio.giov...

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-23 Thread Vittorio Giovara
On Sat, Oct 21, 2023 at 4:20 PM Michael Niedermayer wrote: > On Fri, Oct 20, 2023 at 07:13:00PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Fri, Oct 20, 2023 at 12:14 PM Vittorio Giovara < > > vittorio.giov...@gmail.com> wrote: > > > > > tbh while it's great that this functionality exists *

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-21 Thread Michael Niedermayer
On Fri, Oct 20, 2023 at 07:13:00PM -0400, Ronald S. Bultje wrote: > Hi, > > On Fri, Oct 20, 2023 at 12:14 PM Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > > > tbh while it's great that this functionality exists *somewhere* in ffmpeg, > > this is really something that should belong to

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-20 Thread Ronald S. Bultje
Hi, On Fri, Oct 20, 2023 at 12:14 PM Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > tbh while it's great that this functionality exists *somewhere* in ffmpeg, > this is really something that should belong to the scaler filter (whichever > implementation) > I agree. The reason this was d

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-20 Thread Vittorio Giovara
On Fri, Oct 20, 2023 at 7:30 AM Niklas Haas wrote: > On Fri, 13 Oct 2023 14:33:11 -0400 Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > > On Fri, Oct 13, 2023 at 10:27 AM Niklas Haas wrote: > > > > > Changes since v1: > > > > > > - Remove unneeded patch (AVCodecContext.colorspace init)

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-20 Thread Vittorio Giovara
On Fri, Oct 20, 2023 at 8:14 AM Ronald S. Bultje wrote: > Hi, > > On Sat, Oct 14, 2023 at 11:16 AM Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > > > On Sat, Oct 14, 2023 at 9:11 AM Lynne wrote: > > > colorspace doesn't make it impossible to introduce all that is needed. > > > It's a

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-20 Thread Ronald S. Bultje
Hi, On Sat, Oct 14, 2023 at 11:16 AM Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > On Sat, Oct 14, 2023 at 9:11 AM Lynne wrote: > > colorspace doesn't make it impossible to introduce all that is needed. > > It's a cleaner codebase that we can extend. > > * that only works on a subset o

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-20 Thread Niklas Haas
On Fri, 13 Oct 2023 14:33:11 -0400 Vittorio Giovara wrote: > On Fri, Oct 13, 2023 at 10:27 AM Niklas Haas wrote: > > > Changes since v1: > > > > - Remove unneeded patch (AVCodecContext.colorspace init) > > - Merge auto-range conversion into auto-scale filter > > - Replace vf_zscale by vf_colors

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-14 Thread Lynne
Oct 14, 2023, 17:16 by vittorio.giov...@gmail.com: > On Sat, Oct 14, 2023 at 9:11 AM Lynne wrote: > >> Oct 14, 2023, 00:22 by vittorio.giov...@gmail.com: >> >> > On Fri, Oct 13, 2023 at 5:14 PM Lynne wrote: >> > >> >> Oct 13, 2023, 20:33 by vittorio.giov...@gmail.com: >> >> >> >> > On Fri, Oct 1

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-14 Thread Vittorio Giovara
On Sat, Oct 14, 2023 at 9:11 AM Lynne wrote: > Oct 14, 2023, 00:22 by vittorio.giov...@gmail.com: > > > On Fri, Oct 13, 2023 at 5:14 PM Lynne wrote: > > > >> Oct 13, 2023, 20:33 by vittorio.giov...@gmail.com: > >> > >> > On Fri, Oct 13, 2023 at 10:27 AM Niklas Haas > wrote: > >> > > >> >> Chang

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-14 Thread Lynne
Oct 14, 2023, 00:22 by vittorio.giov...@gmail.com: > On Fri, Oct 13, 2023 at 5:14 PM Lynne wrote: > >> Oct 13, 2023, 20:33 by vittorio.giov...@gmail.com: >> >> > On Fri, Oct 13, 2023 at 10:27 AM Niklas Haas wrote: >> > >> >> Changes since v1: >> >> >> >> - Remove unneeded patch (AVCodecContext.c

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-13 Thread Vittorio Giovara
On Fri, Oct 13, 2023 at 5:14 PM Lynne wrote: > Oct 13, 2023, 20:33 by vittorio.giov...@gmail.com: > > > On Fri, Oct 13, 2023 at 10:27 AM Niklas Haas wrote: > > > >> Changes since v1: > >> > >> - Remove unneeded patch (AVCodecContext.colorspace init) > >> - Merge auto-range conversion into auto-s

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-13 Thread Lynne
Oct 13, 2023, 20:33 by vittorio.giov...@gmail.com: > On Fri, Oct 13, 2023 at 10:27 AM Niklas Haas wrote: > >> Changes since v1: >> >> - Remove unneeded patch (AVCodecContext.colorspace init) >> - Merge auto-range conversion into auto-scale filter >> - Replace vf_zscale by vf_colorspace in fftools

Re: [FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

2023-10-13 Thread Vittorio Giovara
On Fri, Oct 13, 2023 at 10:27 AM Niklas Haas wrote: > Changes since v1: > > - Remove unneeded patch (AVCodecContext.colorspace init) > - Merge auto-range conversion into auto-scale filter > - Replace vf_zscale by vf_colorspace in fftools > Why is this? I haven't checked what vf_colorspace suppor