MAJOR THUMBS UP for
[email protected] (Stormchaser ?) and the rest of the CLU team with the Affliction lock CC rewrite!!!
+10k dps damage done on the training dummies versus the previous version (same toon, same settings)
Furthermore I've seen hardly any doublecasts and the mana/shard consumtion is reasonable (dummies & raiding tests).
All in all, for sure on par with an average Affliction lock which is a big compliment as this class/spec isn't that simple
I do want to offer the following remarks, stuff I noticed & possible improvements:
Affliction CC generic:
- Calling out the felhunter (for GoS) gets double casted out-of-combat
- The code is more "logical/modularized" which appeals to my rational sense but on the other hand the spaghetti monster
is looming around the corner due to the use of many local methods, exotic (in a CC sense) Treesharp constructions and
varying ways of doing the same thing across the board. This is NOT a complaint!!! By all means, please continue to do
your thing

I rather have a good CC with meatballs then a clean plate bad CC. I just have to be more careful when
svn merging my own edits into it
- HandleAoeCombat() might deserve a second look regarding the fact it handles the different AOE techniques with a
PrioritySelector in which HandleSoulSwapAoE()'s predicate will virtually fire in all AOE cases. I suggest using a more
carefull selection here as HandleSoulSwapAoE() is ideal for fights like MSV/StoneGuard but a PITA when confronted
with a bunch of low life mobs. The point being that one would use HandleSeedofCorruptionAoE() on a bunch of mobs
to generate Soul Shards through Corruption ticks and then use HandleSoulSwapAoE() to take on multiple individual
or bigger ones. This would save on the Drain Soul activity involved in handling >4 mobs and provide a substantial
dps increase.
- A major improvement might be to refresh non-intel procced dots @50%. Unfortunately there are currently no methods
to track procced dots across multiple targets which could get you into mana problems if refreshing all dots @50%.
(one might create a DotManager class fed by attached combatlog events though... hint hint)
- Healthstone use is not implemented within the Affliction CC (although a use function exists in Common.cs). There
should also be a pre-combat CreateHealthstone() function though.
- Talent/CD/Glyph management can still be further optimized. Archie's Vengeance can be an awesome dps increase
when triggered at the right moment. This would require an adaptation of the TimeToLive() healer method (hint...)
for the localplayer in order to know the rate of incoming dmg. This could also be used as Tier1 triggers...
- Fel Flame should not be used as the fall-through filler as it has a predicate of "moving", this will cause the CC
to stop mid-combat in some cases, deplete your life through multiple "Fel Flame"/"Life Tap" executions or cause
your toon to become a jerky boddypopper if not facing the mob

Please add some kind of "allwayssuccess()" at
the end of the rotation.
CLU generic:
- Facing issues on single target Agony/Corruption & UA happen sketchly, there shouldn't be... A lock can
cast them without facing. Haven't started to look for them in the code, they don't seem to happen all the time (?)
These are unfortunate because of the lock channeling & facing mechanic which needs to face a mob to start
the channel but after that no facing is needed. However at the end of the channel the player might be not be
facing the mob anymore while he/she is not turning/moving during the channel as this might break it.
- ^^ KJCunning without cast/move penalty might be coming in 5.1 (PTR...), which makes the above a bigger
problem. Changing all !Me.isMoving() to !Me.isMoving() || hasGlyph(KJC) would be supreme
- Transformation detection (stupid name LOL) The bot/CLU etc. doesn't detect when the player gets transformed.
"Reshaping Life" in HoF comes to mind which will provide prime "vote kick" material in LFR

I was lucky to play
with keybindings ON... Some others weren't hehehe...
All of the above are humbly offered as input for improvement and NOT as a complaint AT ANY RATE!!! You guys
ROCK as far as I'm concerned!!!

Let me know if you need more details or further input, I do have some
programming skills although I'm not a codemeister in the general sense.
Keep up the good work,
Good hunting,
Dleihstaem.