Skip to content

Traversal + lerping stuff #5929

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

metalgearsloth
Copy link
Contributor

@metalgearsloth metalgearsloth commented May 14, 2025

  • Undisables cross-parent lerping so grid traversal doesn't lurch.
  • Fixes some subtly incorrect lerp values if traversal was run in the setter.
  • Disable traversal when lerping in the rare case it traverses us onto a grid.
  • Mark the LocalRotation setter as obsolete, was probably just forgotten but this aligns with LocalPosition.

@ElectroJr

I imagine this may break some stuff but content stuff like container setting or w/e should use the no-lerp version maybe? Or we add a setcoordinates no lerp version helper?

Draft because grid <> grid lerps still lurch

- Undisables cross-parent lerping so grid traversal doesn't lurch.
- Fixes some subtly incorrect lerp values if traversal was run in the setter.
- Disable traversal when lerping in the rare case it traverses us onto a grid.
- Mark the LocalRotation setter as obsolete, was probably just forgotten but this aligns with LocalPosition.
@metalgearsloth metalgearsloth marked this pull request as draft May 14, 2025 09:16
@metalgearsloth metalgearsloth marked this pull request as ready for review May 14, 2025 09:27
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.

1 participant