BryanSWeber / CUNYAIModule

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

On frame refactor #240

Closed BryanSWeber closed 2 years ago

BryanSWeber commented 2 years ago

This patch is part of a larger refactor.

1) It removes combat simulators from scattered calls to a separate class that acts as a wrapper for FAP.
They are called and then move out of scope.

2) It improves 2 diagnostic functions: a) DiagnosticBar - which draws a segmented bar (like HP) of arbitrary color under a particular unit. b) DiagnosticClock which tells me if a particular function or code segment runs over 15ms.

3) I pull several lines of code out of onFrame() and shift towards McRave's style of each class having an independent thisClass.onFrame() call.

It contains 1 play style change - altering the retreating patterns to a shorter, more concise definition. It reduces performance against some bots.