search

bsaffel / benchmarker

macro trainer | build optimizer | practice tool
10 stars 0 forks source link

Upgrades displaying incorrectly upon cycle. #312

Closed bsaffel closed 9 years ago

bsaffel commented 9 years ago

screenshot2014-10-20 00_18_21

The active step should be melee weapons, not ground armors. Also, prior to completing ground armors, melee weapons did not display correctly.

bsaffel commented 9 years ago

Probably related, after a cancellation of a hatchery to build a lair, the cycled objective contains overlord speed twice.

screenshot2014-10-20 00_21_04

bsaffel commented 9 years ago

Adding a full debug SS since I have it: screenshot2014-10-20 00_22_15

bsaffel commented 9 years ago

This SS is probably more relevant screenshot2014-10-20 00_23_15

bsaffel commented 9 years ago

This also generally only happens when cycling.

bsaffel commented 9 years ago

Looks like the second time the lair is built as the correct step, the build cycles again when it shouldn't. The overlord speed had already been added, and based on the position of the cancelled objective, no cycle should have been performed.

Need to add a check that will prevent a new step from being added in when it isn't necessary, or enhance the check to prevent cycle from being called.