Skip to content

Commit

Permalink
Merge pull request #1064 from TheBackstabi/master
Browse files Browse the repository at this point in the history
[Rogue] More Combat EP tuning, Sub and Asn Mastery Flooring
  • Loading branch information
1337LutZ authored Sep 30, 2024
2 parents 11c7fee + e810f4b commit e0bddad
Show file tree
Hide file tree
Showing 9 changed files with 1,363 additions and 1,337 deletions.
1,218 changes: 609 additions & 609 deletions sim/rogue/assassination/TestAssassination.results

Large diffs are not rendered by default.

9 changes: 8 additions & 1 deletion sim/rogue/assassination/assassination.go
Original file line number Diff line number Diff line change
@@ -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"
Expand All @@ -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(
Expand Down Expand Up @@ -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 {
Expand Down
Loading

0 comments on commit e0bddad

Please sign in to comment.