search

reattiva / Urho3D-Blender

Blender to Urho3D mesh exporter
Other
124 stars 52 forks source link

Per geometry bone limiting messes up bounding boxes #65

Open rdpeake opened 7 years ago

rdpeake commented 7 years ago

I have a model with a skeleton which has over the default max number of bones per geometry (64) and when exporting this geometry the code at export_urho.py#L1035 executes to remap the bone indexes - which seems to work fine, however, the code at export_Urho.py#L1092 does not account for the remapped indexes and adds extra vertecies to lower index bone bounding boxes - throwing them out of proportion. this also prevents higher index bones from ever getting a bounding box.

I was able to overcome this issue on some platforms by raising the max skinning bones to 128 however not every platform supports this many bones.

reattiva commented 7 years ago

Hello, can you test if this branch helps ? https://github.com/reattiva/Urho3D-Blender/tree/remap_bb

rdpeake commented 7 years ago

That branch seems to solves the issue, thanks

slapin commented 7 years ago

Looks like #61 is clone of this one (or vice versa) so that one can be closed too if the branch is merged.