Skip to content

Commit

Permalink
change magic number range to const surely we get more range soon copium
Browse files Browse the repository at this point in the history
  • Loading branch information
Krealle committed Mar 17, 2024
1 parent f5964d8 commit 051b528
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 10 deletions.
9 changes: 5 additions & 4 deletions src/analysis/retail/evoker/devastation/modules/Abilities.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ import CoreAbilities from 'analysis/retail/evoker/shared/modules/Abilities';
import { SpellbookAbility } from 'parser/core/modules/Ability';
import SPELL_CATEGORY from 'parser/core/SPELL_CATEGORY';
import SPELLS from 'common/SPELLS';
import { BASE_EVOKER_RANGE } from '../../shared';

class Abilities extends CoreAbilities {
spellbook(): SpellbookAbility[] {
Expand All @@ -15,7 +16,7 @@ class Abilities extends CoreAbilities {
gcd: {
base: 1500,
},
range: 25,
range: BASE_EVOKER_RANGE,
enabled: combatant.hasTalent(TALENTS.PYRE_TALENT),
},
{
Expand All @@ -29,7 +30,7 @@ class Abilities extends CoreAbilities {
suggestion: true,
recommendedEfficiency: 0.9,
},
range: 25,
range: BASE_EVOKER_RANGE,
enabled: combatant.hasTalent(TALENTS.FIRESTORM_TALENT),
},
{
Expand All @@ -45,7 +46,7 @@ class Abilities extends CoreAbilities {
suggestion: true,
recommendedEfficiency: 0.95,
},
range: 25,
range: BASE_EVOKER_RANGE,
enabled: combatant.hasTalent(TALENTS.ETERNITY_SURGE_TALENT),
},
{
Expand All @@ -60,7 +61,7 @@ class Abilities extends CoreAbilities {
recommendedEfficiency: 0.9,
extraSuggestion: 'You should aim to use this off CD.',
},
range: 25,
range: BASE_EVOKER_RANGE,
enabled: combatant.hasTalent(TALENTS.SHATTERING_STAR_TALENT),
},
//endregion
Expand Down
2 changes: 2 additions & 0 deletions src/analysis/retail/evoker/shared/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,5 @@ export const INNATE_MAGIC_REGEN = 0.05;
export const BASE_MAX_ESSENCE = 5;

export const POTENT_MANA_MULTIPLIER = 0.03;

export const BASE_EVOKER_RANGE = 25;
13 changes: 7 additions & 6 deletions src/analysis/retail/evoker/shared/modules/Abilities.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ import { SpellbookAbility } from 'parser/core/modules/Ability';
import SPELL_CATEGORY from 'parser/core/SPELL_CATEGORY';
import spells from 'common/SPELLS/dragonflight/trinkets';
import trinkets from 'common/ITEMS/dragonflight/trinkets';
import { BASE_EVOKER_RANGE } from '../constants';

const hasFont = (combatant: Combatant) =>
combatant.hasTalent(TALENTS.FONT_OF_MAGIC_PRESERVATION_TALENT) ||
Expand All @@ -31,7 +32,7 @@ class Abilities extends CoreAbilities {
gcd: {
base: 1500,
},
range: 25,
range: BASE_EVOKER_RANGE,
enabled: combatant.spec !== SPECS.AUGMENTATION_EVOKER,
},
{
Expand All @@ -42,7 +43,7 @@ class Abilities extends CoreAbilities {
gcd: {
base: 1500,
},
range: 25,
range: BASE_EVOKER_RANGE,
damageSpellIds: [SPELLS.EMERALD_BLOSSOM_CAST.id],
isDefensive: true,
},
Expand All @@ -64,15 +65,15 @@ class Abilities extends CoreAbilities {
recommendedEfficiency: 0.95,
},
}),
range: 25,
range: BASE_EVOKER_RANGE,
},
{
spell: SPELLS.LIVING_FLAME_CAST.id,
category: SPELL_CATEGORY.ROTATIONAL,
gcd: {
base: 1500,
},
range: 25,
range: BASE_EVOKER_RANGE,
damageSpellIds: [SPELLS.LIVING_FLAME_DAMAGE.id],
},
{
Expand All @@ -82,7 +83,7 @@ class Abilities extends CoreAbilities {
? SPELL_CATEGORY.HEALER_DAMAGING_SPELL
: SPELL_CATEGORY.ROTATIONAL,
cooldown: 0,
range: 25,
range: BASE_EVOKER_RANGE,
gcd: {
base: 1500,
},
Expand Down Expand Up @@ -186,7 +187,7 @@ class Abilities extends CoreAbilities {
gcd: {
base: 1500,
},
range: 25,
range: BASE_EVOKER_RANGE,
},
{
spell: TALENTS.OPPRESSING_ROAR_TALENT.id,
Expand Down

0 comments on commit 051b528

Please sign in to comment.