mouseas / steamSummerMinigame

Steam Summer Sale 2015 - Auto-play Optimizer
MIT License
136 stars 189 forks source link

Added the use of cooldowns #66

Closed sktan closed 9 years ago

sktan commented 9 years ago

Implemented feature to call cooldown then wait 300ms before using long-cooldown abilities. If cooldown isn't ready to be used, then just use the ability without waiting.

mouseas commented 9 years ago

Have you tested it? My experience is that you have to wait until the server responds, about 1 second, before triggering the next ability.

sktan commented 9 years ago

Tested it and will need to be fixed to wait for the server to respond. Would changing the timer to 1.1-1.5 seconds be too much you reckon?

mouseas commented 9 years ago

I would say 5 seconds is too much (since it expires by then), 1.5 seconds sounds like a good idea.

sktan commented 9 years ago

Updated the timer to 1.5 seconds Looks as though it still needs some modification in order for the cooldowns to work.

mouseas commented 9 years ago

Let me know when you've got it working, tested, and it's ready.

winneon commented 9 years ago

Tell me when you have fixed the issues with your code, and then I'll fix the merge conflicts that were introduced with the filename change.

sktan commented 9 years ago

Not 100% tested, but initial test seems to have done it correctly.

Only calls cooldown if more than 60 seconds on cooldown timer and no cooldown ability currently in lane.

sktan commented 9 years ago

Further tests seem to work.