Skip to content

Fix: World transform computation#80

Merged
brenocq merged 2 commits intodevfrom
fix-world-transform
Apr 28, 2025
Merged

Fix: World transform computation#80
brenocq merged 2 commits intodevfrom
fix-world-transform

Conversation

@brenocq
Copy link
Owner

@brenocq brenocq commented Apr 28, 2025

There was a bug in the cmp::Transform component. The conversion from local transform to world transform was being performed from the leaf node to the root node instead of root node to leaf node, which resulted in wrong world transformation matrices when trying to move/rotate nested nodes with the Gizmo.

Before:

Peek.2025-04-28.20-07.mp4

After:

Peek.2025-04-28.20-09.mp4

@brenocq brenocq added module:component Component Module prio:medium Medium priority type:fix Something needs to be fixed labels Apr 28, 2025
@brenocq brenocq self-assigned this Apr 28, 2025
@brenocq brenocq added the version:patch Increases the patch version when added to a PR label Apr 28, 2025
@brenocq brenocq added the status:done Task completed successfully label Apr 28, 2025
@brenocq brenocq merged commit bcca01f into dev Apr 28, 2025
1 check passed
@brenocq brenocq deleted the fix-world-transform branch April 28, 2025 18:19
@brenocq brenocq changed the title Fix world transform computation Fix: World transform computation Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:component Component Module prio:medium Medium priority status:done Task completed successfully type:fix Something needs to be fixed version:patch Increases the patch version when added to a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant