From 198e77d79045ae67c87e095c1c58651fd0e9e841 Mon Sep 17 00:00:00 2001 From: The Grox Empire <47816228+TheGroxEmpire@users.noreply.github.com> Date: Mon, 5 Feb 2024 19:47:55 +0700 Subject: [PATCH] Some fix Core: adjust fiery to 6ppm Warrior: cleanup on UI --- sim/common/sod/enchant_effects.go | 4 ++-- ui/warrior/presets.ts | 1 - ui/warrior/sim.ts | 4 ++-- 3 files changed, 4 insertions(+), 5 deletions(-) diff --git a/sim/common/sod/enchant_effects.go b/sim/common/sod/enchant_effects.go index 711f6923fe..25d06c8b99 100644 --- a/sim/common/sod/enchant_effects.go +++ b/sim/common/sod/enchant_effects.go @@ -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}, @@ -60,7 +60,7 @@ func init() { }, }) - character.ItemSwap.RegisterOnSwapItemForEffectWithPPMManager(803, 5.0, &ppmm, aura) + character.ItemSwap.RegisterOnSwapItemForEffectWithPPMManager(803, 6.0, &ppmm, aura) }) // Superior Striking diff --git a/ui/warrior/presets.ts b/ui/warrior/presets.ts index 09eb639a15..5c1c516a2c 100644 --- a/ui/warrior/presets.ts +++ b/ui/warrior/presets.ts @@ -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 diff --git a/ui/warrior/sim.ts b/ui/warrior/sim.ts index d30c0614ff..3bd452991b 100644 --- a/ui/warrior/sim.ts +++ b/ui/warrior/sim.ts @@ -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.