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

[WIP] Add animation motion-matching #32408

Closed
wants to merge 1 commit into from

Conversation

fire
Copy link
Member

@fire fire commented Sep 28, 2019

Improved UI and better matching. Frame choosing behavior working

TODO: Correct the cost and trajectory predictions

Try to get #29892 merged. Attempt to fix all the cicd errors.

@akien-mga
Copy link
Member

akien-mga commented Sep 28, 2019

Wouldn't it be worth asking @Aa20475 to do changes in #29892 instead of forking the PR?

Note that you can also do a PR on #29892's branch to propose changes to add there. But it should indeed be squashed beforehand.

@fire fire force-pushed the motion-matching branch 6 times, most recently from 9578813 to 5f9a999 Compare September 28, 2019 07:04
@fire
Copy link
Member Author

fire commented Sep 28, 2019

My goal is to keep it current so the system doesn't rot too much.

@Aa20475 Feel free to comment or take control.

Noticed that cicd is failing on unsigned integers versus integer comparisons.

@fire fire force-pushed the motion-matching branch 3 times, most recently from dc85e7a to 6f88cad Compare September 28, 2019 13:25
@fire fire changed the title Add animation motion-matching [WIP] Add animation motion-matching Sep 30, 2019
@akien-mga akien-mga added this to the 4.0 milestone Sep 30, 2019
@aaronfranke aaronfranke marked this pull request as draft April 9, 2020 00:33
@fire fire force-pushed the motion-matching branch 6 times, most recently from 261f671 to 03de637 Compare November 14, 2020 16:29
Improved UI and better matching. Frame choosing behavior working

TODO: Correct the cost and trajectory predictions
@fire
Copy link
Member Author

fire commented Nov 14, 2020

Resolved CICD errors, but lacking test project and design documentation.

@fire
Copy link
Member Author

fire commented Dec 23, 2020

Closing due to the inability to use this in a game project. This animation motion matching pr is salvageable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants