Merge pull request #1177 from emsimpson92/saeyon

Updated some damage modifiers to be additive rather than multiplicative
wowsims · Nov 23, 2024 · 26aa514 · 26aa514
2 parents a87580c + acf7251
commit 26aa514
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/sim/rogue/eviscerate.go b/sim/rogue/eviscerate.go
@@ -64,7 +64,7 @@ func (rogue *Rogue) registerEviscerate() {
 			return rogue.ComboPoints() > 0
 		},
 
-		DamageMultiplier: 1 +
+		DamageMultiplierAdditive: 1 +
 			[]float64{0, 0.05, 0.10, 0.15}[rogue.Talents.ImprovedEviscerate] +
 			[]float64{0, 0.02, 0.04, 0.06}[rogue.Talents.Aggression],
 		ThreatMultiplier: 1,

diff --git a/sim/rogue/items_sets_pve.go b/sim/rogue/items_sets_pve.go
@@ -412,17 +412,17 @@ var ItemSetEmblemsofVeiledShadows = core.NewItemSet(core.ItemSet{
 var ItemSetDeathdealersThrill = core.NewItemSet(core.ItemSet{
 	Name: "Deathdealer's Thrill",
 	Bonuses: map[int32]core.ApplyEffect{
-		// Increases Saber Slash damage by 20%
+		// Increases Saber Slash and Sinister Strike damage by 20%
 		2: func(agent core.Agent) {
 			rogue := agent.(RogueAgent).GetRogue()
 			rogue.RegisterAura(core.Aura{
 				Label: "S03 - Item - TAQ - Rogue - Damage 2P Bonus",
 				OnInit: func(aura *core.Aura, sim *core.Simulation) {
 					if rogue.HasRune(proto.RogueRune_RuneSaberSlash) {
-						rogue.SaberSlash.DamageMultiplier *= 1.20
-						rogue.saberSlashTick.DamageMultiplier *= 1.20
+						rogue.SaberSlash.DamageMultiplierAdditive += .2
+						rogue.saberSlashTick.DamageMultiplierAdditive += .2
 					}
-					rogue.SinisterStrike.DamageMultiplier *= 1.20
+					rogue.SinisterStrike.DamageMultiplierAdditive += .2
 				},
 			})
 		},