-
Notifications
You must be signed in to change notification settings - Fork 2
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
Animation retarget: add support for root-bone translations #1
Comments
The main difficulty I see is that different models place the root bone differently. For instance, Jaime's root bone is between his feet, while Sinbad's is in his belly. Also, translation introduces the issue of models built at different scales (meter units versus centimeter units, for instance.) For these reasons and others, I think root translation should be handled separately from rotations. The Maud editor includes support for translating models so that their feet touch the ground without sliding. I believe the relevant code could be added to Wes. Would that serve your need? |
I calculate target track translation by finding delta translation from the source track ( delta translation = source track translation - source joint bind translation) and add it to target joint bind translation. This works fine for me but not sure if it will fit into all cases. Here is simplified version of my anim retarget script:
|
Is there anything that should be added to the Wes library to support your solution? |
Nothing needed ATM, I need to thoroughly test my solution for different scenarios. I will let you know the outcome. |
@Ali-RS Any news? |
Really sorry, no progress on this yet. I do not think I can come back to this anytime soon. Please feel free to close this if you want. |
Tried to retarget an AnimClip from one model to another, seems it only retargets bone rotations (makes an in-place animation) and does not have support for root motion export.
While in-place animation might be what one needs most of the time, but yet sometimes we might need to have the root motion also. It will be nice to add support for this as well.
The text was updated successfully, but these errors were encountered: