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

Improvements to brainEngage #421

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nk3nny
Copy link
Owner

@nk3nny nk3nny commented Nov 10, 2024

Improve brainEngage by taking advantage of getUnitState
Improve brainEngage reimplements behaviour not triggering when stealthed*
Fixes doAssault where a unit would sometimes get stuck trying to check a position that the group believed the enemy is-- where it clearly could not be. (such as in the sky)**

*Not triggering brainEngage when a unit is sneaking means that the AI become more predictable for player led groups, and unit set to ambush actually remain-ambushing. Those set to sneak will not get mixed commands from their dangerAI

**It does this by simply forgetting the enemy, which is somewhat controversial but works very well in the context of group memory (i.e., if it is near a building, it will still have likely places to check).

Makes me think we really should make our own getHideFrom function.

Improve brainEngage by taking advantage of getUnitState
Improve brainEngage reimplements behaviour not triggering when stealthed*
Fixes doAssault where a unit would sometimes get stuck trying to check a position that the group believed the enemy is-- where it clearly could not be. (such as in the sky)**

*Not triggering brainEngage when a unit is sneaking means that the AI become more predictable for player led groups, and unit set to ambush actually remain-ambushing.  Those set to sneak will not get mixed commands from their dangerAI

**It does this by simply forgetting the enemy, which is somewhat controversial but works very well in the context of group memory (i.e., if it is near a building, it will still have likely places to check).

Makes me think we really should make our own getHideFrom function.
@nk3nny
Copy link
Owner Author

nk3nny commented Nov 10, 2024

Yeah. Linter will yell

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.

1 participant