Thursday, December 23, 2021

Re: UPDATE: libavif 0.9.3

On Sun, Dec 05, 2021 at 02:24:37PM +0100, Stefan Hagen wrote:
> Brad Smith wrote:
> > Here is an update to libavif 0.9.3.
> >
> >
> > ## [0.9.3] - 2021-10-20
> >
> > ### Added
> > * Support for progressive AVIFs and operating point selection
> > * Add automatic tile scaling to the item's ispe or track's dims
> > * Add diagnostic messages for AV1 decode failures
> > * avifdec: Add PNG compression level arg
> > * Make image size limit configurable, expose to avifdec
> > * Add the AVIF_STRICT_ALPHA_ISPE_REQUIRED flag
> >
> > ### Changed
> > * Mandate ispe and disallow zero width or height (#640).
> > * Re-map libavif speed 7-10 to libaom speed 7-9 (#682)
> > * Refer to https://aomedia-review.googlesource.com/c/aom/+/140624
> > * If you were using libaom with the following avif speed setting:
> > * - speed 0-6: no change is needed
> > * - speed 7: change to speed 6 for the same results
> > * - speed 8-9: re-test and re-adjust speed according to your app needs
> > * Update aom.cmd: v3.2.0
> > * Update dav1d.cmd: 0.9.2
> > * Pass TestCase's minQuantizer, maxQuantizer, speed to encoder.
> > * Regenerate tests.json
> > * Disable JSON-based tests for now, the metrics are inconsistent/unreliable
> > * Set diagnostic message for aom_codec_set_option()
> > * Re-map libavif-libaom speed settings (#682)
> > * Bump of version in CMakeLists.txt was forgotten
> > * avifdec: Better message for unsupported file extension
> > * Do not copy input image when encoding with libaom unless width or height is 1
> > * Fix the comment for AVIF_STRICT_PIXI_REQUIRED
> > * Update libavif.pc.cmake (#692)
> > * In 32-bit builds set dav1d's frame_size_limit setting to 8192*8192
> > * Allocate alpha alongside YUV (if necessary) during y4m decode to avoid incorrect alphaRowBytes math
> > * Change avif_decode_fuzzer to be more like Chrome
> > * Update codec_dav1d.c for the new threading model
> > * Generalized ipco property deduplication
> > * Rename avifParseMoovBox to avifParseMovieBox for consistency
> > * Simplify idat storage for avifMeta structure (#756)
> > * Fix oss-fuzz coverage build failure of dav1d
> > * Redesign AVIF_DECODER_SOURCE_AUTO to honor the FileTypeBox's major brand
> > * Use "C420" as default Y4M color space parametee
>
> Hi Brad,
>
> When I run the test target on the previous version, I get:
> Complete. 18 tests ran, 90 skipped, 0 failed.
>
> On this version, I get:
> Complete. 18 tests ran, 90 skipped, 6 failed.
>
> I assume this is not expected.
> I attached the two test logs.

Upstream says essentially this is expected as the tests are unreliable.

"We have a GitHub issue tracking the rewrite of the JSON-based tests here, as they are unreliable:

#684

They made sense when libavif itself was in its infancy and libaom was locked at v1.0 for a very long time, but with such a myriad of modern settings/optimizations at the codec level, assuming that the variance at the pixel level for these tests will always be exactly under some semi-arbitrary threshold doesn't make sense anymore."

and commited a diff to disable the JSON tests for the time being in their GitHub
CI and suggested the same.


Index: Makefile
===================================================================
RCS file: /home/cvs/ports/graphics/libavif/Makefile,v
retrieving revision 1.16
diff -u -p -u -p -r1.16 Makefile
--- Makefile 28 Jun 2021 17:15:06 -0000 1.16
+++ Makefile 24 Dec 2021 01:23:19 -0000
@@ -4,10 +4,10 @@ COMMENT= library for encoding and decodi

GH_ACCOUNT= AOMediaCodec
GH_PROJECT= libavif
-GH_TAGNAME= v0.9.2
+GH_TAGNAME= v0.9.3
CATEGORIES= graphics

-SHARED_LIBS= avif 4.1
+SHARED_LIBS= avif 4.2

HOMEPAGE= https://github.com/AOMediaCodec/libavif

@@ -33,7 +33,8 @@ CONFIGURE_ARGS+=-DAVIF_BUILD_APPS=ON \
-DAVIF_CODEC_AOM_DECODE=OFF \
-DAVIF_CODEC_DAV1D=ON

+# https://github.com/AOMediaCodec/libavif/commit/11d6725af628b116066f4bd2e5b810498c584438
do-test:
- ${WRKBUILD}/aviftest ${WRKSRC}/tests/data
+ ${WRKBUILD}/aviftest ${WRKSRC}/tests/data --io-only

.include <bsd.port.mk>
Index: distinfo
===================================================================
RCS file: /home/cvs/ports/graphics/libavif/distinfo,v
retrieving revision 1.12
diff -u -p -u -p -r1.12 distinfo
--- distinfo 28 Jun 2021 17:15:06 -0000 1.12
+++ distinfo 29 Oct 2021 07:21:30 -0000
@@ -1,2 +1,2 @@
-SHA256 (libavif-0.9.2.tar.gz) = 1mB9ZUrcQKOS2oPapypP+ALNdQwEWmgTHJMFY5wQ/Fw=
-SIZE (libavif-0.9.2.tar.gz) = 1987058
+SHA256 (libavif-0.9.3.tar.gz) = vNmh9X+YKpYV634vr4cjbciOsdDIhvNHHHRA6tYFBg0=
+SIZE (libavif-0.9.3.tar.gz) = 2000521

No comments:

Post a Comment