I changed hunter_attack_t::dot_behavior to DOT_REFRESH;

This affects ES as well as SS ( tested ingame ) and BA ( cooldown > dot duration ).

Who added the non_consecutive option and why, that I can't tell. I guess it's to disable chain-shooting ES because the first shot is still travelling and the dot not yet applied, but there would have been options to prevent this in simc. Also it has string comparison during simulation, something dedmonwakeen doesn't like at all

We would need to compare
actions+=/explosive_shot,non_consecutive=1

to

actions+=/explosive_shot,non_consecutive=0,if=(remains<tick_time+travel_time)&!in_flight

The recent armory changes broke pet importing, which was fixed in our latest release. That should solve your problem of importing pets that you don't actually own.

The way blizzard implemented pet talents is different than player talents, which means the talent string works, but isn't 'human readable' like the player ones are. Please see the following issue for further discussion on importing from wowhead. Issue 738 - simulationcraft - Hunter Pet Talents - Import from wowhead talent calc? - World of Warcraft DPS Simulator - Google Project Hosting

There's been some recent complains about a bug with the Sic 'Em talent, where it is apparently only applying the discount to the base cost of pet spells, even when the cost is increased due to Wild Hunt (i.e. cost = Base * WildHunt - Base* (SicEm). This is arguably not a bug and just a less than ideal interaction of two talents.

I haven't seen formal results for this, but Female Dwarf has apparently been representing it this way for a long time, and recent threads like this seem to be saying it's still an issue.

Looking at sc_hunter.cpp it looks like SimC is not modelling the bug and is just multiplying the Sic'Em reduction into the whole cost, including the Wild Hunt increase if present. I'm not familiar enough with the code to conclude this has to be the case, but it looks like it on a cursory reading.

Thanks for bringing this to our attention. It has been fixed in r9035 and will be appear in the next release. The automatic reports we generate should update in a couple of hours.

Couple of small improvements to the default hunter profiles generated by the GUI as well as the Hunter_MM_T12H.simc profile used on the automated T12 heroic runs too (I realize these latter runs aren't really meant to represent optimal dps, but they might as well be improved in obvious ways).:


1. For the default action list generated upon import, it says:

when it should say:

to account for the 4.2 change to Careful Aim

2. Similarly the line to hardcast aimed shot in Hunter_MM_T12H.simc hasn't been updated; it should be changed to:

That bumps up the dps for the profile by a tiny amount (50dps @25k runs). Hunter_MM_T12H_Arcane.simc needs a similar change.

3. This next is dependent on gear so I'm not sure if you want to do anything about it, but profiles that include haste proc trinkets like [Hungerer] or [Prestor's Talisman of Machination] often perform better for MM if you hardcast aimedshot during their haste procs. This varies from person to person though, so I suppose it doesn't belong in the defaults - for me it shows as a 100ish dps gain to add "|buff.the_hungerer.up" to the aimed shot hardcasting conditional (and the related inverse conditionals) and changes which trinkets I should be wearing, but for the MM_T12H which mostly spams steadyshots, doing so is a slight DPS loss, likely due to lost T12 procs. However for Hunter_MM_T12H_Arcane.simc it looks to be an 80ish dps gain (again at 25k runs).

#3 leads me to wonder if there's a way to tell SimC to make something conditional on the type of proc from [Matrix Restabilizer] - I see SimC is correctly making it proc different stats based on which stat is highest (which changes because of procs from [Hungerer]), and lists them with separate proccounts for matrix_restabilizer_crit and matrix_restabilizer_haste - but I can't seem to add a conditional for say buff.matrix_restabilizer_haste.up without crashing SimC.

#1 and #2 are fixed in SVN, thanks for pointing that out.

#3 is a unique condition based on gear, so we won't change the defaults. Currently we don't have a way for you to track which type of buff you received from MR, I'll see if that's possible to separate out. For now if MR only procs haste when Hungerer is up, you can check for both of those procs to cast AS, granted it's not 100% since Hunger can expire before the hasted MR does, but it's the best you can do for the time being.

[e] I did #3 in SVN as well, each stat buff is its own unique buff, matrix_restabilizer_haste, etc.

Is the T12 4piece modeled correctly? I tried a few days ago with my hunter replacing 4t11 with 4t12, both hitcapped and 4t12 with the suggested haste rating of ~13% but simc tells me that 4t11 (3/4 heroic) is ~250 dps better than 4t12 (4/4 normal).

I tried to check if I got the T124set procc (buff.burning_adrenaline.up), but it didn't worked. Could you please put it in?
(I want to use KC as an MM, if I got the buff.)

For simplicity we've always referred to tier bonuses as tierX_Xpc. If you look at your buff list to see the available buffs, you'll notice that burning_adrenaline doesn't exist, but tier12_4pc, does.

Right now there is a bug with t12 4-piece: Arcane Shot wastes the t12 4-piece proc (called Burning Adrenaline), so that the proc is consumed but no focus is saved.

Ideally one can cast KC or perhaps even Aimed Shot in the cases where it procs and CS is far from being ready, but it's not always possible to do that.

Thanks for the report, this has been fixed in SVN and will be available in the next release.