diff mbox series

[FFmpeg-devel,v2] HEVC: Export motion vectors to frame side data.

Message ID 20200115091908.81946-2-kaveasaf@gmail.com
State New
Headers show
Series [FFmpeg-devel,v2] HEVC: Export motion vectors to frame side data. | expand

Checks

Context Check Description
andriy/ffmpeg-patchwork success Make fate finished

Commit Message

Asaf Kave Jan. 15, 2020, 9:19 a.m. UTC
---
 libavcodec/hevc_refs.c |  15 ++++
 libavcodec/hevcdec.c   | 179 ++++++++++++++++++++++++++++++++++++++++-
 libavcodec/hevcdec.h   |  13 +++
 3 files changed, 206 insertions(+), 1 deletion(-)

Comments

Lynne Jan. 16, 2020, 1:10 p.m. UTC | #1
Jan 15, 2020, 09:19 by kaveasaf@gmail.com:

> ---
>  libavcodec/hevc_refs.c |  15 ++++
>  libavcodec/hevcdec.c   | 179 ++++++++++++++++++++++++++++++++++++++++-
>  libavcodec/hevcdec.h   |  13 +++
>  3 files changed, 206 insertions(+), 1 deletion(-)
>

Could you wait or instead work on the new block-based API instead?
This API will be removed soon and with it your code, unless it uses the new API.
Carl Eugen Hoyos Jan. 16, 2020, 2:19 p.m. UTC | #2
Am Do., 16. Jan. 2020 um 14:10 Uhr schrieb Lynne <dev@lynne.ee>:
>
> Jan 15, 2020, 09:19 by kaveasaf@gmail.com:
>
> > ---
> >  libavcodec/hevc_refs.c |  15 ++++
> >  libavcodec/hevcdec.c   | 179 ++++++++++++++++++++++++++++++++++++++++-
> >  libavcodec/hevcdec.h   |  13 +++
> >  3 files changed, 206 insertions(+), 1 deletion(-)
> >
>
> Could you wait or instead work on the new block-based API instead?

(That is not a useful suggestion imo)

> This API will be removed soon and with it your code, unless it uses the new API.

Isn't the patch using the "new" api?

Carl Eugen
Asaf Kave Jan. 16, 2020, 2:59 p.m. UTC | #3
On Thu, Jan 16, 2020 at 4:19 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:

> Am Do., 16. Jan. 2020 um 14:10 Uhr schrieb Lynne <dev@lynne.ee>:
> >
> > Jan 15, 2020, 09:19 by kaveasaf@gmail.com:
> >
> > > ---
> > >  libavcodec/hevc_refs.c |  15 ++++
> > >  libavcodec/hevcdec.c   | 179 ++++++++++++++++++++++++++++++++++++++++-
> > >  libavcodec/hevcdec.h   |  13 +++
> > >  3 files changed, 206 insertions(+), 1 deletion(-)
> > >
> >
> > Could you wait or instead work on the new block-based API instead?
>
> (That is not a useful suggestion imo)
>
> > This API will be removed soon and with it your code, unless it uses the
> new API.
>
> Isn't the patch using the "new" api?
>

Maybe i miss something, what is exact the "new" API ? can you elaborate


> Carl Eugen
> _______________________________________________
> 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".
Asaf Kave Jan. 20, 2020, 8:42 a.m. UTC | #4
Ping

On Thu, Jan 16, 2020 at 4:59 PM Asaf Kave <kaveasaf@gmail.com> wrote:

>
>
> On Thu, Jan 16, 2020 at 4:19 PM Carl Eugen Hoyos <ceffmpeg@gmail.com>
> wrote:
>
>> Am Do., 16. Jan. 2020 um 14:10 Uhr schrieb Lynne <dev@lynne.ee>:
>> >
>> > Jan 15, 2020, 09:19 by kaveasaf@gmail.com:
>> >
>> > > ---
>> > >  libavcodec/hevc_refs.c |  15 ++++
>> > >  libavcodec/hevcdec.c   | 179
>> ++++++++++++++++++++++++++++++++++++++++-
>> > >  libavcodec/hevcdec.h   |  13 +++
>> > >  3 files changed, 206 insertions(+), 1 deletion(-)
>> > >
>> >
>> > Could you wait or instead work on the new block-based API instead?
>>
>> (That is not a useful suggestion imo)
>>
>> > This API will be removed soon and with it your code, unless it uses the
>> new API.
>>
>> Isn't the patch using the "new" api?
>>
>
> Maybe i miss something, what is exact the "new" API ? can you elaborate
>
>
>> Carl Eugen
>> _______________________________________________
>> 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".
>
>
Lynne Jan. 21, 2020, 6:17 p.m. UTC | #5
Jan 20, 2020, 08:42 by kaveasaf@gmail.com:

> Ping
>

https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html


I've okayed it, its just not been pushed yet.
You'll need to add the vectors to the block struct. int16_t [2] will do.
Asaf Kave Jan. 22, 2020, 11:09 a.m. UTC | #6
On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:

> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
>
> > Ping
> >
>
> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
>
>
> I've okayed it, its just not been pushed yet.
> You'll need to add the vectors to the block struct. int16_t [2] will do.
>

Thanks Lynne,
I didn't find the 'libavutil: AVEncodeInfo data structures' commit\patch,
even i try searching the code on the latest 'master' branch.
Where can i get the new API ?


> _______________________________________________
> 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".
Asaf Kave Jan. 26, 2020, 10:31 a.m. UTC | #7
On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <kaveasaf@gmail.com> wrote:

>
>
> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:
>
>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
>>
>> > Ping
>> >
>>
>> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
>>
>>
>> I've okayed it, its just not been pushed yet.
>> You'll need to add the vectors to the block struct. int16_t [2] will do.
>>
>
> Thanks Lynne,
> I didn't find the 'libavutil: AVEncodeInfo data structures' commit\patch,
> even i try searching the code on the latest 'master' branch.
> Where can i get the new API ?
>

Hi,
Can someone accelerate this?
Thanks


>
>
>> _______________________________________________
>> 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".
>
>
Asaf Kave Jan. 28, 2020, 9:44 p.m. UTC | #8
On Sun, Jan 26, 2020, 12:31 Asaf Kave <kaveasaf@gmail.com> wrote:

>
>
> On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <kaveasaf@gmail.com> wrote:
>
>>
>>
>> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:
>>
>>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
>>>
>>> > Ping
>>> >
>>>
>>> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
>>>
>>>
>>> I've okayed it, its just not been pushed yet.
>>> You'll need to add the vectors to the block struct. int16_t [2] will do.
>>>
>>
>> Thanks Lynne,
>> I didn't find the 'libavutil: AVEncodeInfo data structures' commit\patch,
>> even i try searching the code on the latest 'master' branch.
>> Where can i get the new API ?
>>
>
> Hi,
> Can someone accelerate this?
> Thanks
>
>

Ping


>>
>>> _______________________________________________
>>> 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".
>>
>>
Asaf Kave Feb. 2, 2020, 9:15 a.m. UTC | #9
On Tue, Jan 28, 2020 at 11:44 PM Asaf Kave <kaveasaf@gmail.com> wrote:

>
>
> On Sun, Jan 26, 2020, 12:31 Asaf Kave <kaveasaf@gmail.com> wrote:
>
>>
>>
>> On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <kaveasaf@gmail.com> wrote:
>>
>>>
>>>
>>> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:
>>>
>>>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
>>>>
>>>> > Ping
>>>> >
>>>>
>>>> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
>>>>
>>>>
>>>> I've okayed it, its just not been pushed yet.
>>>> You'll need to add the vectors to the block struct. int16_t [2] will do.
>>>>
>>>
>>> Thanks Lynne,
>>> I didn't find the 'libavutil: AVEncodeInfo data structures'
>>> commit\patch, even i try searching the code on the latest 'master' branch.
>>> Where can i get the new API ?
>>>
>>
>> Hi,
>> Can someone accelerate this?
>> Thanks
>>
>>
>
> Ping
>

Pinging it again.


>
>
>>>
>>>> _______________________________________________
>>>> 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".
>>>
>>>
Asaf Kave Feb. 6, 2020, 11:17 a.m. UTC | #10
On Sun, Feb 2, 2020 at 11:15 AM Asaf Kave <kaveasaf@gmail.com> wrote:

>
>
> On Tue, Jan 28, 2020 at 11:44 PM Asaf Kave <kaveasaf@gmail.com> wrote:
>
>>
>>
>> On Sun, Jan 26, 2020, 12:31 Asaf Kave <kaveasaf@gmail.com> wrote:
>>
>>>
>>>
>>> On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <kaveasaf@gmail.com> wrote:
>>>
>>>>
>>>>
>>>> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:
>>>>
>>>>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
>>>>>
>>>>> > Ping
>>>>> >
>>>>>
>>>>> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
>>>>>
>>>>>
>>>>> I've okayed it, its just not been pushed yet.
>>>>> You'll need to add the vectors to the block struct. int16_t [2] will
>>>>> do.
>>>>>
>>>>
>>>> Thanks Lynne,
>>>> I didn't find the 'libavutil: AVEncodeInfo data structures'
>>>> commit\patch, even i try searching the code on the latest 'master' branch.
>>>> Where can i get the new API ?
>>>>
>>>
>>> Hi,
>>> Can someone accelerate this?
>>> Thanks
>>>
>>>
>>
>> Ping
>>
>
> Pinging it again.
>

Hello,
Can someone push this to master or if not than give me a feedback,
Thanks


>
>
>>
>>
>>>>
>>>>> _______________________________________________
>>>>> 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".
>>>>
>>>>
Lynne Feb. 6, 2020, 2:10 p.m. UTC | #11
Feb 6, 2020, 11:17 by kaveasaf@gmail.com:

> On Sun, Feb 2, 2020 at 11:15 AM Asaf Kave <kaveasaf@gmail.com> wrote:
>
>>
>>
>> On Tue, Jan 28, 2020 at 11:44 PM Asaf Kave <kaveasaf@gmail.com> wrote:
>>
>>>
>>>
>>> On Sun, Jan 26, 2020, 12:31 Asaf Kave <kaveasaf@gmail.com> wrote:
>>>
>>>>
>>>>
>>>> On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <kaveasaf@gmail.com> wrote:
>>>>
>>>>>
>>>>>
>>>>> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:
>>>>>
>>>>>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
>>>>>>
>>>>>> > Ping
>>>>>> >
>>>>>>
>>>>>> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
>>>>>>
>>>>>>
>>>>>> I've okayed it, its just not been pushed yet.
>>>>>> You'll need to add the vectors to the block struct. int16_t [2] will
>>>>>> do.
>>>>>>
>>>>>
>>>>> Thanks Lynne,
>>>>> I didn't find the 'libavutil: AVEncodeInfo data structures'
>>>>> commit\patch, even i try searching the code on the latest 'master' branch.
>>>>> Where can i get the new API ?
>>>>>
>>>>
>>>> Hi,
>>>> Can someone accelerate this?
>>>> Thanks
>>>>
>>>>
>>>
>>> Ping
>>>
>>
>> Pinging it again.
>>
>
> Hello,
> Can someone push this to master or if not than give me a feedback,
> Thanks
>

You need to download the patch and apply it yourself.
Asaf Kave Feb. 9, 2020, 8:59 a.m. UTC | #12
On Thu, Feb 6, 2020 at 4:10 PM Lynne <dev@lynne.ee> wrote:

> Feb 6, 2020, 11:17 by kaveasaf@gmail.com:
>
> > On Sun, Feb 2, 2020 at 11:15 AM Asaf Kave <kaveasaf@gmail.com> wrote:
> >
> >>
> >>
> >> On Tue, Jan 28, 2020 at 11:44 PM Asaf Kave <kaveasaf@gmail.com> wrote:
> >>
> >>>
> >>>
> >>> On Sun, Jan 26, 2020, 12:31 Asaf Kave <kaveasaf@gmail.com> wrote:
> >>>
> >>>>
> >>>>
> >>>> On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <kaveasaf@gmail.com> wrote:
> >>>>
> >>>>>
> >>>>>
> >>>>> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:
> >>>>>
> >>>>>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
> >>>>>>
> >>>>>> > Ping
> >>>>>> >
> >>>>>>
> >>>>>> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
> >>>>>>
> >>>>>>
> >>>>>> I've okayed it, its just not been pushed yet.
> >>>>>> You'll need to add the vectors to the block struct. int16_t [2] will
> >>>>>> do.
> >>>>>>
> >>>>>
> >>>>> Thanks Lynne,
> >>>>> I didn't find the 'libavutil: AVEncodeInfo data structures'
> >>>>> commit\patch, even i try searching the code on the latest 'master'
> branch.
> >>>>> Where can i get the new API ?
> >>>>>
> >>>>
> >>>> Hi,
> >>>> Can someone accelerate this?
> >>>> Thanks
> >>>>
> >>>>
> >>>
> >>> Ping
> >>>
> >>
> >> Pinging it again.
> >>
> >
> > Hello,
> > Can someone push this to master or if not than give me a feedback,
> > Thanks
> >
>
> You need to download the patch and apply it yourself.
>

Thanks, but i getting this error :
> git push
 fatal: unable to access 'https://git.ffmpeg.org/ffmpeg.git/': The
requested URL returned error: 403

I think i am not approval to push.


> _______________________________________________
> 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".
Asaf Kave Feb. 12, 2020, 7:32 a.m. UTC | #13
On Sun, Feb 9, 2020 at 10:59 AM Asaf Kave <kaveasaf@gmail.com> wrote:

