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

Substituting own animations is not versatile enough #38

Open
SebastianMeinberger opened this issue Jan 23, 2022 · 0 comments · May be fixed by #39
Open

Substituting own animations is not versatile enough #38

SebastianMeinberger opened this issue Jan 23, 2022 · 0 comments · May be fixed by #39
Assignees
Labels
enhancement New feature or request

Comments

@SebastianMeinberger
Copy link

SebastianMeinberger commented Jan 23, 2022

When #37 is merged, it is possible to use own animations and animators, but only the Speed parameter is updated by the AnimationUpdater. For example, when a user decides to use the animatior from the Standard Assets, most of the blend tree is not utelised, since it doesn't recive updates about angular momentum and similar parameters.

The animation updater should provide animators with as much meaningfull parameters as possible so users have more freedome when using own animations.

The names of theses parameters should be easily remapable in the inspector to account for different names of parameters with same funtionality (e.g. in the starter assets the Speed Parameter is called Speed and in the Standart assets its called Forward). They should also be clamped between 0 and 1 for easier handling

@SebastianMeinberger SebastianMeinberger added the enhancement New feature or request label Jan 23, 2022
@SebastianMeinberger SebastianMeinberger self-assigned this Jan 23, 2022
@SebastianMeinberger SebastianMeinberger linked a pull request Jan 24, 2022 that will close this issue
@SebastianMeinberger SebastianMeinberger linked a pull request Jan 24, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant