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

Added a faint animation for the rock critter #437

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mitchtheriault
Copy link

@mitchtheriault mitchtheriault commented May 21, 2021

Hi there! In the Animations card I saw that you need a "faint" animation for the rock critter, so I went ahead and made one.
ezgif-4-276944dad4ca

@amel-unity amel-unity added the art Art or music assets, or tweaks label May 21, 2021
@amel-unity amel-unity self-assigned this May 21, 2021
@amel-unity
Copy link
Contributor

Hi @mitchtheriault, thank you for opening the PR for the faint animation of the rock critter.
Which model did you use to animate? After testing the PR locally, I noticed that the animation was not played from the Animator Controller and after checking in Blender you can notice that the rig is not the same as the fbxs we already have in the project (on the left the Idle fbx from the project, on the right the faint fbx from the PR).

image (4)

You need to take one of the .fbxs already in the project, so the rig matches with the one we already have for the different animations.

@amel-unity amel-unity added the ? Action required: missing information, question to answer, or CLA not signed label Jun 1, 2021
@amel-unity
Copy link
Contributor

@mitchtheriault we have some new suggestions on the rock critter thread which I just replied to including ideas related to the faint animation and I wanted to know what are your thoughts on it :) Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
? Action required: missing information, question to answer, or CLA not signed art Art or music assets, or tweaks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants