sergiotarxz/libvips
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

speed up test suite

Browse Source 
ouch we were making a huge avif image in memory
This commit is contained in:
John Cupitt 2021-10-25 20:40:42 +01:00
parent 29fb557e24
commit a16f118b0c
1 changed files with 28 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

41
test/test-suite/test_foreign.py
View File

@ -1126,31 +1126,44 @@ class TestForeign:
@skip_if_no("heifsave") @skip_if_no("heifsave")
@pytest.mark.skipif(sys.platform == "darwin", reason="fails with latest libheif/aom from Homebrew") @pytest.mark.skipif(sys.platform == "darwin", reason="fails with latest libheif/aom from Homebrew")
def test_heifsave(self): def test_avifsave(self):
# TODO: Reduce the threshold once https://github.com/strukturag/libheif/issues/533 is resolved. # TODO: Reduce the threshold once https://github.com/strukturag/libheif/issues/533 is resolved.
self.save_load_buffer("heifsave_buffer", "heifload_buffer", self.save_load_buffer("heifsave_buffer", "heifload_buffer",
self.colour, 80, compression="av1", self.colour, 80, compression="av1",
lossless=True) lossless=True)
self.save_load("%s.avif", self.colour) self.save_load("%s.avif", self.colour)
# uncomment to test lossless mode, will take a while @skip_if_no("heifsave")
#im = pyvips.Image.new_from_file(AVIF_FILE) @pytest.mark.skipif(sys.platform == "darwin", reason="fails with latest libheif/aom from Homebrew")
#buf = im.heifsave_buffer(lossless=True, compression="av1") @pytest.mark.skip()
#im2 = pyvips.Image.new_from_buffer(buf, "") def test_avifsave_lossless(self):
# this takes FOREVER
im = pyvips.Image.new_from_file(AVIF_FILE)
buf = im.heifsave_buffer(lossless=True, compression="av1")
im2 = pyvips.Image.new_from_buffer(buf, "")
# not in fact quite lossless # not in fact quite lossless
#assert abs(im.avg() - im2.avg()) < 3 assert abs(im.avg() - im2.avg()) < 3
@skip_if_no("heifsave")
@pytest.mark.skipif(sys.platform == "darwin", reason="fails with latest libheif/aom from Homebrew")
def test_avifsave_Q(self):
# higher Q should mean a bigger buffer, needs libheif >= v1.8.0, # higher Q should mean a bigger buffer, needs libheif >= v1.8.0,
# see: https://github.com/libvips/libvips/issues/1757 # see: https://github.com/libvips/libvips/issues/1757
b1 = self.mono.heifsave_buffer(Q=10, compression="av1") b1 = self.mono.heifsave_buffer(Q=10, compression="av1")
b2 = self.mono.heifsave_buffer(Q=90, compression="av1") b2 = self.mono.heifsave_buffer(Q=90, compression="av1")
assert len(b2) > len(b1) assert len(b2) > len(b1)
@skip_if_no("heifsave")
@pytest.mark.skipif(sys.platform == "darwin", reason="fails with latest libheif/aom from Homebrew")
def test_avifsave_chroma(self):
# Chroma subsampling should produce smaller file size for same Q # Chroma subsampling should produce smaller file size for same Q
b1 = self.colour.heifsave_buffer(compression="av1", subsample_mode="on") b1 = self.colour.heifsave_buffer(compression="av1", subsample_mode="on")
b2 = self.colour.heifsave_buffer(compression="av1", subsample_mode="off") b2 = self.colour.heifsave_buffer(compression="av1", subsample_mode="off")
assert len(b2) > len(b1) assert len(b2) > len(b1)
@skip_if_no("heifsave")
@pytest.mark.skipif(sys.platform == "darwin", reason="fails with latest libheif/aom from Homebrew")
def test_avifsave_icc(self):
# try saving an image with an ICC profile and reading it back # try saving an image with an ICC profile and reading it back
# not all libheif have profile support, so put it in an if # not all libheif have profile support, so put it in an if
buf = self.colour.heifsave_buffer(Q=10, compression="av1") buf = self.colour.heifsave_buffer(Q=10, compression="av1")
@ -1160,19 +1173,21 @@ class TestForeign:
p2 = im.get("icc-profile-data") p2 = im.get("icc-profile-data")
assert p1 == p2 assert p1 == p2
# add tests for exif, xmp, ipct # add tests for xmp, ipct
# the exif test will need us to be able to walk the header, # the exif test will need us to be able to walk the header,
# we can't just check exif-data # we can't just check exif-data
# test that exif changes change the output of heifsave @skip_if_no("heifsave")
@pytest.mark.skipif(sys.platform == "darwin", reason="fails with latest libheif/aom from Homebrew")
def test_avifsave_exif(self):
# first make sure we have exif support # first make sure we have exif support
z = pyvips.Image.new_from_file(AVIF_FILE) x = pyvips.Image.new_from_file(JPEG_FILE)
if z.get_typeof("exif-ifd0-Make") != 0: if x.get_typeof("exif-ifd0-Orientation") != 0:
x = z.copy() x = x.copy()
x.set("exif-ifd0-Make", "banana") x.set_type(pyvips.GValue.gstr_type, "exif-ifd0-XPComment", "banana")
buf = x.heifsave_buffer(Q=10, compression="av1") buf = x.heifsave_buffer(Q=10, compression="av1")
y = pyvips.Image.new_from_buffer(buf, "") y = pyvips.Image.new_from_buffer(buf, "")
assert y.get("exif-ifd0-Make").split(" ")[0] == "banana" assert y.get("exif-ifd0-XPComment").startswith("banana")
@skip_if_no("jp2kload") @skip_if_no("jp2kload")
def test_jp2kload(self): def test_jp2kload(self):