Skip to content

Commit

Permalink
Some fix
Browse files Browse the repository at this point in the history
Core: adjust fiery to 6ppm
Warrior: cleanup on UI
  • Loading branch information
TheGroxEmpire committed Feb 5, 2024
1 parent a1e1370 commit 198e77d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 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
1 change: 0 additions & 1 deletion ui/warrior/presets.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ import * as PresetUtils from '../core/preset_utils.js';
import BlankGear from './gear_sets/blank.gear.json';

import DefaultAPL from './apls/default.apl.json';
import { BlackfathomSharpeningStone, ConjuredMinorRecombobulator } from 'ui/core/components/inputs/consumables.js';

// Preset options for this spec.
// Eventually we will import these values for the raid sim too, so its good to
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 198e77d

Please sign in to comment.