Improve Arcane Mage implementation

README.md

@@ -55,7 +55,7 @@ Not yet implemented:
 
 To calculate EPs for a single character definition, use the following command:
 
-`./tbcsim --calc-ep-single <path_to_character_definition_file>`
+`./tbcsim --calc-ep <path_to_character_definition_file>`
 
 This uses the sim defaults of a step interval of 10ms and an iteration count of 10,000 - both can be adjusted to your preference.  See the CLI usage below, or just run `./tbcsim`.
 

src/commonMain/kotlin/character/Mutex.kt

@@ -18,6 +18,7 @@ enum class Mutex {
     BUFF_EXPOSE_WEAKNESS,
     BUFF_FEROCIOUS_INSPIRATION,
     BUFF_FAERIE_FIRE,
+    BUFF_SPIRIT,
 
     // Hunter
     BUFF_HUNTER_ASPECT,

src/commonMain/kotlin/character/Spec.kt

@@ -12,7 +12,7 @@ abstract class Spec {
             Triple("strength", Stats(strength = 50), 50.0),
             Triple("agility", Stats(agility = 50), 50.0),
             Triple("meleeCritRating", Stats(meleeCritRating = 5.0 * Rating.critPerPct), 5.0 * Rating.critPerPct),
-            Triple("physicalHitRating", Stats(physicalHitRating = 2.0 * Rating.physicalHitPerPct), 2.0 * Rating.physicalHitPerPct),
+            Triple("physicalHitRating", Stats(physicalHitRating = -5.0 * Rating.physicalHitPerPct), -5.0 * Rating.physicalHitPerPct),
             Triple("physicalHasteRating", Stats(physicalHasteRating = 5.0 * Rating.hastePerPct), 5.0 * Rating.hastePerPct),
             Triple("expertiseRating", Stats(expertiseRating = 2.0 * Rating.expertisePerPct), 2.0 * Rating.expertisePerPct),
             Triple("armorPen", Stats(armorPen = 100), 100.0),
@@ -33,7 +33,7 @@ abstract class Spec {
         // AKA Enhancement Shaman
         val casterHybridDeltas = listOf(
             Triple("spellCritRating", Stats(spellCritRating = 5.0 * Rating.critPerPct), 5.0 * Rating.critPerPct),
-            Triple("spellHitRating", Stats(spellHitRating = 5.0 * Rating.spellHitPerPct), 5.0 * Rating.spellHitPerPct)
+            Triple("spellHitRating", Stats(spellHitRating = -5.0 * Rating.spellHitPerPct), -5.0 * Rating.spellHitPerPct)
         )
         val defaultCasterDeltas: List<SpecEpDelta> = listOf(
             Triple("intellect", Stats(intellect = 50), 50.0),

src/commonMain/kotlin/character/classes/mage/Mage.kt

@@ -36,6 +36,7 @@ class Mage(talents: Map<String, Talent>, spec: Spec) : Class(talents, spec) {
             IcyVeins.name -> IcyVeins()
             ManaEmerald.name -> ManaEmerald()
             MoltenArmor.name -> MoltenArmor()
+            MageArmor.name -> MageArmor()
             PresenceOfMind.name -> PresenceOfMind()
             Scorch.name -> Scorch()
             SummonWaterElemental.name -> SummonWaterElemental()

src/commonMain/kotlin/character/classes/mage/abilities/Frostbolt.kt

@@ -49,7 +49,7 @@ class Frostbolt : Ability() {
     val spellPowerCoeff = Spell.spellPowerCoeff(baseCastTimeMs)
     override fun cast(sp: SimParticipant) {
         val elementalPrecision: ElementalPrecision? = sp.character.klass.talentInstance(ElementalPrecision.name)
-        val emHit = elementalPrecision?.bonusFireFrostHitPct() ?: 0.0
+        val emHit = 2 * (elementalPrecision?.bonusFireFrostHitPct() ?: 0.0)
 
         val empFb: EmpoweredFrostbolt? = sp.character.klass.talentInstance(EmpoweredFrostbolt.name)
         val bonusFbCrit = empFb?.frostboltAddlCritPct() ?: 0.0

src/commonMain/kotlin/character/classes/mage/abilities/MageArmor.kt

@@ -0,0 +1,5 @@
+package character.classes.mage
+
+class MageArmor {
+
+}

src/commonMain/kotlin/character/classes/mage/specs/Arcane.kt

@@ -2,11 +2,14 @@ package character.classes.mage.specs
 
 import character.Spec
 import character.SpecEpDelta
+import character.Stats
 
 class Arcane : Spec() {
     override val name: String = "Arcane"
     override val epBaseStat: SpecEpDelta = spellPowerBase
-    override val epStatDeltas: List<SpecEpDelta> = defaultCasterDeltas
+    override val epStatDeltas: List<SpecEpDelta> = listOf(Triple("spirit", Stats(spirit = 50), 50.0)) +
+            defaultCasterDeltas
+
 
     override fun redSocketEp(deltas: Map<String, Double>): Double {
         // 12 spell dmg
@@ -15,7 +18,7 @@ class Arcane : Spec() {
 
     override fun yellowSocketEp(deltas: Map<String, Double>): Double {
         // 5 spell haste rating / 6 spell damage
-        return ((deltas["spellHasteRating"] ?: 0.0) * 5.0) + 6.0
+        return ((deltas["intellect"] ?: 0.0) * 10.0)
     }
 
     override fun blueSocketEp(deltas: Map<String, Double>): Double {

src/commonMain/kotlin/data/abilities/generic/AdeptsElixir.kt

@@ -0,0 +1,5 @@
+package data.abilities.generic
+
+class AdeptsElixir : Ability() {
+
+}

src/commonMain/kotlin/data/abilities/generic/ElixirOfDraenicWisdom.kt

@@ -0,0 +1,2 @@
+package data.abilities.generic 
+

src/commonMain/kotlin/data/abilities/generic/GenericAbilities.kt

@@ -6,19 +6,23 @@ import character.Ability
 object GenericAbilities {
     fun byName(name: String): Ability? {
         return when(name) {
+            AdeptsElixir.name -> AdeptsElixir()
             BlackenedBasilisk.name -> BlackenedBasilisk()
             CrunchySerpent.name -> CrunchySerpent()
             DarkRune.name -> DarkRune()
             DemonicRune.name -> DemonicRune()
             DestructionPotion.name -> DestructionPotion()
+            ElixirOfDraenicWisdom.name -> ElixirOfDraenicWisdom()
             ElixirOfMajorAgility.name -> ElixirOfMajorAgility()
             ElixirOfMajorStrength.name -> ElixirOfMajorStrength()
             FlaskOfBlindingLight.name -> FlaskOfBlindingLight()
             FlaskOfPureDeath.name -> FlaskOfPureDeath()
             FlaskOfRelentlessAssault.name -> FlaskOfRelentlessAssault()
             HastePotion.name -> HastePotion()
+            Innervate.name -> Innervate()
             InsaneStrengthPotion.name -> InsaneStrengthPotion()
             RoastedClefthoof.name -> RoastedClefthoof()
+            ScrollOfSpiritV.name -> ScrollOfSpiritV()
             SpicyHotTalbuk.name -> SpicyHotTalbuk()
             SuperManaPotion.name -> SuperManaPotion()
             UseActiveTrinket.name -> UseActiveTrinket()

src/commonMain/kotlin/data/abilities/generic/Innervate.kt

@@ -0,0 +1,2 @@
+package data.abilities.generic 
+

src/commonMain/kotlin/data/abilities/generic/ScrollOfSpiritV.kt

@@ -0,0 +1,2 @@
+package data.abilities.generic 
+

src/commonMain/kotlin/data/abilities/raid/DivineSpirit.kt

@@ -2,6 +2,7 @@ package data.abilities.raid
 
 import character.Ability
 import character.Buff
+import character.Mutex
 import character.Stats
 import mechanics.Rating
 import sim.SimParticipant
@@ -20,6 +21,7 @@ class DivineSpirit : Ability() {
         override val name: String = Companion.name
         override val icon: String = "spell_holy_prayerofspirit.jpg"
         override val durationMs: Int = -1
+        override val mutex: List<Mutex> = listOf(Mutex.BUFF_SPIRIT)
 
         override fun modifyStats(sp: SimParticipant): Stats {
             return Stats(spirit = 50)

src/commonMain/kotlin/data/abilities/raid/ImprovedDivineSpirit.kt

@@ -2,6 +2,7 @@ package data.abilities.raid
 
 import character.Ability
 import character.Buff
+import character.Mutex
 import character.Stats
 import mechanics.Rating
 import sim.SimParticipant
@@ -20,6 +21,7 @@ class ImprovedDivineSpirit : Ability() {
         override val name: String = Companion.name
         override val icon: String = "spell_holy_prayerofspirit.jpg"
         override val durationMs: Int = -1
+        override val mutex: List<Mutex> = listOf(Mutex.BUFF_SPIRIT)
 
         override fun modifyStats(sp: SimParticipant): Stats {
             // assumes max rank

src/jvmMain/kotlin/Main.kt

@@ -153,15 +153,8 @@ class TBCSim : CliktCommand() {
     )
 
     fun singleEpSim(config: Config, opts: SimOptions, epDelta: SpecEpDelta? = null) : Pair<SpecEpDelta?, Double> {
-        // Most presets are hit capped, so apply a universal -2% hit buff so the hit has something to sim against
-        val hitReduction = Stats(
-            physicalHitRating = -2.0 * Rating.physicalHitPerPct,
-            expertiseRating = -2.0 * Rating.expertisePerPct,
-            spellHitRating = -5.0 * Rating.spellHitPerPct,
-        )
-
         val epStatMod = epDelta?.second ?: Stats()
-        val totalStatMod = Stats().add(epStatMod).add(hitReduction)
+        val totalStatMod = Stats().add(epStatMod)//.add(hitReduction)
 
         val iterations = runBlocking { Sim(config, opts, totalStatMod) {}.sim() }
         return Pair(epDelta, SimStats.dps(iterations).entries.sumByDouble { it.value?.mean ?: 0.0 })
@@ -265,7 +258,11 @@ class TBCSim : CliktCommand() {
         val specFilter = specFilterStr?.split(",")
         val categoryFilter = categoryFilterStr?.split(",")
 
-        if (calcEP) {
+        if (calcEP && configFile?.exists() == true) {
+            val config = ConfigMaker.fromYml(configFile!!.readText())
+            println("Starting EP run")
+            val deltas = computeEpDeltas(config, opts)
+        } else if (calcEP) {
             val epTypeRef = object : TypeReference<EpOutput>(){}
             val existing = mapper.readValue(File(epOutputPath).readText(), epTypeRef)
             // EP calculation sim

ui/src/presets/samples/mage_arcane_phase2.yml

@@ -53,7 +53,7 @@ gear:
   mainHand:
     name: The Nexus Key
     enchant: Major Spellpower (Weapon)
-    tempEnchant: Superior Wizard Oil
+    tempEnchant: Brilliant Wizard Oil
   rangedTotemLibram:
     name: Eredar Wand of Obliteration
   head:
@@ -112,14 +112,28 @@ gear:
 rotation:
   autoAttack: false
   precombat:
-    - name: Flask of Blinding Light
+    - name: Elixir of Draenic Wisdom
+    - name: Adept's Elixir
     - name: Crunchy Serpent
     - name: Arcane Intellect
-    - name: Molten Armor
+    - name: Mage Armor
+    - name: Scroll of Spirit V
 
   combat:
+    - name: Evocation
+      criteria:
+        - type: RESOURCE_PCT_LTE
+          pct: 30
+          resourceType: MANA
+        - type: FIGHT_TIME_REMAINING_GTE
+          seconds: 30
     - name: Blood Fury
     - name: Berserking
+    - name: Bloodlust
+      criteria:
+        - type: FIGHT_DURATION_GTE
+          seconds: 15
+    - name: Drums of Battle
     - name: Mana Emerald
       criteria:
         - type: RESOURCE_MISSING_GTE
@@ -131,31 +145,40 @@ rotation:
         - type: RESOURCE_MISSING_GTE
           amount: 3000
           resourceType: MANA
-    - name: Evocation
+        - type: ABILITY_COOLDOWN_GTE
+          ability: Mana Emerald
+          seconds: 1
+    - name: Innervate
       criteria:
         - type: RESOURCE_PCT_LTE
-          pct: 20
+          pct: 35
           resourceType: MANA
-        - type: FIGHT_TIME_REMAINING_GTE
-          seconds: 30
     - name: Cold Snap
       criteria:
+        - type: ABILITY_COOLDOWN_LTE
+          ability: Mana Emerald
+          seconds: 5
         - type: ABILITY_COOLDOWN_GTE
-          ability: Icy Veins
+          ability: Mana Emerald
           seconds: 1
     - name: Icy Veins
       criteria:
-        - type: FIGHT_TIME_ELAPSED_GTE
-          seconds: 5
+        - type: RESOURCE_MISSING_GTE
+          # Account for Serpent-Coil Braid bonus potential over the regular mana gem amount
+          amount: 3125
+          resourceType: MANA
     - name: Arcane Power
       criteria:
-        - type: FIGHT_TIME_ELAPSED_GTE
-          seconds: 5
+        - type: RESOURCE_MISSING_GTE
+          # Account for Serpent-Coil Braid bonus potential over the regular mana gem amount
+          amount: 3125
+          resourceType: MANA
     - name: Presence of Mind
-      criteria:
-        - type: FIGHT_TIME_ELAPSED_GTE
-          seconds: 5
     - name: Use Active Trinket
+      criteria:
+        - type: RESOURCE_MISSING_GTE
+          amount: 3125
+          resourceType: MANA
     # Cast AB if we're using cooldowns, have high mana, or have low mana and low stacks
     - name: Arcane Blast
       criteria:
@@ -170,16 +193,18 @@ rotation:
     - name: Arcane Blast
       criteria:
         - type: RESOURCE_PCT_GTE
-          pct: 20
+          pct: 25
           resourceType: MANA
     - name: Arcane Blast
       criteria:
         - type: BUFF_STACKS_LTE
           buff: Arcane Blast
           stacks: 2
-        - type: RESOURCE_PCT_LTE
-          pct: 20
-          resourceType: MANA
+    - name: Arcane Blast
+      criteria:
+        - type: BUFF_DURATION_LTE
+          buff: Arcane Blast
+          seconds: 1
     - name: Frostbolt
 
 raidBuffs: