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

[Vanilla Fix] Prevent AI allies of human player from losing intelligence early in the game #1510

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

TaranDahl
Copy link
Contributor

Can't say that I understand AI anger-related codes very well, but this fix does solve the stupidity of the player's AI teammates in early game.
And after a period of use, no obvious side effects have been found.

Copy link

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@DeathFishAtEase
Copy link
Collaborator

Regarding AI anger-related, perhaps you can obtain some information by reading this post and trying to ask handama.

@Metadorius
Copy link
Member

Is there a point in a toggle? I don't see a scenario when the bugged behavior is preferred.

@DeathFishAtEase
Copy link
Collaborator

DeathFishAtEase commented Jan 27, 2025

If extensive and diverse testing consistently reveals no side effects, perhaps a manually controlled switch is not necessary.
Perhaps it's just because the internal mechanisms aren't fully understood yet, so it can't be certain that there are no problems at present?
Theoretically, features like [AI]►Paranoid= (boolean) have not worked since before YR.
Have you tried declaring war on the AI during an online game to change the AI's list of allies and see if there are any abnormalities in the AI?
Can it still normally handle allies who become enemies during the game?

@TaranDahl
Copy link
Contributor Author

TaranDahl commented Jan 27, 2025

If extensive and diverse testing consistently reveals no side effects, perhaps a manually controlled switch is not necessary. Perhaps it's just because the internal mechanisms aren't fully understood yet, so it can't be certain that there are no problems at present? Theoretically, features like [AI]►Paranoid= (boolean) have not worked since before YR. Have you tried declaring war on the AI during an online game to change the AI's list of allies and see if there are any abnormalities in the AI? Can it still normally handle allies who become enemies during the game?-----------谷歌翻译-----------如果广泛且多样化的测试始终显示没有副作用,则也许不需要手动控制开关。或许只是因为内部机制还没有完全搞清楚,所以目前还不能确定没有问题?理论上,像 [AI]►Paranoid=(布尔值)这样的功能自去年以来就不起作用了。你是否尝试过在网络游戏中向AI宣战,改变AI的盟友列表,看看AI是否有异常?它还能正常处理游戏中变成敌人的盟友吗?

May test them later

@FS-21
Copy link
Contributor

FS-21 commented Jan 27, 2025

Isn't much better if instead of this PR by default ALL houses anger values are initialized as 1 when the map is loaded? AI will ignore friendly anger values and since they have enemies with anger value > 0 the AI will work as expected...

@TaranDahl
Copy link
Contributor Author

Perhaps it's just because the internal mechanisms aren't fully understood yet, so it can't be certain that there are no problems at present?

Indeed. In fact, I found the hook position by intuition.

@MortonPL
Copy link
Member

Is there a point in a toggle? I don't see a scenario when the bugged behavior is preferred.

It might affect single player missions with AI companions that were designed with AI "passiveness" in mind. Perhaps in the end it won't be a problem, but I think this should be at least tested/consulted with modders.

@MortonPL
Copy link
Member

If it turns out to not a be a problem or that mapmakers are willing to account for this, we can drop the toggle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants