Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Normalize bone weights #185

Closed
emackey opened this issue Apr 30, 2018 · 3 comments
Closed

Normalize bone weights #185

emackey opened this issue Apr 30, 2018 · 3 comments

Comments

@emackey
Copy link
Member

emackey commented Apr 30, 2018

The CesiumMan sample model came through this converter with apparently non-normalized bone weights. Not sure if COLLADA allows such weights, but it's becoming apparent that glTF does not, and the non-normalized weights are showing up as rendering errors in various frameworks now.

Can this converter normalize these weights automatically?

Spec discussion: KhronosGroup/glTF#1213 (comment)

BabylonJS issue: BabylonJS/Babylon.js#4240

Cesium issue: CesiumGS/cesium#6447

Source COLLADA model: https://github.com/KhronosGroup/glTF-Sample-Models/tree/master/sourceModels/CesiumMan

@lasalvavida
Copy link
Contributor

Seems like it should be reasonably simple to do. Normalize the weights here and this should be resolved.

@akulagrawal
Copy link
Contributor

Hi Sir. I am new to open source and interested to work on this issue. Can I take this up?

lasalvavida added a commit that referenced this issue May 23, 2018
@lasalvavida
Copy link
Contributor

Closed by #187.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants