* [FFmpeg-devel] [PATCH] MAINTAINERS: add Niklas Haas for vf_libplacebo.c
@ 2022-03-28 18:33 Niklas Haas
2022-03-28 20:14 ` James Almer
0 siblings, 1 reply; 7+ messages in thread
From: Niklas Haas @ 2022-03-28 18:33 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Niklas Haas
From: Niklas Haas <git@haasn.dev>
So I can merge my own changes to this filter after they pass peer
review, as well as keeping it in sync with upstream API changes / new
features.
Signed-off-by: Niklas Haas <git@haasn.dev>
---
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 931cf4bd2c..76e1332ad8 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -354,6 +354,7 @@ Filters:
vf_il.c Paul B Mahol
vf_(t)interlace Thomas Mundt (CC <thomas.mundt@hr.de>)
vf_lenscorrection.c Daniel Oberhoff
+ vf_libplacebo.c Niklas Haas
vf_mergeplanes.c Paul B Mahol
vf_mestimate.c Davinder Singh
vf_minterpolate.c Davinder Singh
@@ -620,6 +621,7 @@ Loren Merritt ABD9 08F4 C920 3F65 D8BE 35D7 1540 DAA7 060F 56DE
Lynne FE50 139C 6805 72CA FD52 1F8D A2FE A5F0 3F03 4464
Michael Niedermayer 9FF2 128B 147E F673 0BAD F133 611E C787 040B 0FAB
Nicolas George 24CE 01CE 9ACC 5CEB 74D8 8D9D B063 D997 36E5 4C93
+Niklas Haas (haasn) 1DDB 8076 B14D 5B48 32FC 99D9 EB52 DA9C 02BA 6FB4
Nikolay Aleksandrov 8978 1D8C FB71 588E 4B27 EAA8 C4F0 B5FC E011 13B1
Panagiotis Issaris 6571 13A3 33D9 3726 F728 AA98 F643 B12E ECF3 E029
Peter Ross A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B
--
2.35.1
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] MAINTAINERS: add Niklas Haas for vf_libplacebo.c
2022-03-28 18:33 [FFmpeg-devel] [PATCH] MAINTAINERS: add Niklas Haas for vf_libplacebo.c Niklas Haas
@ 2022-03-28 20:14 ` James Almer
2022-03-28 20:31 ` Lynne
0 siblings, 1 reply; 7+ messages in thread
From: James Almer @ 2022-03-28 20:14 UTC (permalink / raw)
To: ffmpeg-devel
On 3/28/2022 3:33 PM, Niklas Haas wrote:
> From: Niklas Haas <git@haasn.dev>
>
> So I can merge my own changes to this filter after they pass peer
> review, as well as keeping it in sync with upstream API changes / new
> features.
>
> Signed-off-by: Niklas Haas <git@haasn.dev>
> ---
> MAINTAINERS | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 931cf4bd2c..76e1332ad8 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -354,6 +354,7 @@ Filters:
> vf_il.c Paul B Mahol
> vf_(t)interlace Thomas Mundt (CC <thomas.mundt@hr.de>)
> vf_lenscorrection.c Daniel Oberhoff
> + vf_libplacebo.c Niklas Haas
> vf_mergeplanes.c Paul B Mahol
> vf_mestimate.c Davinder Singh
> vf_minterpolate.c Davinder Singh
> @@ -620,6 +621,7 @@ Loren Merritt ABD9 08F4 C920 3F65 D8BE 35D7 1540 DAA7 060F 56DE
> Lynne FE50 139C 6805 72CA FD52 1F8D A2FE A5F0 3F03 4464
> Michael Niedermayer 9FF2 128B 147E F673 0BAD F133 611E C787 040B 0FAB
> Nicolas George 24CE 01CE 9ACC 5CEB 74D8 8D9D B063 D997 36E5 4C93
> +Niklas Haas (haasn) 1DDB 8076 B14D 5B48 32FC 99D9 EB52 DA9C 02BA 6FB4
> Nikolay Aleksandrov 8978 1D8C FB71 588E 4B27 EAA8 C4F0 B5FC E011 13B1
> Panagiotis Issaris 6571 13A3 33D9 3726 F728 AA98 F643 B12E ECF3 E029
> Peter Ross A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B
Applied.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] MAINTAINERS: add Niklas Haas for vf_libplacebo.c
2022-03-28 20:14 ` James Almer
@ 2022-03-28 20:31 ` Lynne
2022-03-28 20:44 ` Hendrik Leppkes
2022-03-28 21:13 ` Niklas Haas
0 siblings, 2 replies; 7+ messages in thread
From: Lynne @ 2022-03-28 20:31 UTC (permalink / raw)
To: FFmpeg development discussions and patches
28 Mar 2022, 22:14 by jamrial@gmail.com:
> On 3/28/2022 3:33 PM, Niklas Haas wrote:
>
>> From: Niklas Haas <git@haasn.dev>
>>
>> So I can merge my own changes to this filter after they pass peer
>> review, as well as keeping it in sync with upstream API changes / new
>> features.
>>
>> Signed-off-by: Niklas Haas <git@haasn.dev>
>> ---
>> MAINTAINERS | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 931cf4bd2c..76e1332ad8 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -354,6 +354,7 @@ Filters:
>> vf_il.c Paul B Mahol
>> vf_(t)interlace Thomas Mundt (CC <thomas.mundt@hr.de>)
>> vf_lenscorrection.c Daniel Oberhoff
>> + vf_libplacebo.c Niklas Haas
>> vf_mergeplanes.c Paul B Mahol
>> vf_mestimate.c Davinder Singh
>> vf_minterpolate.c Davinder Singh
>> @@ -620,6 +621,7 @@ Loren Merritt ABD9 08F4 C920 3F65 D8BE 35D7 1540 DAA7 060F 56DE
>> Lynne FE50 139C 6805 72CA FD52 1F8D A2FE A5F0 3F03 4464
>> Michael Niedermayer 9FF2 128B 147E F673 0BAD F133 611E C787 040B 0FAB
>> Nicolas George 24CE 01CE 9ACC 5CEB 74D8 8D9D B063 D997 36E5 4C93
>> +Niklas Haas (haasn) 1DDB 8076 B14D 5B48 32FC 99D9 EB52 DA9C 02BA 6FB4
>> Nikolay Aleksandrov 8978 1D8C FB71 588E 4B27 EAA8 C4F0 B5FC E011 13B1
>> Panagiotis Issaris 6571 13A3 33D9 3726 F728 AA98 F643 B12E ECF3 E029
>> Peter Ross A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B
>>
>
> Applied.
>
I'm not okay with this. I don't mind anyone pushing approved patches,
but I still want to review and have final say on all Vulkan code. Could you
change that?
I think haasn was more of a maintainer on the film grain stuff.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] MAINTAINERS: add Niklas Haas for vf_libplacebo.c
2022-03-28 20:31 ` Lynne
@ 2022-03-28 20:44 ` Hendrik Leppkes
2022-03-28 21:13 ` Niklas Haas
1 sibling, 0 replies; 7+ messages in thread
From: Hendrik Leppkes @ 2022-03-28 20:44 UTC (permalink / raw)
To: FFmpeg development discussions and patches
On Mon, Mar 28, 2022 at 10:31 PM Lynne <dev@lynne.ee> wrote:
>
> 28 Mar 2022, 22:14 by jamrial@gmail.com:
>
> > On 3/28/2022 3:33 PM, Niklas Haas wrote:
> >
> >> From: Niklas Haas <git@haasn.dev>
> >>
> >> So I can merge my own changes to this filter after they pass peer
> >> review, as well as keeping it in sync with upstream API changes / new
> >> features.
> >>
> >> Signed-off-by: Niklas Haas <git@haasn.dev>
> >> ---
> >> MAINTAINERS | 2 ++
> >> 1 file changed, 2 insertions(+)
> >>
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index 931cf4bd2c..76e1332ad8 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -354,6 +354,7 @@ Filters:
> >> vf_il.c Paul B Mahol
> >> vf_(t)interlace Thomas Mundt (CC <thomas.mundt@hr.de>)
> >> vf_lenscorrection.c Daniel Oberhoff
> >> + vf_libplacebo.c Niklas Haas
> >> vf_mergeplanes.c Paul B Mahol
> >> vf_mestimate.c Davinder Singh
> >> vf_minterpolate.c Davinder Singh
> >> @@ -620,6 +621,7 @@ Loren Merritt ABD9 08F4 C920 3F65 D8BE 35D7 1540 DAA7 060F 56DE
> >> Lynne FE50 139C 6805 72CA FD52 1F8D A2FE A5F0 3F03 4464
> >> Michael Niedermayer 9FF2 128B 147E F673 0BAD F133 611E C787 040B 0FAB
> >> Nicolas George 24CE 01CE 9ACC 5CEB 74D8 8D9D B063 D997 36E5 4C93
> >> +Niklas Haas (haasn) 1DDB 8076 B14D 5B48 32FC 99D9 EB52 DA9C 02BA 6FB4
> >> Nikolay Aleksandrov 8978 1D8C FB71 588E 4B27 EAA8 C4F0 B5FC E011 13B1
> >> Panagiotis Issaris 6571 13A3 33D9 3726 F728 AA98 F643 B12E ECF3 E029
> >> Peter Ross A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B
> >>
> >
> > Applied.
> >
>
> I'm not okay with this. I don't mind anyone pushing approved patches,
> but I still want to review and have final say on all Vulkan code. Could you
> change that?
> I think haasn was more of a maintainer on the film grain stuff.
He wrote this filter and maintains it, as it also interacts with his
library, I don't see the problem here.
The interaction with the Vulkan code in FFmpeg seems minimal, mostly
just using it through the public hwcontext interface and avfilter
hwframe support to read vulkan frames, which is basically public API.
- Hendrik
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] MAINTAINERS: add Niklas Haas for vf_libplacebo.c
2022-03-28 20:31 ` Lynne
2022-03-28 20:44 ` Hendrik Leppkes
@ 2022-03-28 21:13 ` Niklas Haas
2022-03-28 23:12 ` Lynne
1 sibling, 1 reply; 7+ messages in thread
From: Niklas Haas @ 2022-03-28 21:13 UTC (permalink / raw)
To: FFmpeg development discussions and patches
On Mon, 28 Mar 2022 22:31:36 +0200 Lynne <dev@lynne.ee> wrote:
> I'm not okay with this. I don't mind anyone pushing approved patches,
> but I still want to review and have final say on all Vulkan code. Could you
> change that?
The contents of vf_libplacebo is not "Vulkan code". The only thing
Vulkan about it is the minimal plumbing between AVFrame and libplacebo,
most of which is actually already happening inside libplacebo's codebase
(<utils/libav.h`) where I already have complete control over it.
If anything, I still want to move this filter away from being
Vulkan-exclusive in the future, which would make it even more decoupled
from your concern.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] MAINTAINERS: add Niklas Haas for vf_libplacebo.c
2022-03-28 21:13 ` Niklas Haas
@ 2022-03-28 23:12 ` Lynne
2022-03-29 12:12 ` Niklas Haas
0 siblings, 1 reply; 7+ messages in thread
From: Lynne @ 2022-03-28 23:12 UTC (permalink / raw)
To: FFmpeg development discussions and patches
28 Mar 2022, 23:13 by ffmpeg@haasn.xyz:
> On Mon, 28 Mar 2022 22:31:36 +0200 Lynne <dev@lynne.ee> wrote:
>
>> I'm not okay with this. I don't mind anyone pushing approved patches,
>> but I still want to review and have final say on all Vulkan code. Could you
>> change that?
>>
>
> The contents of vf_libplacebo is not "Vulkan code". The only thing
> Vulkan about it is the minimal plumbing between AVFrame and libplacebo,
> most of which is actually already happening inside libplacebo's codebase
> (<utils/libav.h`) where I already have complete control over it.
>
Device creation's still done from the lavu-side of things.
> If anything, I still want to move this filter away from being
> Vulkan-exclusive in the future, which would make it even more decoupled
> from your concern.
>
I explicitly told you that's okay when the filter first got merged,
but you didn't change it back then, and I've been waiting for you
to enable it.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] MAINTAINERS: add Niklas Haas for vf_libplacebo.c
2022-03-28 23:12 ` Lynne
@ 2022-03-29 12:12 ` Niklas Haas
0 siblings, 0 replies; 7+ messages in thread
From: Niklas Haas @ 2022-03-29 12:12 UTC (permalink / raw)
To: FFmpeg development discussions and patches
On Tue, 29 Mar 2022 01:12:36 +0200 Lynne <dev@lynne.ee> wrote:
> 28 Mar 2022, 23:13 by ffmpeg@haasn.xyz:
>
> > On Mon, 28 Mar 2022 22:31:36 +0200 Lynne <dev@lynne.ee> wrote:
> >
> >> I'm not okay with this. I don't mind anyone pushing approved patches,
> >> but I still want to review and have final say on all Vulkan code. Could you
> >> change that?
> >>
> >
> > The contents of vf_libplacebo is not "Vulkan code". The only thing
> > Vulkan about it is the minimal plumbing between AVFrame and libplacebo,
> > most of which is actually already happening inside libplacebo's codebase
> > (<utils/libav.h`) where I already have complete control over it.
> >
>
> Device creation's still done from the lavu-side of things.
Right, but this is outside the scope of vf_libplacebo which is what I'm
being added as the maintainer for.
With that being said, is there still any objection to me being added as
a maintainer for **vf_libplacebo** specifically?
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-03-29 12:12 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-28 18:33 [FFmpeg-devel] [PATCH] MAINTAINERS: add Niklas Haas for vf_libplacebo.c Niklas Haas
2022-03-28 20:14 ` James Almer
2022-03-28 20:31 ` Lynne
2022-03-28 20:44 ` Hendrik Leppkes
2022-03-28 21:13 ` Niklas Haas
2022-03-28 23:12 ` Lynne
2022-03-29 12:12 ` Niklas Haas
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git