search

BryanSWeber / CUNYAIModule

CUNYBot, an AI that plays complete games of Starcraft.
MIT License
6 stars 3 forks source link

Sometimes closest unit lacks valid position? #177

Open BryanSWeber opened 5 years ago

BryanSWeber commented 5 years ago

See here: 5b1af44cc623aa2ba473c1c4a2be3ca7269543ca

BryanSWeber commented 5 years ago

Might lead to broad combat problems, combatmanager getclosest checks for validity of the position.

BryanSWeber commented 4 years ago

Found primary problem:

Double was implicitly converted to int. Resolved, but I haven't found the exact commit. In retrospect, I should be better about placing these. The commit is roughly around: 977cbbbb14718d86bb02bd98d4aa8144cfd50c07

BryanSWeber commented 4 years ago

Issue still persists but is greatly reduced in frequency.