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 AttackOrFleeFuzzy improvements #14468

Merged
merged 4 commits into from Dec 28, 2017

Conversation

Projects
None yet
4 participants
@reaperrr
Contributor

reaperrr commented Dec 2, 2017

I needed a break from the other stuff I'm working on, so I gave this a look.

The idea behind the string shortenings is the assumption that the FuzzyLibrary must be doing some kind of string comparison, so shorter strings should speed things up a little (likely insignificant, but who knows...).

The other commits should, at least in theory, make the AI behave a little smarter when making attack-or-flee decisions.

@reaperrr

This comment has been minimized.

Show comment
Hide comment
@reaperrr

reaperrr Dec 26, 2017

Contributor

Minor update to 2nd commit, pulled the .Clamp out of the foreach and added a comment detailing the reasons for the clamp values.

Contributor

reaperrr commented Dec 26, 2017

Minor update to 2nd commit, pulled the .Clamp out of the foreach and added a comment detailing the reasons for the clamp values.

@GraionDilach

👍

reaperrr added some commits Dec 2, 2017

Make sure AttackOrFlee returns Attack when it makes sense
Not listing enemy "NearDead" and own relative atk power "Strong" here looked wrong, and might've resulted in no decision being made.
@reaperrr

This comment has been minimized.

Show comment
Hide comment
@reaperrr

reaperrr Dec 27, 2017

Contributor

Updated.

Contributor

reaperrr commented Dec 27, 2017

Updated.

@pchote

pchote approved these changes Dec 28, 2017

Code changes look sensible and I didn't notice any obvious regressions ingame.

@pchote pchote merged commit 3d4095c into OpenRA:bleed Dec 28, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@reaperrr reaperrr deleted the reaperrr:atk-or-flee-improvements branch Feb 22, 2018

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