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

Give bots ability to auto-replace harvesters #15991

Merged
merged 6 commits into from Jan 6, 2019

Conversation

Projects
None yet
3 participants
@reaperrr
Copy link
Contributor

reaperrr commented Jan 3, 2019

...when their number drops below the number of refineries.

Depends on #15982.

Closes #15962.

reaperrr added some commits Jan 3, 2019

Make bots deploy MCV on first tick
And use a boolean instead of counting ticks.
Increase default scan interval for idle bot harvesters
Every 2 seconds (at default speed) should be enough.

@reaperrr reaperrr force-pushed the reaperrr:bot-harv-replace branch from 9a0867e to d9e37e4 Jan 6, 2019

reaperrr added some commits Jan 3, 2019

Add plumbing for bots auto-replacing harvesters
If their number drops below refinery count.

@reaperrr reaperrr force-pushed the reaperrr:bot-harv-replace branch from d9e37e4 to 3f5d635 Jan 6, 2019

reaperrr added some commits Jan 6, 2019

Remove unused BuildUnit overload from UnitBuilderBotModule
Unused and didn't check things like Buildable, so better just remove it.
Add BotDebug message for external unit build requests
For easier bot debugging of things like MCV- and harvester replacement.
@reaperrr

This comment has been minimized.

Copy link
Contributor

reaperrr commented Jan 6, 2019

Updated. Moved the 'should build' decision making inline into BotTick, and added a BotDebug message for external unit build requests in the unit builder module.

Since the pt hasn't been tagged yet, maybe we can still sneak this in?

I tested the latest changes and everything seems to work as intended.

@abcdefg30 abcdefg30 merged commit b05d246 into OpenRA:bleed Jan 6, 2019

2 checks passed

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

This comment has been minimized.

Copy link
Member

abcdefg30 commented Jan 6, 2019

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