[TF2] tf_bots work up to 100 players
Modifies tf_bots so they work all the way up to 100 players without failing to spawn in or causing significant class imbalances.
Implementation Details:
-
CTFBot::m_didReselectClass- Added a Getter so that
CTFBotMainAction::Updatecan avoid unnecessarily triggering class reevaluations. - Moved
falseassignment toCTFBot::Event_Killedto prevent an infinite loop of class reevalulation.
- Added a Getter so that
- CVars:
-
tf_bot_reevaluate_class_in_spawnroom: NowFCVAR_NONE(wasFCVAR_CHEATS). -
tf_bot_spawn_use_preset_roster: NowFCVAR_NONE(wasFCVAR_CHEATS). Now defaults to 0 (was 1).
-
- Preset Roster (
tf_bot_spawn_use_preset_roster 1):- If team size > 12, the preset roster will be reused for bots 13-24, 25-36, etc.
- Prevents bots from failing to spawn, but may still result in imbalanced class counts. Still only intended for 12v12.
- Dynamic roster (
tf_bot_spawn_use_preset_roster 0):- Now the default, as it is more dynamic and responsive to gamestate.
- Class limits specified in rosters scale according to team size. Roster definitions assume team size of 12. Team sizes under 12 unaffected.
- Team composition levels out with larger team sizes (prevents massive class stacks).
- ~~Added more bot names so there's a pool of at least 100 to choose from.~~ Now in a separate PR: https://github.com/ValveSoftware/source-sdk-2013/pull/892
I really like this pull request as it fixes and changes things that I have had issues with for years. However the extra bot names here kinda feel really ¿forced? They seem like something better fit for a different PR.
added to that, it's a bad idea to reference the infamous coconut myth as it might just re-ignite the whole discussion again
Additional bot names and relevant discussion have been moved to their own PR: https://github.com/ValveSoftware/source-sdk-2013/pull/892