timhul / ClassicSim

An event-driven simulation tool written in C++ for World of Warcraft Classic.
Other
116 stars 70 forks source link

Enhancement Shaman rotations & Swing timer #198

Open Mizzajl opened 4 years ago

Mizzajl commented 4 years ago

Lightning Bolt resets the Attack swing timer after it finishes casting, even if you would have had enough time to finish casting in between attacks, so it should be removed from the "Stormstrike" rotation.

All non-instant shaman spells also resets the attack swing timer. Instant shaman spells does not reset the attack swing timer. Getting a Windfury Procc does however reset it, so Stormstrike would only reset the attack swing timer if it proccs Windfury. The windfury procc have a 1.5 second cooldown.

personally i try to cast stormstrike directly after an auto-swing if that swing didn't procc windfury. would be nice to be able to compare that to just casting stormstrike on cd.

also i would like to be able to have shocks in my rotation (mostly using frost shock in raids since it doesn't take up a debuff slot on bosses or cause a ton of threat like earth shock)

and dropping Searing Totem on singletarget and for aoe Fire Nova Totem on cd otherwise Magma Totem.

timhul commented 4 years ago

Thanks for the report, all of your notes sound reasonable to me.