-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 citizen focus change #11908
Merged
Merged
AI citizen focus change #11908
Changes from all commits
Commits
Show all changes
71 commits
Select commit
Hold shift + click to select a range
7ed4c0d
AI behaviour changes
EmperorPinguin 47184b0
Merge branch 'master' of https://github.com/EmperorPinguin/Unciv
EmperorPinguin e856a76
Merge branch 'master' of https://github.com/EmperorPinguin/Unciv
EmperorPinguin b571e76
Update Automation.kt
EmperorPinguin cee0287
Update Automation.kt
EmperorPinguin 16ae6d9
Update Automation.kt
EmperorPinguin a769a52
Update Automation.kt
EmperorPinguin 9c8f2e2
Update Automation.kt
EmperorPinguin 777b828
Update ConstructionAutomation.kt
EmperorPinguin 2a4f9b0
Update Automation.kt
EmperorPinguin 33b9fc7
Reverting some changes
EmperorPinguin e311e17
Changes
EmperorPinguin cade007
revert changes
EmperorPinguin 318c396
revert changes
EmperorPinguin 012afb5
revert changes
EmperorPinguin f4d02c8
revert changes
EmperorPinguin cae12b6
Update CityLocationTileRanker.kt
EmperorPinguin 9040bab
Citizen assignment for stat conversion
EmperorPinguin a33d7f4
Merge branch 'yairm210:master' into master
EmperorPinguin e53f887
Merge branch 'yairm210:master' into master
EmperorPinguin 3f540a8
Update CityLocationTileRanker.kt
EmperorPinguin 0e5db36
Merge branch 'yairm210:master' into master
EmperorPinguin 43cf8a3
Merge branch 'yairm210:master' into master
EmperorPinguin 85743fb
Merge branch 'yairm210:master' into master
EmperorPinguin a1f5905
Reduce AI settling
EmperorPinguin d061b67
Merge branch 'yairm210:master' into master
EmperorPinguin 4257a0f
Avoid AI building units when in negative Supply
EmperorPinguin 44c8a90
Merge branch 'yairm210:master' into master
EmperorPinguin 5f42454
Update CityLocationTileRanker.kt
EmperorPinguin 6b49cdc
Merge branch 'master' of https://github.com/EmperorPinguin/Unciv
EmperorPinguin 383a083
Update CityLocationTileRanker.kt
EmperorPinguin 31e3c44
Update CityLocationTileRanker.kt
EmperorPinguin cd7c911
Merge branch 'yairm210:master' into master
EmperorPinguin 958b646
Update ConstructionAutomation.kt
EmperorPinguin 5f91c79
Merge branch 'yairm210:master' into master
EmperorPinguin 809b0fc
Update build.gradle.kts
EmperorPinguin f2573d8
Update gradle-wrapper.properties
EmperorPinguin 9bef165
Update CityLocationTileRanker.kt
EmperorPinguin 3239649
Update CityLocationTileRanker.kt
EmperorPinguin 6f7c7c8
Merge branch 'yairm210:master' into master
EmperorPinguin 301ab4e
Update ConstructionAutomation.kt
EmperorPinguin f8f364e
Merge branch 'master' of https://github.com/EmperorPinguin/Unciv
EmperorPinguin df80b10
Update CityLocationTileRanker.kt
EmperorPinguin ef4f189
Merge branch 'master' of https://github.com/EmperorPinguin/Unciv
EmperorPinguin 0fda42b
Merge branch 'yairm210:master' into master
EmperorPinguin 55e1f7c
Merge branch 'yairm210:master' into master
EmperorPinguin bc6172b
Merge branch 'yairm210:master' into master
EmperorPinguin 213c896
AI changes for humans
EmperorPinguin 5f4e093
Fix puppet focus
EmperorPinguin 61d8299
Merge branch 'yairm210:master' into master
EmperorPinguin 7ff9018
Merge branch 'yairm210:master' into master
EmperorPinguin 428193b
Merge branch 'yairm210:master' into master
EmperorPinguin 001269a
Update Automation.kt
EmperorPinguin 344dd81
Puppet focus
EmperorPinguin bf0a8e7
Update Automation.kt
EmperorPinguin 545f8c2
Update Automation.kt
EmperorPinguin 395ac8f
Update Automation.kt
EmperorPinguin 96acd16
Update Automation.kt
EmperorPinguin 9e73243
Update Automation.kt
EmperorPinguin 5de758d
Merge branch 'yairm210:master' into master
EmperorPinguin 6fc1b0c
Update Automation.kt
EmperorPinguin 53057d9
Merge branch 'master' of https://github.com/EmperorPinguin/Unciv
EmperorPinguin d5e270b
Update Automation.kt
EmperorPinguin c34b62b
Update Automation.kt
EmperorPinguin 611378b
Update Automation.kt
EmperorPinguin 1e4d88b
Update Stats.kt
EmperorPinguin 5863788
Merge branch 'yairm210:master' into master
EmperorPinguin c896436
Update CityTurnManager.kt
EmperorPinguin 8bb6b37
Remove specialist science modifier
EmperorPinguin 0d3d37b
Merge branch 'yairm210:master' into master
EmperorPinguin a73bf52
Merge branch 'master' of https://github.com/EmperorPinguin/Unciv
EmperorPinguin File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The AI generally has a problem with obtaining gold. This is partly because there aren't many gold opportunities early on. But reducing the value of gold wouldn't help.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not just the AI who uses AI citizen assignment, but also many humans do it. In the early game, the most efficient use of gold is via unit upgrades, which is a gold:production conversion of 2:1. Late in the game, with stock exchanges built, the conversion of base gold to base production is more like 2*(1+0.1+0.1+0.15+0.2) : 1*(1+0.25+0.25+0.25) = 1.77:1, or even 1.19:1 with Professional Army/Pentagon. It makes sense to scale the value of gold with era, but if I'm not mistaken the AI already takes modifiers from buildings into account.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The reason AI loses as much gold as it does, got more to do with how it picks city constructions, and by building roads without connecting cities. Improving their worker management might help quite a bit, but I'm not sure how to do so.