search

google / filament

Filament is a real-time physically based rendering engine for Android, iOS, Windows, Linux, macOS, and WebGL2
https://google.github.io/filament/
Apache License 2.0
17.35k stars 1.83k forks source link

gltf sparse accessors bug #7900

Open August1996 opened 3 weeks ago

August1996 commented 3 weeks ago

Describe the bug

https://github.com/google/filament/assets/15972489/6f3312d6-43f9-4e20-abec-bddf7bcfbf26

Firstly,the "male_sparse.glb" is from male.glb(use gltf-transform sparse to process male.glb)

As the video shown, there is two bugs 1.There is only light was effected when i use seekbar to change morph target weights of male_sparse.glb. 2.The face will be broken when i change the model as this sequence(male.glb -> male_sparse.glb -> male.glb -> male_sparse.glb)

To Reproduce the sample code has been upload to github.

https://github.com/August1996/filament-bugreport/tree/main/SparseTest

Smartphone (please complete the following information):

Filament version is 1.52.0

@romainguy @pixelflinger @bejado @poweifeng

August1996 commented 3 weeks ago

image

The face broken will fixed when i cancel destroy morph target.

August1996 commented 3 weeks ago

image The face broken will be fixed alose when i cancel to delete mTbHandle and mPbHandle.It seems that the previous delete operation affects the next texture generation.

August1996 commented 3 weeks ago

@pixelflinger @poweifeng Hello, How is this issue going?

pixelflinger commented 3 weeks ago

No progress. It's unfortunately not a priority at the moment. We'll get to it eventually.