Skip to content

Instantly share code, notes, and snippets.

View hey_look_overpowered_still_better_than_everything.simc
html=dikembe.html
single_actor_batch=1
chart_show_relative_difference=1
target_error=0.2
mage=dikembe
spec=arcane
level=120
race=troll
View time_anomaly.patch
commit e51691d169ae55d909d2854be29b4188a5523378
Author: Charles D'Angelo <c.g.dangelo@gmail.com>
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
View still_dont_care_about_mana.simc
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
View diff.patch
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 };
View gist:55b9d1212c948525716b7dea7ebf682a
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
View simfantasy.rs
#![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),
View wtf.rs
fn main() {
use simfantasy::*;
let ss = StraightShot {};
ss.perform();
let hs = HeavyShot {};
hs.perform();
}
View bad.rs
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>,
}
View gist:3d8ad028790e5b67683390a2cf33f8eb
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
View mana_adept.md

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