Some transformations missing after transforming gltf file

pmndrs / gltfjsx

🎮 Turns GLTFs into JSX components

https://gltf.pmnd.rs

MIT License

4.9k stars 318 forks source link

Some transformations missing after transforming gltf file #277

Open benthillerkus opened 2 months ago

benthillerkus commented 2 months ago

Using this viewer: https://gltf-viewer.donmccurdy.com/

Here's both a transformed and the original GLTF. With the transformed GLTF the window shutters are misplaced. venice-transformed.zip

hichemfantar commented 2 months ago

could be related to #235

donmccurdy commented 2 months ago

I think this issue is unrelated to removing UVs and normals: upstream issue in https://github.com/donmccurdy/glTF-Transform/issues/1505 with details.

donmccurdy commented 2 months ago

To be fixed upstream by https://github.com/donmccurdy/glTF-Transform/pull/1507. The cause of the bug was that glTF Transform's instance() function did not correctly handle files that already used glTF's GPU instancing extension, EXT_mesh_gpu_instancing.