simulationcraft / simc

Simulationcraft engine/GUI
GNU General Public License v3.0
1.4k stars 694 forks source link

Arms single target action priority list #2799

Closed dfherr closed 8 years ago

dfherr commented 8 years ago

the arms single target action priority list seems subpar.

Colossus Smash should be a priority other MS, when Shattered Defenses is not up.

During Deadly Calm Slam should always be used, when CS/MS/Execute are not available, and not only when there is enough rage or a CS debuff.

Focused Rage should not be used, when you have less rage than required for a MS left after using it. (This is only a minor dps increase)

Here is an improved version of the single target rotation, socring ~3,5% / ~11k dps higher than the current profile:

actions.single+=colossus_smash,if=buff.shattered_defenses.down&buff.precise_strikes.down
actions.single+=/mortal_strike
actions.single+=/execute,if=buff.stone_heart.react|buff.battle_cry_deadly_calm.up
actions.single+=/warbreaker,if=buff.shattered_defenses.down
actions.single+=/execute,if=debuff.colossus_smash.up&rage.deficit<60
actions.single+=/focused_rage,if=(buff.focused_rage.stack<3&((talent.dauntless.enabled&rage>27)|(!talent.dauntless.enabled&rage>34)))|buff.battle_cry_deadly_calm.up
actions.single+=/slam,if=target.health.pct>20&!talent.fervor_of_battle.enabled&(debuff.colossus_smash.up|rage.deficit<40|buff.battle_cry_deadly_calm.up)&(!talent.focused_rage.enabled|buff.battle_cry_deadly_calm.up)
actions.single+=/whirlwind,if=target.health.pct>20&talent.fervor_of_battle.enabled&(debuff.colossus_smash.up|rage.deficit<50)&(!talent.focused_rage.enabled|buff.battle_cry_deadly_calm.up|buff.cleave.up)
actions.single+=/rend,if=remains<=duration*0.3
actions.single+=/whirlwind,if=target.health.pct>20&talent.fervor_of_battle.enabled&(!talent.focused_rage.enabled|rage>100|buff.focused_rage.stack=3)
actions.single+=/slam,if=target.health.pct>20&!talent.fervor_of_battle.enabled&(!talent.focused_rage.enabled|rage>100|buff.focused_rage.stack=3)
actions.single+=/execute,if=equipped.137060
actions.single+=/slam,if=equipped.137060
actions.single+=/shockwave
actions.single+=/storm_bolt
actions.single+=/bladestorm,if=raid_event.adds.in>90|!raid_event.adds.exists|spell_targets.bladestorm_mh>desired_targets

Additionally the haste enchants on rings are quite a bit better than the mastery enchants at the current stats of the gear improving dps by an additional 2k. (although this could confuse people looking at the gear)

mrdmnd commented 8 years ago

very nice. someone will definitely incorporate these changes soon.

On Sat, Sep 3, 2016, 1:06 PM Dennis-Florian Herr notifications@github.com wrote:

the arms single target action priority list seems subpar.

Colossus Smash should be a priority other MS, when Shattered Defenses is not up.

During Deadly Calm Slam should always be used, when CS/MS/Execute are not available, and not only when there is enough rage or a CS debuff.

Focused Rage should not be used, when you have less rage than required for a MS left after using it. (This is only a minor dps increase)

Here is an improved version of the single target rotation, socring ~3,5% / ~11k dps higher than the current profile:

