Merge pull request #1064 from TheBackstabi/master

[Rogue] More Combat EP tuning, Sub and Asn Mastery Flooring
wowsims · Sep 30, 2024 · e0bddad · e0bddad
2 parents 11c7fee + e810f4b
commit e0bddad
Show file tree

Hide file tree

Showing 9 changed files with 1,363 additions and 1,337 deletions.
diff --git a/sim/rogue/assassination/TestAssassination.results b/sim/rogue/assassination/TestAssassination.results
diff --git a/sim/rogue/assassination/assassination.go b/sim/rogue/assassination/assassination.go
@@ -1,6 +1,8 @@
 package assassination
 
 import (
+	"math"
+
 	"github.com/wowsims/cata/sim/core"
 	"github.com/wowsims/cata/sim/core/proto"
 	"github.com/wowsims/cata/sim/core/stats"
@@ -9,6 +11,7 @@ import (
 
 const masteryDamagePerPoint = 0.035
 const masteryBaseEffect = 0.28
+const masteryFloored = true // Toggled locally for stat weight calculations
 
 func RegisterAssassinationRogue() {
 	core.RegisterAgentFactory(
@@ -81,7 +84,11 @@ func (sinRogue *AssassinationRogue) Initialize() {
 }
 
 func getMasteryBonus(masteryRating float64) float64 {
-	return masteryBaseEffect + core.MasteryRatingToMasteryPoints(masteryRating)*masteryDamagePerPoint
+	var effect = masteryBaseEffect + core.MasteryRatingToMasteryPoints(masteryRating)*masteryDamagePerPoint
+	if masteryFloored {
+		return math.Floor(effect*100) / 100
+	}
+	return effect
 }
 
 func NewAssassinationRogue(character *core.Character, options *proto.Player) *AssassinationRogue {