Skip to content

[NC] Fix #179 Node Constraints OutSmooth not working#197

Merged
ManlyMarco merged 2 commits intoIllusionMods:mainfrom
Lusiocc:fix-issue-179
Feb 14, 2026
Merged

[NC] Fix #179 Node Constraints OutSmooth not working#197
ManlyMarco merged 2 commits intoIllusionMods:mainfrom
Lusiocc:fix-issue-179

Conversation

@Lusiocc
Copy link
Copy Markdown
Contributor

@Lusiocc Lusiocc commented Feb 5, 2026

Fixes #179
Prevent Nodes from desyncing with the actual object
Fix resetChild not respecting animation in/out timers

Tested a few different scenes in the games and none had problems from that change.

[NC] Prevent Nodes from desyncing with the actual object
[NC] Fix resetChild not respecting animation in/out timers

Signed-off-by: Lusiocc <252649741+Lusiocc@users.noreply.github.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes Node Constraints “OutSmooth” behavior (Issue #179) by making disconnect smoothing restore original transforms correctly and by preventing immediate resets from bypassing animation in/out timers, reducing visible desync/drift between backend node data and rendered transforms.

Changes:

  • Refactors disconnect smoothing to lerp/slerp back to original position/rotation/scale when resetOriginal* is enabled.
  • Adds an updateTimer parameter to SmoothDisconnectUpdate to avoid decrementing timers multiple times per frame.
  • Captures smoothStartValue on enable/disable transitions and ensures originalChildScale is recorded when enabling.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread NodesConstraints.Core/NodesConstraints.cs Outdated
Signed-off-by: Lusiocc <252649741+Lusiocc@users.noreply.github.com>
@ManlyMarco ManlyMarco merged commit ab2de91 into IllusionMods:main Feb 14, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Node Constraints OutSmooth not working

3 participants