Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: "mypopy@gmail.com" <mypopy@gmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Cc: Thomas Siedel <thomas.ff@spin-digital.com>
Subject: Re: [FFmpeg-devel] [PATCH v1 00/11] Add support for H266/VVC
Date: Tue, 10 Jan 2023 19:08:23 +0800
Message-ID: <CACYjbn3RWTyBZf2J5EkP5evVktJVDLMD7kWkwxROfHTn0Wvk8g@mail.gmail.com> (raw)
In-Reply-To: <CAFXK13en-t6MAHDdTyPwnMEvQKUzhDY5H6YY0hPP7Ag8yJF+Kg@mail.gmail.com>

On Tue, Jan 10, 2023 at 10:56 AM Nuo Mi <nuomi2021@gmail.com> wrote:
>
> Hi Thomas
> It works for me.
>
> BTW, I will send out a C version of VVC native decoder very soon.
> Please also help review and improve it.
> Thank you.
Ha, it's great news
>
> On Tue, Jan 3, 2023 at 10:02 PM Thomas Siedel <thomas.ff@spin-digital.com>
> wrote:
>
> > On Thu, 15 Dec 2022 at 10:11, Thomas Siedel <thomas.ff@spin-digital.com>
> > wrote:
> >
> >> On Tue, 13 Dec 2022 at 07:19, Nuo Mi <nuomi2021@gmail.com> wrote:
> >>
> >>> Hi Thomas,
> >>> Thank you for sending the patch set.
> >>> It seems the patchset is based on
> >>> https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=3487
> >>> Please do not change the author's name.
> >>>
> >>> thank you
> >>>
> >>
> >> Some code regarding VVC parsing is based on another FFmpeg fork. This
> >> fork had been based on your patch set, it seems.
> >> On top of this, I did additional modifications to the parsing code and
> >> added the other code regarding format support and the decoder and encoder
> >> integration.
> >>
> >> It was not my intent to hide you as an author of the parsing code.
> >> Your patch set has been in an unmerged state for over 1.5 years now
> >> without new progress, so I assumed that these patches are kind of
> >> discontinued.
> >> Therefore I started submitting my own patchset to get VVC integrated into
> >> FFmpeg.
> >> I kept your original copyright notices in libavformat/vvcdec.c and
> >> livavcodec/vvc_parser.c.
> >> Apart from this, I am not sure how / where else this could be documented
> >> and how other authors can be appropriately referenced.
> >>
> >> Could you explain in more detail what you mean by changing the author's
> >> name?
> >> Do you have any suggestions on what to do in this case or how to change
> >> it?
> >>
> >>
> > I just submitted a new version of the patch set.
> > Among other changes, I now put you as the author of the first three
> > patches, as most of their content is based on your original patch set.
> > I made some modifications to them, so I added myself as a co-author.
> > For patches 4 and 5, I put you as the co-author because some of their
> > content is based on your original patch set, but only to a smaller extent.
> > All other patches are independent.
> >
> > I hope that this solution is OK for you. If not, please let me know.
> >
> >
> >>
> >> On Wed, Oct 19, 2022 at 3:26 PM <thomas.ff@spin-digital.com> wrote:
> >>>
> >>>> From: Thomas Siedel <thomas.ff@spin-digital.com>
> >>>>
> >>>> This patch set adds H266/VVC support.
> >>>> This includes parsing, muxing, demuxing, decoding and encoding.
> >>>> Decoding is done using the external library VVdeC
> >>>> (https://github.com/fraunhoferhhi/vvdec.git) and can be enabled with
> >>>> --enable-libvvdec.
> >>>> Encoding is done using the external library VVenC
> >>>> (https://github.com/fraunhoferhhi/vvenc.git) and can be enabled with
> >>>> --enable-libvvenc.
> >>>>
> >>>> Thomas Siedel (11):
> >>>>   avcodec: add enum types for H266/VVC
> >>>>   avcodec: add cbs for H266/VVC
> >>>>   avcodec: enable cbs for H266/VVC
> >>>>   avcodec: add bitstream parser for H266/VVC
> >>>>   avcodec: add MP4 to annexb support for H266/VVC
> >>>>   avformat: add demuxer and probe support for H266/VVC
> >>>>   avformat: add muxer support for H266/VVC
> >>>>   avcodec: add external decoder libvvdec for H266/VVC
> >>>>   avcodec: add external encoder libvvenc for H266/VVC
> >>>>   avformat: add ts stream types for H266/VVC
> >>>>   avcodec: increase minor version for H266/VVC
> >>>>
> >>>>  configure                             |   16 +-
> >>>>  libavcodec/Makefile                   |    6 +
> >>>>  libavcodec/allcodecs.c                |    2 +
> >>>>  libavcodec/bitstream_filters.c        |    2 +
> >>>>  libavcodec/cbs.c                      |    6 +
> >>>>  libavcodec/cbs_h2645.c                |  384 +++-
> >>>>  libavcodec/cbs_h266.h                 |  791 +++++++
> >>>>  libavcodec/cbs_h266_syntax_template.c | 3010 +++++++++++++++++++++++++
> >>>>  libavcodec/cbs_internal.h             |    3 +-
> >>>>  libavcodec/cbs_sei.c                  |   29 +
> >>>>  libavcodec/h2645_parse.c              |   71 +-
> >>>>  libavcodec/h266_metadata_bsf.c        |  145 ++
> >>>>  libavcodec/libvvdec.c                 |  511 +++++
> >>>>  libavcodec/libvvenc.c                 |  432 ++++
> >>>>  libavcodec/parsers.c                  |    1 +
> >>>>  libavcodec/version.h                  |    2 +-
> >>>>  libavcodec/vvc.h                      |  142 ++
> >>>>  libavcodec/vvc_mp4toannexb_bsf.c      |  318 +++
> >>>>  libavcodec/vvc_paramset.c             |  972 ++++++++
> >>>>  libavcodec/vvc_paramset.h             |  429 ++++
> >>>>  libavcodec/vvc_parse_extradata.c      |  241 ++
> >>>>  libavcodec/vvc_parse_extradata.h      |   36 +
> >>>>  libavcodec/vvc_parser.c               |  588 +++++
> >>>>  libavformat/Makefile                  |    8 +-
> >>>>  libavformat/allformats.c              |    2 +
> >>>>  libavformat/demux.c                   |    7 +-
> >>>>  libavformat/isom.c                    |    1 +
> >>>>  libavformat/isom_tags.c               |    3 +
> >>>>  libavformat/mov.c                     |    6 +
> >>>>  libavformat/movenc.c                  |   41 +-
> >>>>  libavformat/mpeg.c                    |    3 +
> >>>>  libavformat/mpeg.h                    |    1 +
> >>>>  libavformat/mpegts.c                  |    2 +
> >>>>  libavformat/mpegts.h                  |    1 +
> >>>>  libavformat/mpegtsenc.c               |   65 +
> >>>>  libavformat/rawenc.c                  |   23 +
> >>>>  libavformat/vvc.c                     |  918 ++++++++
> >>>>  libavformat/vvc.h                     |   99 +
> >>>>  libavformat/vvcdec.c                  |   61 +
> >>>>  39 files changed, 9366 insertions(+), 12 deletions(-)
> >>>>  create mode 100644 libavcodec/cbs_h266.h
> >>>>  create mode 100644 libavcodec/cbs_h266_syntax_template.c
> >>>>  create mode 100644 libavcodec/h266_metadata_bsf.c
> >>>>  create mode 100644 libavcodec/libvvdec.c
> >>>>  create mode 100644 libavcodec/libvvenc.c
> >>>>  create mode 100644 libavcodec/vvc.h
> >>>>  create mode 100644 libavcodec/vvc_mp4toannexb_bsf.c
> >>>>  create mode 100644 libavcodec/vvc_paramset.c
> >>>>  create mode 100644 libavcodec/vvc_paramset.h
> >>>>  create mode 100644 libavcodec/vvc_parse_extradata.c
> >>>>  create mode 100644 libavcodec/vvc_parse_extradata.h
> >>>>  create mode 100644 libavcodec/vvc_parser.c
> >>>>  create mode 100644 libavformat/vvc.c
> >>>>  create mode 100644 libavformat/vvc.h
> >>>>  create mode 100644 libavformat/vvcdec.c
> >>>>
> >>>> --
> >>>> 2.25.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".

      reply	other threads:[~2023-01-10 11:08 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-19  7:24 thomas.ff
2022-10-19  7:24 ` [FFmpeg-devel] [PATCH v1 01/11] avcodec: add enum types " thomas.ff
2022-10-19  7:24 ` [FFmpeg-devel] [PATCH v1 02/11] avcodec: add cbs " thomas.ff
2022-10-19  7:25 ` [FFmpeg-devel] [PATCH v1 03/11] avcodec: enable " thomas.ff
2022-10-19 23:07   ` James Almer
2022-10-24 14:23     ` Thomas Siedel
2022-10-19  7:25 ` [FFmpeg-devel] [PATCH v1 04/11] avcodec: add bitstream parser " thomas.ff
2022-10-19  7:25 ` [FFmpeg-devel] [PATCH v1 05/11] avcodec: add MP4 to annexb support " thomas.ff
2022-10-19  7:25 ` [FFmpeg-devel] [PATCH v1 06/11] avformat: add demuxer and probe " thomas.ff
2022-10-19 12:06   ` Michael Niedermayer
2022-10-24 14:19     ` Thomas Siedel
2022-10-19  7:25 ` [FFmpeg-devel] [PATCH v1 07/11] avformat: add muxer " thomas.ff
2022-10-19  7:25 ` [FFmpeg-devel] [PATCH v1 08/11] avcodec: add external decoder libvvdec " thomas.ff
2022-10-19  7:25 ` [FFmpeg-devel] [PATCH v1 09/11] avcodec: add external encoder libvvenc " thomas.ff
2022-10-19  7:25 ` [FFmpeg-devel] [PATCH v1 10/11] avformat: add ts stream types " thomas.ff
2022-10-19  7:25 ` [FFmpeg-devel] [PATCH v1 11/11] avcodec: increase minor version " thomas.ff
2022-12-13  6:19 ` [FFmpeg-devel] [PATCH v1 00/11] Add support " Nuo Mi
2022-12-15  9:11   ` Thomas Siedel
2023-01-03 14:02     ` Thomas Siedel
2023-01-10  2:56       ` Nuo Mi
2023-01-10 11:08         ` mypopy [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CACYjbn3RWTyBZf2J5EkP5evVktJVDLMD7kWkwxROfHTn0Wvk8g@mail.gmail.com \
    --to=mypopy@gmail.com \
    --cc=ffmpeg-devel@ffmpeg.org \
    --cc=thomas.ff@spin-digital.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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