Skip to content

Commit

Permalink
Merge pull request #52 from wowsims/warrior-fix
Browse files Browse the repository at this point in the history
Some fix
  • Loading branch information
TheGroxEmpire authored Feb 5, 2024
2 parents 84fb1cc + ed368ea commit c2b4c8a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions sim/common/sod/enchant_effects.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ func init() {
character := agent.GetCharacter()

procMask := character.GetProcMaskForEnchant(803)
ppmm := character.AutoAttacks.NewPPMManager(5.0, procMask)
ppmm := character.AutoAttacks.NewPPMManager(6.0, procMask)

procSpell := character.RegisterSpell(core.SpellConfig{
ActionID: core.ActionID{SpellID: 13898},
Expand Down Expand Up @@ -60,7 +60,7 @@ func init() {
},
})

character.ItemSwap.RegisterOnSwapItemForEffectWithPPMManager(803, 5.0, &ppmm, aura)
character.ItemSwap.RegisterOnSwapItemForEffectWithPPMManager(803, 6.0, &ppmm, aura)
})

// Superior Striking
Expand Down
4 changes: 2 additions & 2 deletions ui/warrior/sim.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@ const SPEC_CONFIG = registerSpecConfig(Spec.SpecWarrior, {
// List any known bugs / issues here and they'll be shown on the site.
knownIssues: [
"Rage conversion is a bit off for non 60s",
"Prot oriented Rune: Devastate and Sword and Board is not yet implemented",
"Head and Bracers Rune is not yet implemented"
"Prot oriented Runes (Devastate and Sword and Board) are not yet implemented",
"Head and Bracers Runes are not yet implemented"
],

// All stats for which EP should be calculated.
Expand Down

0 comments on commit c2b4c8a

Please sign in to comment.