>
>
> On Thu, Feb 6, 2020 at 4:10 PM Lynne <dev@lynne.ee> wrote:
>
>> Feb 6, 2020, 11:17 by kaveasaf@gmail.com:
>>
>> > On Sun, Feb 2, 2020 at 11:15 AM Asaf Kave <kaveasaf@gmail.com> wrote:
>> >
>> >>
>> >>
>> >> On Tue, Jan 28, 2020 at 11:44 PM Asaf Kave <kaveasaf@gmail.com> wrote:
>> >>
>> >>>
>> >>>
>> >>> On Sun, Jan 26, 2020, 12:31 Asaf Kave <kaveasaf@gmail.com> wrote:
>> >>>
>> >>>>
>> >>>>
>> >>>> On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <kaveasaf@gmail.com>
>> wrote:
>> >>>>
>> >>>>>
>> >>>>>
>> >>>>> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:
>> >>>>>
>> >>>>>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
>> >>>>>>
>> >>>>>> > Ping
>> >>>>>> >
>> >>>>>>
>> >>>>>> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
>> >>>>>>
>> >>>>>>
>> >>>>>> I've okayed it, its just not been pushed yet.
>> >>>>>> You'll need to add the vectors to the block struct. int16_t [2]
>> will
>> >>>>>> do.
>> >>>>>>
>> >>>>>
>> >>>>> Thanks Lynne,
>> >>>>> I didn't find the 'libavutil: AVEncodeInfo data structures'
>> >>>>> commit\patch, even i try searching the code on the latest 'master'
>> branch.
>> >>>>> Where can i get the new API ?
>> >>>>>
>> >>>>
>> >>>> Hi,
>> >>>> Can someone accelerate this?
>> >>>> Thanks
>> >>>>
>> >>>>
>> >>>
>> >>> Ping
>> >>>
>> >>
>> >> Pinging it again.
>> >>
>> >
>> > Hello,
>> > Can someone push this to master or if not than give me a feedback,
>> > Thanks
>> >
>>
>> You need to download the patch and apply it yourself.
>>
>
> Thanks, but i getting this error :
> > git push
>  fatal: unable to access 'https://git.ffmpeg.org/ffmpeg.git/': The
> requested URL returned error: 403
>
> I think i am not approval to push.
>
can someone push 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".
>
>
Paul B Mahol Feb. 12, 2020, 9:28 a.m. UTC | #14
On 2/12/20, Asaf Kave <kaveasaf@gmail.com> wrote:
> On Sun, Feb 9, 2020 at 10:59 AM Asaf Kave <kaveasaf@gmail.com> wrote:
>
>>
>>
>> On Thu, Feb 6, 2020 at 4:10 PM Lynne <dev@lynne.ee> wrote:
>>
>>> Feb 6, 2020, 11:17 by kaveasaf@gmail.com:
>>>
>>> > On Sun, Feb 2, 2020 at 11:15 AM Asaf Kave <kaveasaf@gmail.com> wrote:
>>> >
>>> >>
>>> >>
>>> >> On Tue, Jan 28, 2020 at 11:44 PM Asaf Kave <kaveasaf@gmail.com> wrote:
>>> >>
>>> >>>
>>> >>>
>>> >>> On Sun, Jan 26, 2020, 12:31 Asaf Kave <kaveasaf@gmail.com> wrote:
>>> >>>
>>> >>>>
>>> >>>>
>>> >>>> On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <kaveasaf@gmail.com>
>>> wrote:
>>> >>>>
>>> >>>>>
>>> >>>>>
>>> >>>>> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:
>>> >>>>>
>>> >>>>>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
>>> >>>>>>
>>> >>>>>> > Ping
>>> >>>>>> >
>>> >>>>>>
>>> >>>>>> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
>>> >>>>>>
>>> >>>>>>
>>> >>>>>> I've okayed it, its just not been pushed yet.
>>> >>>>>> You'll need to add the vectors to the block struct. int16_t [2]
>>> will
>>> >>>>>> do.
>>> >>>>>>
>>> >>>>>
>>> >>>>> Thanks Lynne,
>>> >>>>> I didn't find the 'libavutil: AVEncodeInfo data structures'
>>> >>>>> commit\patch, even i try searching the code on the latest 'master'
>>> branch.
>>> >>>>> Where can i get the new API ?
>>> >>>>>
>>> >>>>
>>> >>>> Hi,
>>> >>>> Can someone accelerate this?
>>> >>>> Thanks
>>> >>>>
>>> >>>>
>>> >>>
>>> >>> Ping
>>> >>>
>>> >>
>>> >> Pinging it again.
>>> >>
>>> >
>>> > Hello,
>>> > Can someone push this to master or if not than give me a feedback,
>>> > Thanks
>>> >
>>>
>>> You need to download the patch and apply it yourself.
>>>
>>
>> Thanks, but i getting this error :
>> > git push
>>  fatal: unable to access 'https://git.ffmpeg.org/ffmpeg.git/': The
>> requested URL returned error: 403
>>
>> I think i am not approval to push.
>>
> can someone push it?

Your patch can not be pushed as is.
Instead it needs to be combined with another patch.

>
>>
>>
>>> _______________________________________________
>>> 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".
>>
>>
> _______________________________________________
> 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".
Asaf Kave Feb. 12, 2020, 1:33 p.m. UTC | #15
On Wed, Feb 12, 2020 at 11:35 AM Paul B Mahol <onemda@gmail.com> wrote:

