donmccurdy / glTF-Transform

glTF 2.0 SDK for JavaScript and TypeScript, on Web and Node.js.
https://gltf-transform.dev
MIT License
1.3k stars 145 forks source link

perf(functions): Merge lossy/lossless weld implementations #1356

Closed donmccurdy closed 2 months ago

donmccurdy commented 2 months ago

The newer lossless weld implementation — based on Meshoptimizer's approach — is vastly faster than the previous lossy welder, which would crash on large meshes. This PR re-implements the lossy welder using the same approach, with pre-processing to snap vertex data to a quantized grid before welding.

Remaining:

Related:

donmccurdy commented 2 months ago

Closing in favor of https://github.com/donmccurdy/glTF-Transform/pull/1357.