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

Robot animation fixes #3

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sticks-stuff
Copy link
Contributor

This change makes MvM robots fallback to human animations if the robot lacks a human animation. This is done by adding $includemodel "player/<class>_animations.mdl" under $includemodel "bots/<class>/bot_<class>_animations.mdl" in each bots QC file. Note that the engineer bot model already does this.

@agrastiOs
Copy link
Contributor

Great idea. This would allow for the players of servers with robot player models to use taunts with more than just the Engineer.

@mastercoms
Copy link
Member

Please fix conflicts.

@agrastiOs
Copy link
Contributor

I'd suggest removing the Sentry Buster's and the boss Bots model changes and keeping just the regular bots in, that'd fix the conflicts and also would clean up the PR, as I can't see an use for the Buster or giant bots to have this.

@sticks-stuff
Copy link
Contributor Author

Conflicts fixed

@agrastiOs
Copy link
Contributor

Seems good to me. The giants having human animations might have potential uses, I guess.

@sticks-stuff
Copy link
Contributor Author

Also can this repo be opted in hacktoberfest as well?

@agrastiOs
Copy link
Contributor

agrastiOs commented Oct 9, 2020

@sigsegv-mvm would need to sign the CLA for this.

As far as I'm aware he doesn't want to do that, but these could be potential fixes for the Ultimate TF2 Visual Fix Pack, which will have all its updates ported to TC2. That could be a good workaround, we would make the fixes in-house.

@sticks-stuff
Copy link
Contributor Author

Doaes he need to sign the CLA for this though? It's a one line change to each model, not really his.

@micwoj92
Copy link

micwoj92 commented Oct 9, 2020

Doaes he need to sign the CLA for this though? It's a one line change to each model, not really his.

I make 1 letter change in words I need to sign, rules are rules!

@agrastiOs
Copy link
Contributor

agrastiOs commented Oct 9, 2020

Well, he did a two-line change to each boss bot model, and he used the particular compile software that he used.

I take your point, but it's better to still follow the rules. Better to have everything as legally unambiguous as possible for Valve to take this.

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.

4 participants