KHR_draco_mesh_compression increase glb file load time

Bugs
1

in GLTFLoader._loadVertexDataAsync，geometry._boundingInfo is created according position accessor, and geometry.useBoundingInfoFromGeometry is set to true.

企业微信截图_4a73ad82-2da1-41d3-ae5d-f1ed48db3aff
企业微信截图_4a73ad82-2da1-41d3-ae5d-f1ed48db3aff1920×1609 186 KB

But In KHR_draco_mesh_compression, the above process is absent，As a result, mesh._refreshBoundInfo is always executed in GLTFLoader.loadNodeAsync

企业微信截图_72037765-888f-4681-bf2a-193f78929f42
企业微信截图_72037765-888f-4681-bf2a-193f78929f421680×1150 203 KB

so the file load time becomes larger.

2

Will be great to have @bghgary’s opinion here :slight_smile:

3

@Starryi That sounds like an oversight. Do you want to try and fix it?

4

@bghgary Yes, I want to fix it.

When the number of meshes and vertex data is particularly large, the performance gap between refreshBoundingInfo and updateBoundingInfo is obvious. I want to keep the load time as low as possible to enhance the user experience

5

@RaananW @bghgary pr is here fix: Load boundingInfo when use KHR_draco_mesh_compression by Starryi · Pull Request #15882 · BabylonJS/Babylon.js · GitHub

4 Likes