> On 2/12/20, Asaf Kave <kaveasaf@gmail.com> wrote:
> > On Sun, Feb 9, 2020 at 10:59 AM Asaf Kave <kaveasaf@gmail.com> wrote:
> >
> >>
> >>
> >> On Thu, Feb 6, 2020 at 4:10 PM Lynne <dev@lynne.ee> wrote:
> >>
> >>> Feb 6, 2020, 11:17 by kaveasaf@gmail.com:
> >>>
> >>> > On Sun, Feb 2, 2020 at 11:15 AM Asaf Kave <kaveasaf@gmail.com>
> wrote:
> >>> >
> >>> >>
> >>> >>
> >>> >> On Tue, Jan 28, 2020 at 11:44 PM Asaf Kave <kaveasaf@gmail.com>
> wrote:
> >>> >>
> >>> >>>
> >>> >>>
> >>> >>> On Sun, Jan 26, 2020, 12:31 Asaf Kave <kaveasaf@gmail.com> wrote:
> >>> >>>
> >>> >>>>
> >>> >>>>
> >>> >>>> On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <kaveasaf@gmail.com>
> >>> wrote:
> >>> >>>>
> >>> >>>>>
> >>> >>>>>
> >>> >>>>> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:
> >>> >>>>>
> >>> >>>>>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
> >>> >>>>>>
> >>> >>>>>> > Ping
> >>> >>>>>> >
> >>> >>>>>>
> >>> >>>>>>
> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
> >>> >>>>>>
> >>> >>>>>>
> >>> >>>>>> I've okayed it, its just not been pushed yet.
> >>> >>>>>> You'll need to add the vectors to the block struct. int16_t [2]
> >>> will
> >>> >>>>>> do.
> >>> >>>>>>
> >>> >>>>>
> >>> >>>>> Thanks Lynne,
> >>> >>>>> I didn't find the 'libavutil: AVEncodeInfo data structures'
> >>> >>>>> commit\patch, even i try searching the code on the latest
> 'master'
> >>> branch.
> >>> >>>>> Where can i get the new API ?
> >>> >>>>>
> >>> >>>>
> >>> >>>> Hi,
> >>> >>>> Can someone accelerate this?
> >>> >>>> Thanks
> >>> >>>>
> >>> >>>>
> >>> >>>
> >>> >>> Ping
> >>> >>>
> >>> >>
> >>> >> Pinging it again.
> >>> >>
> >>> >
> >>> > Hello,
> >>> > Can someone push this to master or if not than give me a feedback,
> >>> > Thanks
> >>> >
> >>>
> >>> You need to download the patch and apply it yourself.
> >>>
> >>
> >> Thanks, but i getting this error :
> >> > git push
> >>  fatal: unable to access 'https://git.ffmpeg.org/ffmpeg.git/': The
> >> requested URL returned error: 403
> >>
> >> I think i am not approval to push.
> >>
> > can someone push it?
>
> Your patch can not be pushed as is.
> Instead it needs to be combined with another patch.
>
OK thank you, do i need to do something or only wait that somebody will
combine 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".
> >>
> >>
> > _______________________________________________
> > 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".
> _______________________________________________
> 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".
Paul B Mahol Feb. 12, 2020, 2:01 p.m. UTC | #16
On 2/12/20, Asaf Kave <kaveasaf@gmail.com> wrote:
> On Wed, Feb 12, 2020 at 11:35 AM Paul B Mahol <onemda@gmail.com> wrote:
>
>> On 2/12/20, Asaf Kave <kaveasaf@gmail.com> wrote:
>> > On Sun, Feb 9, 2020 at 10:59 AM Asaf Kave <kaveasaf@gmail.com> wrote:
>> >
>> >>
>> >>
>> >> On Thu, Feb 6, 2020 at 4:10 PM Lynne <dev@lynne.ee> wrote:
>> >>
>> >>> Feb 6, 2020, 11:17 by kaveasaf@gmail.com:
>> >>>
>> >>> > On Sun, Feb 2, 2020 at 11:15 AM Asaf Kave <kaveasaf@gmail.com>
>> wrote:
>> >>> >
>> >>> >>
>> >>> >>
>> >>> >> On Tue, Jan 28, 2020 at 11:44 PM Asaf Kave <kaveasaf@gmail.com>
>> wrote:
>> >>> >>
>> >>> >>>
>> >>> >>>
>> >>> >>> On Sun, Jan 26, 2020, 12:31 Asaf Kave <kaveasaf@gmail.com> wrote:
>> >>> >>>
>> >>> >>>>
>> >>> >>>>
>> >>> >>>> On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <kaveasaf@gmail.com>
>> >>> wrote:
>> >>> >>>>
>> >>> >>>>>
>> >>> >>>>>
>> >>> >>>>> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:
>> >>> >>>>>
>> >>> >>>>>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
>> >>> >>>>>>
>> >>> >>>>>> > Ping
>> >>> >>>>>> >
>> >>> >>>>>>
>> >>> >>>>>>
>> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
>> >>> >>>>>>
>> >>> >>>>>>
>> >>> >>>>>> I've okayed it, its just not been pushed yet.
>> >>> >>>>>> You'll need to add the vectors to the block struct. int16_t [2]
>> >>> will
>> >>> >>>>>> do.
>> >>> >>>>>>
>> >>> >>>>>
>> >>> >>>>> Thanks Lynne,
>> >>> >>>>> I didn't find the 'libavutil: AVEncodeInfo data structures'
>> >>> >>>>> commit\patch, even i try searching the code on the latest
>> 'master'
>> >>> branch.
>> >>> >>>>> Where can i get the new API ?
>> >>> >>>>>
>> >>> >>>>
>> >>> >>>> Hi,
>> >>> >>>> Can someone accelerate this?
>> >>> >>>> Thanks
>> >>> >>>>
>> >>> >>>>
>> >>> >>>
>> >>> >>> Ping
>> >>> >>>
>> >>> >>
>> >>> >> Pinging it again.
>> >>> >>
>> >>> >
>> >>> > Hello,
>> >>> > Can someone push this to master or if not than give me a feedback,
>> >>> > Thanks
>> >>> >
>> >>>
>> >>> You need to download the patch and apply it yourself.
>> >>>
>> >>
>> >> Thanks, but i getting this error :
>> >> > git push
>> >>  fatal: unable to access 'https://git.ffmpeg.org/ffmpeg.git/': The
>> >> requested URL returned error: 403
>> >>
>> >> I think i am not approval to push.
>> >>
>> > can someone push it?
>>
>> Your patch can not be pushed as is.
>> Instead it needs to be combined with another patch.
>>
> OK thank you, do i need to do something or only wait that somebody will
> combine it?

Very unlikely somebody will combine your patch with another for free.
So it all is up to you.

>
>>
>> >
>> >>
>> >>
>> >>> _______________________________________________
>> >>> 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".
>> >>
>> >>
>> > _______________________________________________
>> > 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".
>> _______________________________________________
>> 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".
> _______________________________________________
> 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".
Asaf Kave Feb. 13, 2020, 1:44 p.m. UTC | #17
On Wed, Feb 12, 2020 at 4:02 PM Paul B Mahol <onemda@gmail.com> wrote:

> On 2/12/20, Asaf Kave <kaveasaf@gmail.com> wrote:
> > On Wed, Feb 12, 2020 at 11:35 AM Paul B Mahol <onemda@gmail.com> wrote:
> >
> >> On 2/12/20, Asaf Kave <kaveasaf@gmail.com> wrote:
> >> > On Sun, Feb 9, 2020 at 10:59 AM Asaf Kave <kaveasaf@gmail.com> wrote:
> >> >
> >> >>
> >> >>
> >> >> On Thu, Feb 6, 2020 at 4:10 PM Lynne <dev@lynne.ee> wrote:
> >> >>
> >> >>> Feb 6, 2020, 11:17 by kaveasaf@gmail.com:
> >> >>>
> >> >>> > On Sun, Feb 2, 2020 at 11:15 AM Asaf Kave <kaveasaf@gmail.com>
> >> wrote:
> >> >>> >
> >> >>> >>
> >> >>> >>
> >> >>> >> On Tue, Jan 28, 2020 at 11:44 PM Asaf Kave <kaveasaf@gmail.com>
> >> wrote:
> >> >>> >>
> >> >>> >>>
> >> >>> >>>
> >> >>> >>> On Sun, Jan 26, 2020, 12:31 Asaf Kave <kaveasaf@gmail.com>
> wrote:
> >> >>> >>>
> >> >>> >>>>
> >> >>> >>>>
> >> >>> >>>> On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <kaveasaf@gmail.com>
> >> >>> wrote:
> >> >>> >>>>
> >> >>> >>>>>
> >> >>> >>>>>
> >> >>> >>>>> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:
> >> >>> >>>>>
> >> >>> >>>>>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
> >> >>> >>>>>>
> >> >>> >>>>>> > Ping
> >> >>> >>>>>> >
> >> >>> >>>>>>
> >> >>> >>>>>>
> >> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
> >> >>> >>>>>>
> >> >>> >>>>>>
> >> >>> >>>>>> I've okayed it, its just not been pushed yet.
> >> >>> >>>>>> You'll need to add the vectors to the block struct. int16_t
> [2]
> >> >>> will
> >> >>> >>>>>> do.
> >> >>> >>>>>>
> >> >>> >>>>>
> >> >>> >>>>> Thanks Lynne,
> >> >>> >>>>> I didn't find the 'libavutil: AVEncodeInfo data structures'
> >> >>> >>>>> commit\patch, even i try searching the code on the latest
> >> 'master'
> >> >>> branch.
> >> >>> >>>>> Where can i get the new API ?
> >> >>> >>>>>
> >> >>> >>>>
> >> >>> >>>> Hi,
> >> >>> >>>> Can someone accelerate this?
> >> >>> >>>> Thanks
> >> >>> >>>>
> >> >>> >>>>
> >> >>> >>>
> >> >>> >>> Ping
> >> >>> >>>
> >> >>> >>
> >> >>> >> Pinging it again.
> >> >>> >>
> >> >>> >
> >> >>> > Hello,
> >> >>> > Can someone push this to master or if not than give me a feedback,
> >> >>> > Thanks
> >> >>> >
> >> >>>
> >> >>> You need to download the patch and apply it yourself.
> >> >>>
> >> >>
> >> >> Thanks, but i getting this error :
> >> >> > git push
> >> >>  fatal: unable to access 'https://git.ffmpeg.org/ffmpeg.git/': The
> >> >> requested URL returned error: 403
> >> >>
> >> >> I think i am not approval to push.
> >> >>
> >> > can someone push it?
> >>
> >> Your patch can not be pushed as is.
> >> Instead it needs to be combined with another patch.
> >>
> > OK thank you, do i need to do something or only wait that somebody will
> > combine it?
>
> Very unlikely somebody will combine your patch with another for free.
> So it all is up to you.
>

