-
Notifications
You must be signed in to change notification settings - Fork 286
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
first crack at bot minefield avoidance #2898
Conversation
This pull request introduces 1 alert when merging 4ad1b45 into 588f8b0 - view on LGTM.com new alerts:
|
This needs a CI fix, as it is causing a test to NPE |
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.
Looks good in general, just a few things first
if (s == 0) { | ||
return super.compare(first, second); | ||
} else { | ||
return s; |
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.
Ternary return candidate
This pull request introduces 1 alert when merging d4eff68 into 588f8b0 - view on LGTM.com new alerts:
|
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.
Just the one missing bracket set
|
||
double hazardAccumulator = 0; | ||
// hovercraft and WIGEs grinding along the ground detonate minefields on a 12 | ||
boolean hoverMovement = movingUnit.getMovementMode() == EntityMovementMode.HOVER || |
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.
Brackets
This pull request introduces 1 alert when merging 2397621 into 588f8b0 - view on LGTM.com new alerts:
|
This is the first attempt at having the bot try to avoid minefields. The basic gist is that the bot will generally prefer not to step on mines if she thinks she will detonate them. The bot client only has access to minefields it knows about.
More detailed breakdown: