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.

Sign up for GitHub

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

Jump to bottom

Remove idle AI from MP computer player selection menu #2095

Closed
mattsc opened this issue Oct 10, 2017 · 0 comments
Closed

Remove idle AI from MP computer player selection menu #2095

mattsc opened this issue Oct 10, 2017 · 0 comments
Assignees
@CelticMinstrel @mattsc
Labels
AI Issues with the AI engine, including micro AIs. MP Issues with multiplayer support or bundled multiplayer content. Regression Issues that were not present in previous releases.

Comments

@mattsc
Copy link
Member

mattsc commented Oct 10, 2017

As a result of fixing issue #1955, the idle AI is now available in the MP computer player selection menu even when the game was not started in debug mode. As discussed on irc, this is undesirable. The preferred fix seems to be adding a hidden= attribute to the [ai] tag.
@mattsc mattsc added AI Issues with the AI engine, including micro AIs. MP Issues with multiplayer support or bundled multiplayer content. Regression Issues that were not present in previous releases. labels Oct 10, 2017
GregoryLundberg pushed a commit that referenced this issue Nov 30, 2017
@Vultraz @GregoryLundberg
Hide AIs with hidden=yes from list of available AI options (fixes #2095)
c15c428 
This is enough to hide them as AI options in the UI, but I don't know if other parts of the AI
engine need to know about this key.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CelticMinstrel CelticMinstrel

@mattsc mattsc

Labels
AI Issues with the AI engine, including micro AIs. MP Issues with multiplayer support or bundled multiplayer content. Regression Issues that were not present in previous releases.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@CelticMinstrel @mattsc