Release: Fix WebP Export (#2499)

- test and release #2498

Author: mikedh

pyproject.toml

@@ -5,7 +5,7 @@ requires = ["setuptools >= 61.0", "wheel"]
 [project]
 name = "trimesh"
 requires-python = ">=3.8"
-version = "4.11.0"
+version = "4.11.1"
 authors = [{name = "Michael Dawson-Haggerty", email = "[email protected]"}]
 license = {file = "LICENSE.md"}
 description = "Import, export, process, analyze and view triangular meshes."

tests/test_gltf.py

@@ -1134,6 +1134,23 @@ def test_simple_material_conversion(self):
         simple = mesh.visual.material.to_simple()
         assert simple.name == mesh.visual.material.name
 
+    def test_webp_roundtrip(self):
+        m = g.get_mesh("fuze.obj")
+        e = m.export(file_type="glb", extension_webp=True)
+        r = g.trimesh.load_mesh(g.trimesh.util.wrap_as_stream(e), file_type="glb")
+
+        # compare RGBA images for the roundtripped texture
+        # make sure the webp roundtrip wasn't crazy
+        a = g.np.array(m.visual.material.image)
+        b = g.np.array(r.visual.material.baseColorTexture)
+
+        assert a.shape == b.shape
+
+        # roundtrip with a codec produces artifacts
+        # if they are much different this number will be absolutely huge
+        mean_squared_error = ((a - b) ** 2).sum() / g.np.prod(a.shape)
+        assert mean_squared_error < 10.0
+
 
 if __name__ == "__main__":
     g.trimesh.util.attach_to_log()

trimesh/exchange/gltf/extensions.py

@@ -194,7 +194,7 @@ def handle_extensions(
         try:
             # Build context dict with data + all kwargs
             context = {"data": data, **kwargs}
-            if result := _handlers[scope][ext_name](context):
+            if (result := _handlers[scope][ext_name](context)) is not None:
                 results[ext_name] = result
         except Exception as e:
             log.warning(f"failed to process extension {ext_name}: {e}")