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

AI will no longer cheat upgrades #1277

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

Conversation

Nivaturimika
Copy link
Contributor

AI having an advantage over players is unacceptable
Additionally, we make AI choose the BEST available units
Not objectively the best, but the ones with most attack
So guards and such can be built if needed
And airplanes and armored units too! which makes stomping on the AI 100 easy because THEY NEVER BUILD THEM HOLY SHIT

@schombert
Copy link
Owner

I am not convinced your picks for "best" are actually best. Also, if you want the AI to upgrade, it has to be able to slowly replace units (i.e intentionally disband). Otherwise out of date units will just clog up its available slots. But it can't disband all at once or it will leave itself defenseless when an upgrade is available.

@Nivaturimika
Copy link
Contributor Author

I am not convinced your picks for "best" are actually best. Also, if you want the AI to upgrade, it has to be able to slowly replace units (i.e intentionally disband). Otherwise out of date units will just clog up its available slots. But it can't disband all at once or it will leave itself defenseless when an upgrade is available.

I made some changes
a) Restored free upgrades (one day we should add a mechanism to upgrade units for the player too?)
b) Best pickings now account attack, defense, siege, and speed
c) Limited upgrades to units according to regiment source (i.e you cant upgrade a ukrainian to a guard as russia - that would INDEED be cheating)

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.

None yet

2 participants