Thanks,
So just to to be sure on what i need to do:
1. Download this patch '[FFmpeg-devel,1/3] libavutil: AVEncodeInfo data
structures' (
https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html), and
apply it locally.
2. Then i need to apply again my patch locally, and

> *add the vectors to the block struct. int16_t [2] will do*

 as Lynne mention before.

Now how i can push my patch?(i am not have the right permission) do i need
to push both patch's? or to combine them to one commit patch?

I looked at patch ' libavutil: AVEncodeInfo data structures', and i wasn't
convinced that both patch are so dependent, can you motivate me or direct
me to info why i need to add vectors at the 'AVEncodeInfoBlock' structure?
Thanks for your time.


> >
> >>
> >> >
> >> >>
> >> >>
> >> >>> _______________________________________________
> >> >>> 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".
> >> >>
> >> >>
> >> > _______________________________________________
> >> > 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".
> >> _______________________________________________
> >> 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".
> > _______________________________________________
> > 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".
> _______________________________________________
> 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".
Paul B Mahol Feb. 13, 2020, 1:53 p.m. UTC | #18
On 2/13/20, Asaf Kave <kaveasaf@gmail.com> wrote:
> On Wed, Feb 12, 2020 at 4:02 PM Paul B Mahol <onemda@gmail.com> wrote:
>
>> On 2/12/20, Asaf Kave <kaveasaf@gmail.com> wrote:
>> > On Wed, Feb 12, 2020 at 11:35 AM Paul B Mahol <onemda@gmail.com> wrote:
>> >
>> >> On 2/12/20, Asaf Kave <kaveasaf@gmail.com> wrote:
>> >> > On Sun, Feb 9, 2020 at 10:59 AM Asaf Kave <kaveasaf@gmail.com> wrote:
>> >> >
>> >> >>
>> >> >>
>> >> >> On Thu, Feb 6, 2020 at 4:10 PM Lynne <dev@lynne.ee> wrote:
>> >> >>
>> >> >>> Feb 6, 2020, 11:17 by kaveasaf@gmail.com:
>> >> >>>
>> >> >>> > On Sun, Feb 2, 2020 at 11:15 AM Asaf Kave <kaveasaf@gmail.com>
>> >> wrote:
>> >> >>> >
>> >> >>> >>
>> >> >>> >>
>> >> >>> >> On Tue, Jan 28, 2020 at 11:44 PM Asaf Kave <kaveasaf@gmail.com>
>> >> wrote:
>> >> >>> >>
>> >> >>> >>>
>> >> >>> >>>
>> >> >>> >>> On Sun, Jan 26, 2020, 12:31 Asaf Kave <kaveasaf@gmail.com>
>> wrote:
>> >> >>> >>>
>> >> >>> >>>>
>> >> >>> >>>>
>> >> >>> >>>> On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <kaveasaf@gmail.com>
>> >> >>> wrote:
>> >> >>> >>>>
>> >> >>> >>>>>
>> >> >>> >>>>>
>> >> >>> >>>>> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee> wrote:
>> >> >>> >>>>>
>> >> >>> >>>>>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
>> >> >>> >>>>>>
>> >> >>> >>>>>> > Ping
>> >> >>> >>>>>> >
>> >> >>> >>>>>>
>> >> >>> >>>>>>
>> >> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
>> >> >>> >>>>>>
>> >> >>> >>>>>>
>> >> >>> >>>>>> I've okayed it, its just not been pushed yet.
>> >> >>> >>>>>> You'll need to add the vectors to the block struct. int16_t
>> [2]
>> >> >>> will
>> >> >>> >>>>>> do.
>> >> >>> >>>>>>
>> >> >>> >>>>>
>> >> >>> >>>>> Thanks Lynne,
>> >> >>> >>>>> I didn't find the 'libavutil: AVEncodeInfo data structures'
>> >> >>> >>>>> commit\patch, even i try searching the code on the latest
>> >> 'master'
>> >> >>> branch.
>> >> >>> >>>>> Where can i get the new API ?
>> >> >>> >>>>>
>> >> >>> >>>>
>> >> >>> >>>> Hi,
>> >> >>> >>>> Can someone accelerate this?
>> >> >>> >>>> Thanks
>> >> >>> >>>>
>> >> >>> >>>>
>> >> >>> >>>
>> >> >>> >>> Ping
>> >> >>> >>>
>> >> >>> >>
>> >> >>> >> Pinging it again.
>> >> >>> >>
>> >> >>> >
>> >> >>> > Hello,
>> >> >>> > Can someone push this to master or if not than give me a
>> >> >>> > feedback,
>> >> >>> > Thanks
>> >> >>> >
>> >> >>>
>> >> >>> You need to download the patch and apply it yourself.
>> >> >>>
>> >> >>
>> >> >> Thanks, but i getting this error :
>> >> >> > git push
>> >> >>  fatal: unable to access 'https://git.ffmpeg.org/ffmpeg.git/': The
>> >> >> requested URL returned error: 403
>> >> >>
>> >> >> I think i am not approval to push.
>> >> >>
>> >> > can someone push it?
>> >>
>> >> Your patch can not be pushed as is.
>> >> Instead it needs to be combined with another patch.
>> >>
>> > OK thank you, do i need to do something or only wait that somebody will
>> > combine it?
>>
>> Very unlikely somebody will combine your patch with another for free.
>> So it all is up to you.
>>
>
> Thanks,
> So just to to be sure on what i need to do:
> 1. Download this patch '[FFmpeg-devel,1/3] libavutil: AVEncodeInfo data
> structures' (
> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html), and
> apply it locally.
> 2. Then i need to apply again my patch locally, and
>
>> *add the vectors to the block struct. int16_t [2] will do*
>
>  as Lynne mention before.
>
> Now how i can push my patch?(i am not have the right permission) do i need
> to push both patch's? or to combine them to one commit patch?
>
> I looked at patch ' libavutil: AVEncodeInfo data structures', and i wasn't
> convinced that both patch are so dependent, can you motivate me or direct
> me to info why i need to add vectors at the 'AVEncodeInfoBlock' structure?

Mostly because other developers want to use that new API instead of
current old one to export motion vectors.

> Thanks for your time.
>
>
>> >
>> >>
>> >> >
>> >> >>
>> >> >>
>> >> >>> _______________________________________________
>> >> >>> 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".
>> >> >>
>> >> >>
>> >> > _______________________________________________
>> >> > 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".
>> >> _______________________________________________
>> >> 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".
>> > _______________________________________________
>> > 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".
>> _______________________________________________
>> 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".
> _______________________________________________
> 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".
Asaf Kave Feb. 19, 2020, 9:11 a.m. UTC | #19
On Thu, Feb 13, 2020 at 3:54 PM Paul B Mahol <onemda@gmail.com> wrote:

