Skip to content

fix: Add Assimp "LimitBoneWeights" aiPostProcessSteps#3110

Open
Nicogo1705 wants to merge 1 commit intostride3d:masterfrom
Nicogo1705:Fix-bones-weights
Open

fix: Add Assimp "LimitBoneWeights" aiPostProcessSteps#3110
Nicogo1705 wants to merge 1 commit intostride3d:masterfrom
Nicogo1705:Fix-bones-weights

Conversation

@Nicogo1705
Copy link
Copy Markdown
Contributor

fix: Add aiProcess_LimitBoneWeights to Assimp post-processing flags

When importing animated models, we got a lot of warn about bones excess weights bones per vertex at compile time, it was slowing down the build process and log reading.

This fix adds aiProcess_LimitBoneWeights to the Assimp post-processing step. This flag caps bone influences per vertex to 4 during import, which is the standard hardware limit and what Stride's runtime expects. The compiler no longer needs to deal with excess influences, eliminating the warnings and reducing build time on large projects.

Related Issue

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My change requires a change to the documentation.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have built and run the editor to try this change out.

@Eideren Eideren changed the title Add Assimp "LimitBoneWeights" aiPostProcessSteps fix: Add Assimp "LimitBoneWeights" aiPostProcessSteps Apr 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants