Kahorie's DK simulator 2.0 Beta - Including Cataclysm. - Page 38

Afabar · 10/11/09, 1:41 PM

Originally Posted by Fugazor

I used 1.1.7.0 and calculated EP for my 17/0/54 build over 1000h. Everything went fine but hit and expertise values which were negative. I tried to remove trinkets and reducing hit/exp in my template to 0 but no matter what I did I got negative values for hit and expertise. When I try to calculate EP manually (single sim runs) it all works fine.

I don't see anything in the code that could be wrong. Could you send me your Character, template, Priority/Rotation xml files to this email adress: dksimulator@gmail.com

Conditional priorities (in XML) may be needed. For example a way to express in frost...
BTW, if a death rune is in place on its own, does it proc a BS? I guess obviously not but making sure.

Put the Rime between your Obliterate and your Blood Strike wouldn't do the trick? And of course, Blood Strike do not use a Death Rune.

athinker · 10/12/09, 8:28 AM

Originally Posted by Afabar

Put the Rime between your Obliterate and your Blood Strike wouldn't do the trick?

I want to test the merits of:

"If Rime and Obl are ready and BS is not ready, use Rime 1st; if BS and Rime and Obl are ready, go with Obl first".

If I put Rime between Obl and BS, Obl will always be used first, not satisfying the 1st case.

PS. The reasoning for the interest about it is that if rune CDs should be kept to a minimum but at the same time a Rime may be a benefit for DPS, it might be interesting to investigate whether using it before 'some' of the runes are ready is better and it becomes worse if 'all' runes are ready.

PPS. It might be interesting to test also (the more general) "If less than any 4 runes are ready, use Rime first in that case".

Afabar · 10/12/09, 12:41 PM

If you can get me a name for this priority that would be obvious for anyone else than you, I will see what I can do.

Afabar · 10/15/09, 11:26 AM

1.1.7.1 Release Note:
Scourge Strike updated to the latest found on the Unholy thread.

Kahorie's DK Simulator 1.1.7.1

If you want to double check, here is the code:
ScourgeStrike.vb

Larisroth · 10/15/09, 10:38 PM

Hey thanks for the SS update.
Would it be possible to fix the desolation priority? Looking at the code it appears to blood strike when desolation is up rather than blood strike to put up desolation when it's down.

Bensch78 · 10/16/09, 4:45 AM

Originally Posted by Afabar

1.1.7.1 Release Note:
Scourge Strike updated to the latest found on the Unholy thread.

Kahorie's DK Simulator 1.1.7.1

If you want to double check, here is the code:
ScourgeStrike.vb

You are calculating the crit damage of the shadow part with vicious strikes, as you are calculating the damage based on the function CritCoef.

public Overrides Function CritCoef() As Double                 
CritCoef = 1 + TalentUnholy.ViciousStrikes * 15 / 100                 
CritCoef = CritCoef * (1+0.06*sim.mainstat.CSD)
End Function

I think this is incorrect. It will only crit with 200%, when fixed. The shadow part is only modified from modifiers that increase shadow damage as GC mentioned in part 4 of his post. Vicious Strikes should not apply.

If the crit-chance of Subversion and/ or Vicious Strikes will be applied to the shadow part after the fix has to be tested either.

public Overrides Function CritChance() As Double                 
dim tmp as Double                 
tmp = sim.MainStat.crit + TalentUnholy.ViciousStrikes * 3 / 100 + sim.MainStat.T72PDPS * 5 / 100 + talentblood.Subversion * 3 / 100                 
return  tmp         
End Function

If not, you would have to take another crit chance for the shadow part.

kelben · 10/17/09, 11:46 PM

Alright, time for one of my stupid questions - regarding expertise, I have 133 expertise rating. Since I'm human I get +3 and +5 from my talents resulting in a total of 24 expertise. The sim wants rating so do I just enter 133? How does that account for the human racial? I do realize that sim accounts for the talents that add expertise.

The main reason i'm asking is that I feel that the sim currently is over valuing expertise rating for me, because I don't understand how to properly enter it. I did read the readme but it didn't answer my questions.

Afabar · 10/19/09, 10:23 AM

1.1.7.2 Release notes
* Release notes on a separated file.
* Desolation priority correction.
* Scourge strike magical crit modifier corrected.
* Scourge strike magical part crit change corrected.
* Scourge strike double crit (Physical + magical) correction.
* Automatic Horn usage optimization. Use less often, but do not mess up EP value on some simulation. USe <Horn/> at the last priority item to go back to the old behavior.
* Some Tab resorting.