> On 2/13/20, Asaf Kave <kaveasaf@gmail.com> wrote:
> > On Wed, Feb 12, 2020 at 4:02 PM Paul B Mahol <onemda@gmail.com> wrote:
> >
> >> On 2/12/20, Asaf Kave <kaveasaf@gmail.com> wrote:
> >> > On Wed, Feb 12, 2020 at 11:35 AM Paul B Mahol <onemda@gmail.com>
> wrote:
> >> >
> >> >> On 2/12/20, Asaf Kave <kaveasaf@gmail.com> wrote:
> >> >> > On Sun, Feb 9, 2020 at 10:59 AM Asaf Kave <kaveasaf@gmail.com>
> wrote:
> >> >> >
> >> >> >>
> >> >> >>
> >> >> >> On Thu, Feb 6, 2020 at 4:10 PM Lynne <dev@lynne.ee> wrote:
> >> >> >>
> >> >> >>> Feb 6, 2020, 11:17 by kaveasaf@gmail.com:
> >> >> >>>
> >> >> >>> > On Sun, Feb 2, 2020 at 11:15 AM Asaf Kave <kaveasaf@gmail.com>
> >> >> wrote:
> >> >> >>> >
> >> >> >>> >>
> >> >> >>> >>
> >> >> >>> >> On Tue, Jan 28, 2020 at 11:44 PM Asaf Kave <
> kaveasaf@gmail.com>
> >> >> wrote:
> >> >> >>> >>
> >> >> >>> >>>
> >> >> >>> >>>
> >> >> >>> >>> On Sun, Jan 26, 2020, 12:31 Asaf Kave <kaveasaf@gmail.com>
> >> wrote:
> >> >> >>> >>>
> >> >> >>> >>>>
> >> >> >>> >>>>
> >> >> >>> >>>> On Wed, Jan 22, 2020 at 1:09 PM Asaf Kave <
> kaveasaf@gmail.com>
> >> >> >>> wrote:
> >> >> >>> >>>>
> >> >> >>> >>>>>
> >> >> >>> >>>>>
> >> >> >>> >>>>> On Tue, Jan 21, 2020 at 8:17 PM Lynne <dev@lynne.ee>
> wrote:
> >> >> >>> >>>>>
> >> >> >>> >>>>>> Jan 20, 2020, 08:42 by kaveasaf@gmail.com:
> >> >> >>> >>>>>>
> >> >> >>> >>>>>> > Ping
> >> >> >>> >>>>>> >
> >> >> >>> >>>>>>
> >> >> >>> >>>>>>
> >> >> https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html
> >> >> >>> >>>>>>
> >> >> >>> >>>>>>
> >> >> >>> >>>>>> I've okayed it, its just not been pushed yet.
> >> >> >>> >>>>>> You'll need to add the vectors to the block struct.
> int16_t
> >> [2]
> >> >> >>> will
> >> >> >>> >>>>>> do.
> >> >> >>> >>>>>>
> >> >> >>> >>>>>
> >> >> >>> >>>>> Thanks Lynne,
> >> >> >>> >>>>> I didn't find the 'libavutil: AVEncodeInfo data structures'
> >> >> >>> >>>>> commit\patch, even i try searching the code on the latest
> >> >> 'master'
> >> >> >>> branch.
> >> >> >>> >>>>> Where can i get the new API ?
> >> >> >>> >>>>>
> >> >> >>> >>>>
> >> >> >>> >>>> Hi,
> >> >> >>> >>>> Can someone accelerate this?
> >> >> >>> >>>> Thanks
> >> >> >>> >>>>
> >> >> >>> >>>>
> >> >> >>> >>>
> >> >> >>> >>> Ping
> >> >> >>> >>>
> >> >> >>> >>
> >> >> >>> >> Pinging it again.
> >> >> >>> >>
> >> >> >>> >
> >> >> >>> > Hello,
> >> >> >>> > Can someone push this to master or if not than give me a
> >> >> >>> > feedback,
> >> >> >>> > Thanks
> >> >> >>> >
> >> >> >>>
> >> >> >>> You need to download the patch and apply it yourself.
> >> >> >>>
> >> >> >>
> >> >> >> Thanks, but i getting this error :
> >> >> >> > git push
> >> >> >>  fatal: unable to access 'https://git.ffmpeg.org/ffmpeg.git/':
> The
> >> >> >> requested URL returned error: 403
> >> >> >>
> >> >> >> I think i am not approval to push.
> >> >> >>
> >> >> > can someone push it?
> >> >>
> >> >> Your patch can not be pushed as is.
> >> >> Instead it needs to be combined with another patch.
> >> >>
> >> > OK thank you, do i need to do something or only wait that somebody
> will
> >> > combine it?
> >>
> >> Very unlikely somebody will combine your patch with another for free.
> >> So it all is up to you.
> >>
> >
> > Thanks,
> > So just to to be sure on what i need to do:
> > 1. Download this patch '[FFmpeg-devel,1/3] libavutil: AVEncodeInfo data
> > structures' (
> > https://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/248836.html), and
> > apply it locally.
> > 2. Then i need to apply again my patch locally, and
> >
> >> *add the vectors to the block struct. int16_t [2] will do*
> >
> >  as Lynne mention before.
> >
> > Now how i can push my patch?(i am not have the right permission) do i
> need
> > to push both patch's? or to combine them to one commit patch?
>
Can you give me your thoughts for those questions, and how i can get
progress.

> Now how i can push my patch?(i am not have the right permission, i tried
> it without success) do i need to push both patch's? or to combine them to
> one commit patch?

>
> > I looked at patch ' libavutil: AVEncodeInfo data structures', and i
> wasn't
> > convinced that both patch are so dependent, can you motivate me or direct
> > me to info why i need to add vectors at the 'AVEncodeInfoBlock'
> structure?
>
> Mostly because other developers want to use that new API instead of
> current old one to export motion vectors.
>
> > Thanks for your time.
> >
> >
> >> >
> >> >>
> >> >> >
> >> >> >>
> >> >> >>
> >> >> >>> _______________________________________________
> >> >> >>> 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".
> >> >> >>
> >> >> >>
> >> >> > _______________________________________________
> >> >> > 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".
> >> >> _______________________________________________
> >> >> 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".
> >> > _______________________________________________
> >> > 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".
> >> _______________________________________________
> >> 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".
> > _______________________________________________
> > 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".
> _______________________________________________
> 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".
diff mbox series

Patch

diff --git a/libavcodec/hevc_refs.c b/libavcodec/hevc_refs.c
index 7870a72fd6..20f028fa73 100644
--- a/libavcodec/hevc_refs.c
+++ b/libavcodec/hevc_refs.c
@@ -42,6 +42,9 @@  void ff_hevc_unref_frame(HEVCContext *s, HEVCFrame *frame, int flags)
         av_buffer_unref(&frame->tab_mvf_buf);
         frame->tab_mvf = NULL;
 
+        av_buffer_unref(&frame->cuh_buf);
+        frame->cuh = NULL;
+
         av_buffer_unref(&frame->rpl_buf);
         av_buffer_unref(&frame->rpl_tab_buf);
         frame->rpl_tab    = NULL;
@@ -101,11 +104,17 @@  static HEVCFrame *alloc_frame(HEVCContext *s)
             goto fail;
         frame->tab_mvf = (MvField *)frame->tab_mvf_buf->data;
 
