Skip to content

Commit

Permalink
Re-enabled Primal Madness cancellation during Berserk with a 0 Energy
Browse files Browse the repository at this point in the history
threshold, matching the same logic for the correction term applied to
cancellations during TF windows.

DPS impacts under default settings: P1 BiS = +86, P3 loot priority
reference = +100, P3 BiS = +65

 On branch feral
 Changes to be committed:
	modified:   sim/druid/feral/rotation.go
  • Loading branch information
NerdEgghead committed Sep 23, 2024
1 parent 851110c commit e2436db
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion sim/druid/feral/rotation.go
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ func (cat *FeralDruid) OnGCDReady(sim *core.Simulation) {
}

// Check for an opportunity to cancel Primal Madness if we just casted a spell.
if !cat.GCD.IsReady(sim) && cat.PrimalMadnessAura.IsActive() && !cat.BerserkAura.IsActive() && cat.Rotation.CancelPrimalMadness {
if !cat.GCD.IsReady(sim) && cat.PrimalMadnessAura.IsActive() && cat.Rotation.CancelPrimalMadness {
// Determine cancellation threshold based on the expected Energy
// loss when Primal Madness will naturally expire.
energyThresh := 10.0 * float64(cat.Talents.PrimalMadness)
Expand Down

0 comments on commit e2436db

Please sign in to comment.