cgdangelo

html=dikembe.html
single_actor_batch=1
chart_show_relative_difference=1
target_error=0.2

mage=dikembe
spec=arcane
level=120
race=troll

cgdangelo

commit e51691d169ae55d909d2854be29b4188a5523378
Author: Charles D'Angelo <[email protected]>
Date:   Thu Jun 21 11:54:40 2018 -0400

    [Mage] Add Time Anomaly in place of Temporal Flux

diff --git a/engine/class_modules/sc_mage.cpp b/engine/class_modules/sc_mage.cpp
index 3ade63c9bf..2db785156e 100755
--- a/engine/class_modules/sc_mage.cpp
+++ b/engine/class_modules/sc_mage.cpp

cgdangelo

single_actor_batch=1
chart_show_relative_difference=1
html=dikembe.html
target_error=0.1
optimal_raid=0
default_actions=0

mage=dikembe
spec=arcane
level=110

cgdangelo

diff --git a/engine/class_modules/sc_mage.cpp b/engine/class_modules/sc_mage.cpp
index 25ce93d64..eccfab486 100755
--- a/engine/class_modules/sc_mage.cpp
+++ b/engine/class_modules/sc_mage.cpp
@@ -902,6 +902,27 @@ struct mage_spell_base_t : public spell_t
       {
         sim -> errorf( "Player %s firestarer expression: unknown operation '%s'", player -> name(), splits[ 1 ].c_str() );
       }
+    } else if ( splits.size() == 4 && util::str_compare_ci( splits[ 0 ], "action" ) && util::str_compare_ci( splits[ 1 ], "arcane_blast" ) ) {
+      enum arcane_blast_expr_type_e { BASE_COST, CHARGED_COST };

cgdangelo

single_actor_batch=1
chart_show_relative_difference=1
html=dikembe.html
target_error=0.1

mage=dikembe
spec=arcane
level=110
race=troll
gear_crit_rating=400

cgdangelo

#![allow(dead_code)]

fn main() {
    use simfantasy::*;
    use std::time::Duration;

    let a = Box::new(Bard {});

    let options = SimulationOptions {
        combat_length: Duration::from_secs(10),

cgdangelo

fn main() {
    use simfantasy::*;

    let ss = StraightShot {};
    ss.perform();

    let hs = HeavyShot {};
    hs.perform();
}

cgdangelo

mod simfantasy {
    use std::time::Duration;
    use std::time::Instant;

    pub struct Simulation {
        pub combat_length: Duration,
        start_time: Option<Instant>,
        current_time: Option<Instant>,
    }

cgdangelo

4.444 dutch schedules execute for arcane_blast
4.444 dutch gains casting_1 ( value=-0.00 )
5.286 dutch performs arcane_blast (22772)
5.286 dutch arcane_blast hits Fluffy_Pillow for 7632 arcane damage (hit)
5.286 Fluffy_Pillow gains erosion_6 ( value=0.01 )
5.286 dutch gains 437.23 (437.23) mana from mana_regen (23208.91/25964.00)
5.286 dutch consumes 1100.0 mana for arcane_blast (22109)
5.286 dutch gains clearcasting_1 ( value=-1.00 )
5.286 dutch gains 0.00 (1.00) none from arcane_blast (0.00/0.00)
5.286 dutch loses casting

cgdangelo

Why Mana Adept is broken

• Disproportionately disruptive to rotation, compared to T15 talents of the other specs and classes. Talents like that are typically reserved for the higher tiers, often the L100 tier. Examples: Glacial Spike versus Thermal Void, Kindling versus Meteor. These talents provide the most noticeable playstyle variations and are confined to the same row.

• MA would not even be the first disruptive talent to make the jump from T15 to T100. Ray of Frost probably never belonged in the first tier, and was moved accordingly. I'm NOT implying that simply swapping Arcane Orb and Mana Adept would completely fix the issue. It would just place MA into a spot where it's allowed to be tuned more appropriately given its overall effects on rotation. (Extra bonus: T15 gets an AoE talent that is good.)

• Enforces a constrictive rotation onto the player for a spec that is already rigidly synchronized with cooldown timings (Rune of Power, Arcane Power). Being unable to spend extra mana to take down a ne