+        frame->cuh_buf = av_buffer_pool_get(s->cuh_pool);
+        if (!frame->cuh_buf)
+            goto fail;
+        frame->cuh = (CodingUnitHelper *)frame->cuh_buf->data;
+
         frame->rpl_tab_buf = av_buffer_pool_get(s->rpl_tab_pool);
         if (!frame->rpl_tab_buf)
             goto fail;
         frame->rpl_tab   = (RefPicListTab **)frame->rpl_tab_buf->data;
         frame->ctb_count = s->ps.sps->ctb_width * s->ps.sps->ctb_height;
+        frame->cu_count = 0;
         for (j = 0; j < frame->ctb_count; j++)
             frame->rpl_tab[j] = (RefPicListTab *)frame->rpl_buf->data;
 
@@ -161,6 +170,10 @@  int ff_hevc_set_new_ref(HEVCContext *s, AVFrame **frame, int poc)
     else
         ref->flags = HEVC_FRAME_FLAG_SHORT_REF;
 
+    if (s->avctx->flags2 & AV_CODEC_FLAG2_EXPORT_MVS) {
+        ref->flags |= HEVC_FRAME_FLAG_MV;
+    }
+
     ref->poc      = poc;
     ref->sequence = s->seq_decode;
     ref->frame->crop_left   = s->ps.sps->output_window.left_offset;
@@ -216,6 +229,8 @@  int ff_hevc_output_frame(HEVCContext *s, AVFrame *out, int flush)
             if (ret < 0)
                 return ret;
 
+            s->output_frame_poc = frame->poc;
+
             av_log(s->avctx, AV_LOG_DEBUG,
                    "Output frame with POC %d.\n", frame->poc);
             return 1;
diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c
index 19b0cd815d..367745c6ee 100644
--- a/libavcodec/hevcdec.c
+++ b/libavcodec/hevcdec.c
@@ -32,6 +32,7 @@ 
 #include "libavutil/opt.h"
 #include "libavutil/pixdesc.h"
 #include "libavutil/stereo3d.h"
+#include "libavutil/motion_vector.h"
 
 #include "bswapdsp.h"
 #include "bytestream.h"
@@ -80,6 +81,7 @@  static void pic_arrays_free(HEVCContext *s)
     av_freep(&s->sh.offset);
 
     av_buffer_pool_uninit(&s->tab_mvf_pool);
+    av_buffer_pool_uninit(&s->cuh_pool);
     av_buffer_pool_uninit(&s->rpl_tab_pool);
 }
 
@@ -128,9 +130,11 @@  static int pic_arrays_init(HEVCContext *s, const HEVCSPS *sps)
 
     s->tab_mvf_pool = av_buffer_pool_init(min_pu_size * sizeof(MvField),
                                           av_buffer_allocz);
+    s->cuh_pool = av_buffer_pool_init(min_pu_size * sizeof(CodingUnitHelper),
+                                          av_buffer_allocz);
     s->rpl_tab_pool = av_buffer_pool_init(ctb_count * sizeof(RefPicListTab),
                                           av_buffer_allocz);
-    if (!s->tab_mvf_pool || !s->rpl_tab_pool)
+    if (!s->tab_mvf_pool || !s->rpl_tab_pool || !s->cuh_pool)
         goto fail;
 
     return 0;
