Merge pull request #1095 from Adamrch/pallyMinorFixes

remove double judge and fix HoW hit
wowsims · Sep 25, 2024 · 88010bb · 88010bb
2 parents d5c39e4 + 122de92
commit 88010bb
Show file tree

Hide file tree

Showing 4 changed files with 213 additions and 211 deletions.
diff --git a/sim/paladin/hammer_of_wrath.go b/sim/paladin/hammer_of_wrath.go
@@ -40,6 +40,7 @@ func (paladin *Paladin) registerHammerOfWrath() {
  DefenseType: core.DefenseTypeRanged,
  ProcMask: core.ProcMaskRangedSpecial, // TODO to be tested
  Flags: core.SpellFlagMeleeMetrics | core.SpellFlagAPL,
+ CastType: proto.CastType_CastTypeRanged,
 
  Rank: i + 1,
  RequiredLevel: int(rank.level),
@@ -59,6 +60,7 @@ func (paladin *Paladin) registerHammerOfWrath() {
  DamageMultiplier: 1,
  ThreatMultiplier: 1,
  BonusCoefficient: 0.429,
+ BonusHitRating: -float64(paladin.Talents.Precision) * core.MeleeHitRatingPerHitChance,
 
  ExtraCastCondition: func(sim *core.Simulation, target *core.Unit) bool {
  return sim.IsExecutePhase20()

diff --git a/sim/paladin/paladin.go b/sim/paladin/paladin.go
@@ -145,7 +145,7 @@ func (paladin *Paladin) Initialize() {
  paladin.registerBlessingOfSanctuary()
  paladin.registerLayOnHands()
 
- paladin.enableMultiJudge = true // change this to baseline false when P5 launches
+ paladin.enableMultiJudge = false // Was previously true in Phase 4 but disabled in Phase 5
  paladin.lingerDuration = time.Millisecond * 400
  paladin.consumeSealsOnJudge = true