Skip to content
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

Main: NodeAnimationTrack - add setUseLocalTranslation #2848

Conversation

chillywillysoft
Copy link
Contributor

Allows to select local space for translations on NodeAnimationTrack.

More focused version of #2840 that doesn't affect rotations and only allows choosing between parent and local.

Allows to select local space for translations on NodeAnimationTrack
@paroj
Copy link
Member

paroj commented May 27, 2023

@chillywillysoft after having looked at it some more:
could the same not be achieved by introducing an intermediate scene node to orient the animation? i.e.

animation(local) -> node(rotated)
== 
node(rotated)  -> [animation(parent) -> node]

@chillywillysoft
Copy link
Contributor Author

@paroj Yes, that would work. I didn't want to introduce the extra node. But doing it that way (with the extra node) actually makes it almost identical to how I handle Skeletal animations that cause movement, so that sounds like a good idea.

@chillywillysoft chillywillysoft deleted the node-animation-local-translation branch May 31, 2023 02:32
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.

2 participants