From: Marcos Ashton via ffmpeg-devel <ffmpeg-devel@ffmpeg.org>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Cc: Marcos Ashton <marcosashiglesias@gmail.com>
Subject: [FFmpeg-devel] Re: [PATCH 0/3] tests: improve FATE coverage for film_grain_params, CNG, aderivative/aintegral
Date: Sun, 22 Feb 2026 15:44:00 +0000
Message-ID: <CAC=oWdrZ-EO=W5O+8aQXgvDgewYWW4gtyAXEOTPV4ACwSnomoQ@mail.gmail.com> (raw)
In-Reply-To: <CANzXe1NaTZh6yaxq_g4-JOj4FKX6Dca5yVho8PjAGx=iPorSeQ@mail.gmail.com>
Hi Hassan,
Thanks for the suggestion! I submitted via the mailing list as that's what
the developer docs and the GSoC project page pointed me to is
code.ffmpeg.org now the preferred place for new patches? Happy to mirror
there too if it helps with review.
Marcos
On Sun, 22 Feb 2026 at 13:59, hassan hany via ffmpeg-devel <
ffmpeg-devel@ffmpeg.org> wrote:
> you probably want to make a pr over at https://code.ffmpeg.org
>
> On Fri, Feb 20, 2026 at 8:41 PM marcos ashton via ffmpeg-devel <
> ffmpeg-devel@ffmpeg.org> wrote:
>
> > Hi, this is my qualification task for the GSoC 2026
> > "Improving FATE Test Coverage" project.
> >
> > This series adds 7 new FATE tests for three previously untested files:
> >
> > 1/3: Unit test for libavutil/film_grain_params.c (~98% coverage)
> > 2/3: Encode/decode roundtrip for libavcodec/cngdec.c (~92% coverage)
> > 3/3: Filter tests for libavfilter/af_aderivative.c (~87% coverage)
> >
> > marcos ashton (3):
> > tests/fate/libavutil: add FATE test for film_grain_params
> > tests/fate/voice: add FATE test for CNG encoder roundtrip
> > tests/fate/filter-audio: add FATE tests for aderivative and aintegral
> >
> > libavutil/Makefile | 1 +
> > libavutil/tests/film_grain_params.c | 240 +++++++++++++++++++++++++
> > tests/fate/filter-audio.mak | 25 +++
> > tests/fate/libavutil.mak | 4 +
> > tests/fate/voice.mak | 8 +
> > tests/ref/fate/cng-encode | 84 +++++++++
> > tests/ref/fate/film_grain_params | 30 ++++
> > tests/ref/fate/filter-aderivative | 70 ++++++++
> > tests/ref/fate/filter-aderivative-fltp | 70 ++++++++
> > tests/ref/fate/filter-aderivative-s32p | 70 ++++++++
> > tests/ref/fate/filter-aintegral | 70 ++++++++
> > tests/ref/fate/filter-aintegral-dblp | 70 ++++++++
> > 12 files changed, 742 insertions(+)
> > create mode 100644 libavutil/tests/film_grain_params.c
> > create mode 100644 tests/ref/fate/cng-encode
> > create mode 100644 tests/ref/fate/film_grain_params
> > create mode 100644 tests/ref/fate/filter-aderivative
> > create mode 100644 tests/ref/fate/filter-aderivative-fltp
> > create mode 100644 tests/ref/fate/filter-aderivative-s32p
> > create mode 100644 tests/ref/fate/filter-aintegral
> > create mode 100644 tests/ref/fate/filter-aintegral-dblp
> >
> > --
> > 2.53.0
> >
> > _______________________________________________
> > ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
> > To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org
> >
> _______________________________________________
> ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
> To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org
>
_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org
next prev parent reply other threads:[~2026-02-22 18:43 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-20 16:25 [FFmpeg-devel] " marcos ashton via ffmpeg-devel
2026-02-20 16:25 ` [FFmpeg-devel] [PATCH 1/3] tests/fate/libavutil: add FATE test for film_grain_params marcos ashton via ffmpeg-devel
2026-02-20 16:25 ` [FFmpeg-devel] [PATCH 2/3] tests/fate/voice: add FATE test for CNG encoder roundtrip marcos ashton via ffmpeg-devel
2026-02-20 16:25 ` [FFmpeg-devel] [PATCH 3/3] tests/fate/filter-audio: add FATE tests for aderivative and aintegral marcos ashton via ffmpeg-devel
2026-02-22 13:59 ` [FFmpeg-devel] Re: [PATCH 0/3] tests: improve FATE coverage for film_grain_params, CNG, aderivative/aintegral hassan hany via ffmpeg-devel
2026-02-22 15:44 ` Marcos Ashton via ffmpeg-devel [this message]
2026-02-22 15:51 ` hassan hany via ffmpeg-devel
2026-02-22 17:14 ` Marcos Ashton via ffmpeg-devel
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='CAC=oWdrZ-EO=W5O+8aQXgvDgewYWW4gtyAXEOTPV4ACwSnomoQ@mail.gmail.com' \
--to=ffmpeg-devel@ffmpeg.org \
--cc=marcosashiglesias@gmail.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