@@ -1806,6 +1810,7 @@  static void hls_prediction_unit(HEVCContext *s, int x0, int y0,
     int min_pu_width = s->ps.sps->min_pu_width;
 
     MvField *tab_mvf = s->ref->tab_mvf;
+    CodingUnitHelper *cuh = s->ref->cuh;
     RefPicList  *refPicList = s->ref->refPicList;
     HEVCFrame *ref0 = NULL, *ref1 = NULL;
     uint8_t *dst0 = POS(0, x0, y0);
@@ -1843,6 +1848,10 @@  static void hls_prediction_unit(HEVCContext *s, int x0, int y0,
         for (i = 0; i < nPbW >> s->ps.sps->log2_min_pu_size; i++)
             tab_mvf[(y_pu + j) * min_pu_width + x_pu + i] = current_mv;
 
+    cuh[s->ref->cu_count].cu = lc->cu;
+    cuh[s->ref->cu_count].log2_cu_size = log2_cb_size;
+    s->ref->cu_count++;
+
     if (current_mv.pred_flag & PF_L0) {
         ref0 = refPicList[0].ref[current_mv.ref_idx[0]];
         if (!ref0)
@@ -3192,6 +3201,163 @@  static int hevc_decode_extradata(HEVCContext *s, uint8_t *buf, int length, int f
     return 0;
 }
 
+static int set_mv(AVMotionVector *mv, int puW, int puH,
+                  int dst_x, int dst_y,
+                  int motion_x, int motion_y, int motion_scale,
+                  int direction)
+{
+    mv->w = puW;
+    mv->h = puH;
+    mv->motion_x = motion_x;
+    mv->motion_y = motion_y;
+    mv->motion_scale = motion_scale;
+    mv->dst_x = dst_x;
+    mv->dst_y = dst_y;
+    mv->src_x = dst_x + motion_x / motion_scale;
+    mv->src_y = dst_y + motion_y / motion_scale;
+    mv->source = direction ? 1 : -1;
+    mv->flags = 0;
+
+    return 1;
+}
+
+static int add_mv(HEVCContext *s, AVMotionVector *mvs, MvField current_mv, int x0, int y0, int width, int height)
+{
+    int sx, sy, count = 0;
+    const int scale = 4;
+
+    sx = x0 + (width / 2);
+    sy = y0 + (height / 2);
+
+    if (current_mv.pred_flag & PF_L0) {
+      count += set_mv(mvs + count, width, height , sx, sy, current_mv.mv[0].x, current_mv.mv[0].y, scale, 0);
+    }
+
+    if (current_mv.pred_flag & PF_L1) {
+      count += set_mv(mvs + count, width, height , sx, sy, current_mv.mv[1].x, current_mv.mv[1].y, scale, 1);
+    }
+
+    return count;
+}
+
+static int export_mvs(HEVCContext *s, AVFrame *out)
+{
+    int x0, y0, i, log2_cb_size;
+    int mv_count = 0;
+    HEVCFrame* pframe = NULL;
+    MvField *tab_mvf = NULL;
+    struct MvField current_mv = {{{ 0 }}};
+    enum PartMode part_mode;
+    int cb_size;
+    int half_cb_size;
+    AVMotionVector *mvs = NULL;
+
+    const int min_pu_width = s->ps.sps->min_pu_width;
+    const unsigned log2_min_pu_size = s->ps.sps->log2_min_pu_size;
+
+    /* Find the next output picture\frame tp export it VMs */
+    for (i = 0; i < FF_ARRAY_ELEMS(s->DPB); i++) {
+        HEVCFrame *frame = &s->DPB[i];
+        if (frame->flags & (HEVC_FRAME_FLAG_OUTPUT | HEVC_FRAME_FLAG_MV) &&
+            frame->poc == s->output_frame_poc) {
+            pframe = frame;
+            break;
+        }
+    }
+
+    if(pframe == NULL) {
+        av_log(s->avctx, AV_LOG_WARNING, "Not exporting MVs for frame POC %d", s->poc);
+        return AVERROR_INVALIDDATA;
+    }
+
+    tab_mvf = pframe->tab_mvf;
+
+    /* size is number of [coding units * 2 * 4] where 2 is for directions and 4 is
+     * for the maximum number of part mode */
+    mvs = av_malloc_array(pframe->cu_count, 2 * 4 * sizeof(AVMotionVector));
+    if (!mvs) {
+        av_log(s->avctx, AV_LOG_WARNING, "Failed to allocate Motion Vector array for frame POC %d", s->poc);
+        ff_hevc_unref_frame(s, pframe, HEVC_FRAME_FLAG_MV);
+        return AVERROR(ENOMEM);
+    }
+
+    for (i = 0; i < pframe->cu_count; ++i) {
+
+        const CodingUnitHelper current_cu = pframe->cuh[i];
+        /* Export only INTER prediction coding units */
+        if(current_cu.cu.pred_mode != MODE_INTER)
+            continue;
+
+        y0 = current_cu.cu.y;
+        x0 = current_cu.cu.x;
+        log2_cb_size = current_cu.log2_cu_size;
+        part_mode = current_cu.cu.part_mode;
+
+        cb_size = 1 << log2_cb_size;
+
+        current_mv = tab_mvf[(y0 >> log2_min_pu_size) * min_pu_width + (x0 >> log2_min_pu_size)];
+
+        half_cb_size = 1 << (log2_cb_size - 1);
+        switch (part_mode) {
+            case PART_2Nx2N:
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0, y0, cb_size, cb_size);
+            break;
+            case PART_NxN:
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0, y0, cb_size / 2, cb_size / 2);
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0 + half_cb_size, y0, cb_size / 2, cb_size / 2);
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0, y0 + half_cb_size, cb_size / 2, cb_size / 2);
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0 + half_cb_size, y0 + half_cb_size, cb_size / 2, cb_size / 2);
+            break;
+            case PART_2NxN:
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0, y0, cb_size, cb_size / 2);
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0, y0 + half_cb_size, cb_size, cb_size / 2);
+            break;
+            case PART_Nx2N:
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0, y0, cb_size / 2, cb_size);
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0 + half_cb_size, y0, cb_size / 2, cb_size);
+            break;
+            case PART_2NxnU:
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0, y0, cb_size, cb_size / 4);
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0, y0 + cb_size / 4, cb_size, cb_size * 3 / 4);
+            break;
+            case PART_2NxnD:
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0, y0, cb_size, cb_size * 3 / 4);
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0, y0 + cb_size * 3 / 4, cb_size, cb_size / 4);
+            break;
+            case PART_nLx2N:
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0, y0, cb_size / 4, cb_size);
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0 + cb_size / 4, y0, cb_size * 3 / 4, cb_size);
+            break;
+            case PART_nRx2N:
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0, y0, cb_size * 3 / 4, cb_size);
+            mv_count += add_mv(s, mvs + mv_count, current_mv, x0 * 3 / 4, y0, cb_size / 4, cb_size);
+            break;
+            default:
+            break;
+        }
+    }
+
+    if (mv_count) {
+        AVFrameSideData *sd;
+
+        av_log(s->avctx, AV_LOG_DEBUG, "Adding %d MVs info to frame with POC %d", mv_count, s->poc);
+        sd = av_frame_new_side_data(out, AV_FRAME_DATA_MOTION_VECTORS, mv_count * sizeof(AVMotionVector));
+        if (!sd) {
+            av_log(s->avctx, AV_LOG_WARNING, "Failed to create side data MVs info to frame POC %d", s->poc);
+            av_freep(&mvs);
+            ff_hevc_unref_frame(s, pframe, HEVC_FRAME_FLAG_MV);
+            return AVERROR(ENOMEM);
+        }
+        memcpy(sd->data, mvs, mv_count * sizeof(AVMotionVector));
+    }
+
+    /* Cleanup and release */
+    av_freep(&mvs);
+    ff_hevc_unref_frame(s, pframe, HEVC_FRAME_FLAG_MV);
+
+    return 0;
+}
+
 static int hevc_decode_frame(AVCodecContext *avctx, void *data, int *got_output,
                              AVPacket *avpkt)
 {
@@ -3249,6 +3415,11 @@  static int hevc_decode_frame(AVCodecContext *avctx, void *data, int *got_output,
 
     if (s->output_frame->buf[0]) {
         av_frame_move_ref(data, s->output_frame);
+        if (s->avctx->flags2 & AV_CODEC_FLAG2_EXPORT_MVS) {
+            if(export_mvs(s, data) < 0 ) {
+                av_log(avctx, AV_LOG_WARNING, "Failed to export Motion Vectors for frame with POC %d", s->poc);
+            }
+        }
         *got_output = 1;
     }
 
@@ -3277,8 +3448,14 @@  static int hevc_ref_frame(HEVCContext *s, HEVCFrame *dst, HEVCFrame *src)
     if (!dst->rpl_buf)
         goto fail;
 
+    dst->cuh_buf = av_buffer_ref(src->cuh_buf);
+    if (!dst->cuh_buf)
+        goto fail;
+    dst->cuh = src->cuh;
+
     dst->poc        = src->poc;
     dst->ctb_count  = src->ctb_count;
+    dst->cu_count   = src->cu_count;
     dst->flags      = src->flags;
     dst->sequence   = src->sequence;
 
diff --git a/libavcodec/hevcdec.h b/libavcodec/hevcdec.h
index 89e0809850..187c6ec7a2 100644
--- a/libavcodec/hevcdec.h
+++ b/libavcodec/hevcdec.h
@@ -252,6 +252,12 @@  typedef struct CodingUnit {
     uint8_t cu_transquant_bypass_flag;
 } CodingUnit;
 
+typedef struct CodingUnitHelper {
+    CodingUnit cu;
+
+    uint8_t log2_cu_size;
+} CodingUnitHelper;
+
 typedef struct Mv {
     int16_t x;  ///< horizontal component of motion vector
     int16_t y;  ///< vertical component of motion vector
@@ -307,20 +313,24 @@  typedef struct DBParams {
 #define HEVC_FRAME_FLAG_SHORT_REF (1 << 1)
 #define HEVC_FRAME_FLAG_LONG_REF  (1 << 2)
 #define HEVC_FRAME_FLAG_BUMPING   (1 << 3)
+#define HEVC_FRAME_FLAG_MV        (1 << 4)
 
 typedef struct HEVCFrame {
     AVFrame *frame;
     ThreadFrame tf;
     MvField *tab_mvf;
+    CodingUnitHelper *cuh;
     RefPicList *refPicList;
     RefPicListTab **rpl_tab;
     int ctb_count;
     int poc;
+    int cu_count;
     struct HEVCFrame *collocated_ref;
 
     AVBufferRef *tab_mvf_buf;
     AVBufferRef *rpl_tab_buf;
     AVBufferRef *rpl_buf;
+    AVBufferRef *cuh_buf;
 
     AVBufferRef *hwaccel_priv_buf;
     void *hwaccel_picture_private;
@@ -405,11 +415,14 @@  typedef struct HEVCContext {
     uint8_t *sao_pixel_buffer_h[3];
     uint8_t *sao_pixel_buffer_v[3];
 
+    int output_frame_poc;
+
     HEVCParamSets ps;
     HEVCSEI sei;
     struct AVMD5 *md5_ctx;
 
     AVBufferPool *tab_mvf_pool;
+    AVBufferPool *cuh_pool;
     AVBufferPool *rpl_tab_pool;
 
     ///< candidate references for the current frame