Skip to content

fix: xmodels with more than one root bone are not linkable#752

Merged
Laupetin merged 4 commits intomainfrom
fix/relink-dumped-xmodels
May 3, 2026
Merged

fix: xmodels with more than one root bone are not linkable#752
Laupetin merged 4 commits intomainfrom
fix/relink-dumped-xmodels

Conversation

@Laupetin
Copy link
Copy Markdown
Owner

@Laupetin Laupetin commented Apr 20, 2026

Apparently I thought there were no models with more than one root bone, but there are.
e.g.

  • IW3 code_post_gfx void
  • T6 common_mp t6_wpn_smg_pdw57_view

and probably a bunch more.

TODOs:

  • Properly add all root bones to dumped models skin
  • Load bones and child bones for all root bones and don't error on more than one root bone
  • Reorder bones if they are not sorted by parent (they are exported as such, but this order is not guaranteed to come from external tools)

fixes: #733

@Laupetin Laupetin added the bug Something isn't working label Apr 20, 2026
@Laupetin Laupetin force-pushed the fix/relink-dumped-xmodels branch 2 times, most recently from 9aa9101 to db0a177 Compare April 27, 2026 15:23
@Laupetin Laupetin force-pushed the fix/relink-dumped-xmodels branch 2 times, most recently from 6997815 to c619cc5 Compare May 3, 2026 13:40
@Laupetin Laupetin force-pushed the fix/relink-dumped-xmodels branch from c619cc5 to 875e1fb Compare May 3, 2026 14:35
@Laupetin Laupetin marked this pull request as ready for review May 3, 2026 18:47
@Laupetin Laupetin force-pushed the fix/relink-dumped-xmodels branch from 56de916 to 582d8ab Compare May 3, 2026 19:01
@Laupetin Laupetin merged commit 1df727a into main May 3, 2026
7 checks passed
@Laupetin Laupetin deleted the fix/relink-dumped-xmodels branch May 3, 2026 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

IW3: dumped xmodel "void" cannot be linked again

1 participant