actions.single+=colossus_smash,if=buff.shattered_defenses.down&buff.precise_strikes.down actions.single+=/mortal_strike actions.single+=/execute,if=buff.stone_heart.react|buff.battle_cry_deadly_calm.up actions.single+=/warbreaker,if=buff.shattered_defenses.down actions.single+=/execute,if=debuff.colossus_smash.up&rage.deficit<60 actions.single+=/focused_rage,if=(buff.focused_rage.stack<3&((talent.dauntless.enabled&rage>27)|(!talent.dauntless.enabled&rage>34)))|buff.battle_cry_deadly_calm.up actions.single+=/slam,if=target.health.pct>20&!talent.fervor_of_battle.enabled&(debuff.colossus_smash.up|rage.deficit<40|buff.battle_cry_deadly_calm.up)&(!talent.focused_rage.enabled|buff.battle_cry_deadly_calm.up) actions.single+=/whirlwind,if=target.health.pct>20&talent.fervor_of_battle.enabled&(debuff.colossus_smash.up|rage.deficit<50)&(!talent.focused_rage.enabled|buff.battle_cry_deadly_calm.up|buff.cleave.up) actions.single+=/rend,if=remains<=duration*0.3 actions.single+=/whirlwind,if=target.health.pct>20&talent.fervor_of_battle.enabled&(!talent.focused_rage.enabled|rage>100|buff.focused_rage.stack=3) actions.single+=/slam,if=target.health.pct>20&!talent.fervor_of_battle.enabled&(!talent.focused_rage.enabled|rage>100|buff.focused_rage.stack=3) actions.single+=/execute,if=equipped.137060 actions.single+=/slam,if=equipped.137060 actions.single+=/shockwave actions.single+=/storm_bolt actions.single+=/bladestorm,if=raid_event.adds.in>90|!raid_event.adds.exists|spell_targets.bladestorm_mh>desired_targets

Additionally the haste enchants on rings are quite a bit better than the mastery enchants at the current stats of the gear improving dps by an additional 2k. (although this could confuse people looking at the gear)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/simulationcraft/simc/issues/2799, or mute the thread https://github.com/notifications/unsubscribe-auth/AAgFyolcCJQSdyaJMaNOaT5N2tclHxyvks5qmdNDgaJpZM4J0Yjc .

Collisionc commented 8 years ago

As if arms needed more dps. :)

dfherr commented 8 years ago

arms always needs more dps :smiling_imp:

i guess it could be improved even further. this was just the stuff that seemed obvious to me and is still understandable from a player point of few. I even got 400dps more when setting the min rage for Focused Rage to 20 instead of 28 by mistake, but I dunno why :D

dfherr commented 8 years ago

pushing mortal strike below execute scores even more dps in my local sims. Also makes sense since it's more damage per execute when there is enough rage and also has a higher chance on triggering tactician. (and this is true even though the deathblow trait isn't used yet)

edit: additionally use rage.deficit<20 instead of rage>100 for the second slam condition. Without Unending Rage trait rage>100 can never be true

actions.single+=colossus_smash,if=buff.shattered_defenses.down&buff.precise_strikes.down
actions.single+=/warbreaker,if=buff.shattered_defenses.down
actions.single+=/execute,if=buff.stone_heart.react|buff.battle_cry_deadly_calm.up
actions.single+=/execute,if=debuff.colossus_smash.up&rage.deficit<60
actions.single+=/mortal_strike
actions.single+=/focused_rage,if=(buff.focused_rage.stack<3&((talent.dauntless.enabled&rage>27)|(!talent.dauntless.enabled&rage>34)))|buff.battle_cry_deadly_calm.up
actions.single+=/slam,if=target.health.pct>20&!talent.fervor_of_battle.enabled&(debuff.colossus_smash.up|rage.deficit<40|buff.battle_cry_deadly_calm.up)&(!talent.focused_rage.enabled|buff.battle_cry_deadly_calm.up)
actions.single+=/whirlwind,if=target.health.pct>20&talent.fervor_of_battle.enabled&(debuff.colossus_smash.up|rage.deficit<50)&(!talent.focused_rage.enabled|buff.battle_cry_deadly_calm.up|buff.cleave.up)
actions.single+=/rend,if=remains<=duration*0.3
actions.single+=/whirlwind,if=target.health.pct>20&talent.fervor_of_battle.enabled&(!talent.focused_rage.enabled|rage.deficit<20|buff.focused_rage.stack=3)
actions.single+=/slam,if=target.health.pct>20&!talent.fervor_of_battle.enabled&(!talent.focused_rage.enabled|rage.deficit<20|buff.focused_rage.stack=3)
actions.single+=/execute,if=equipped.137060
actions.single+=/slam,if=equipped.137060
actions.single+=/shockwave
actions.single+=/storm_bolt
actions.single+=/bladestorm,if=raid_event.adds.in>90|!raid_event.adds.exists|spell_targets.bladestorm_mh>desired_targets
Collisionc commented 8 years ago

Updated.