Kahorie's DK Simulator 1.1.7.2

There is no simple way to add racial expertise. For a 2H character, just convert your expertise bonus into rating and add it to you character file. For DW character, there is no real way to add it.

Lamperouqe · 10/21/09, 6:42 PM

I noticed something strange about the new version of the sim. According to the release notes, Frost DW dps shouldn't be affected at all, but I have about ~300 lower dps on the newest version of the sim than what I had on 1.1.7.0. When I started looking for the cause, I noticed that my Obliterate's crit% had dropped to 52,2% on the sim, even though the chance for my Obliterate to crit is 49,2% self buffed. I don't think raid buffs would increase my critical strike chance on Obliterate by only 3%. I remember the crit% on Obliterate on the sim being much higher before. I have 2/3 Subversion in my spec. And yes, I've double checked my template, character sheet, raid buffs enabled and priority. They are the same as before, simmed with my own stats. I'm not sure about all abilities critting less now, since I only remember Obliterate's crit% being higher.

Ability Total % Landed Hit% Crit% Miss% Average

Obliterate 908615259 31,1 88251 47,8 52,2 0 10295,8

FrostStrike 612124071 20,9 70705 48,5 51,5 0 8657,4

MainHand 324822538 11,1 148651 57,9 27,1 15 2185,1

FrostFever 247285161 8,5 119999 69,8 30,2 0 2060,7

BloodPlague 224936380 7,7 119998 69,7 30,3 0 1874,5

OffHand 185793574 6,3 153620 57,8 27 15,1 1209,4

HowlingBlast 118220199 4 12676 33,9 66,1 0 9326,3

Necrosis 102143311 3,5 302271 100 0 0 337,9

BloodCakedBlade 74813284 2,6 91171 100 0 0 820,6

Ghoul 64261930 2,2 81181 86,8 13,2 0 791,6

BloodStrike 61581925 2,1 16303 63,7 36,3 0 3777,3

Raz 1536200 0,1 153620 100 0 0 10

IcyTouch 14443 0 4 50 50 0 3610,8

PlagueStrike 7934 0 3 66,7 33,3 0 2644,7

Horn 0 0 7790 100 0 0 0

Pestilence 0 0 19318 100 0 0 0

DPS 8128

Total Damage 2926,16m in 100h

Threat Per Second 5332

Generated in 146s

Template: Frost 2-51-18 GoD

Priority: Frost

Presence: Blood

Sigil: Awareness

RuneEnchant: FallenCrusader / Razorice

Pet Calculation: True

Chimichanga · 10/22/09, 12:03 PM

While I don't have data to provide at the moment (at work), I can confirm Lamperouqe's observation regarding crit percentage and overall reduced DPS as compared to previous versions of the sim.

Edit: By the way Afabar, thanks for all the time you've put into this simulator. It's an amazing tool. The community is very lucky to have contributors like yourself.

Afabar · 10/22/09, 12:40 PM

I must have forgot to add that in the release notes, but some times ago, Doc made me a comment about base crit that was added to the character. I have correct this, so I have removed the 5% base crit that are in fact from base agility and don't have to be added.

dr_AllCOM3 · 10/22/09, 4:24 PM

I'm very sure my crit version was correct. There is also a crit reduction versus bosses of about 5%, that's most likely what you guys didn't factor in.

Bifurcamet · 10/23/09, 6:12 AM

Has anyone done math for dps given a certain fps rate ?

GravityDK · 10/23/09, 3:53 PM

Originally Posted by Bifurcamet

Has anyone done math for dps given a certain fps rate ?

that would be fascinating. I think it's a big factor if you're below 40 FPS, but no idea how you could model or maths it.
Certainly, using addons like OptionHouse and cpu-efficient addons is important. I doubt it'd be in-scope for this tool to take any FPS impact unless someone were to provide a formula.

Lamperouqe · 10/23/09, 10:27 PM

Originally Posted by Afabar

I must have forgot to add that in the release notes, but some times ago, Doc made me a comment about base crit that was added to the character. I have correct this, so I have removed the 5% base crit that are in fact from base agility and don't have to be added.

Thanks, that cleared it up. Since I didn't see that in the release notes I thought something was wrong.