Stormboltz / dksimulator

Automatically exported from code.google.com/p/dksimulator
1 stars 1 forks source link

Estimated Haste EP values are slightly wrong. #19

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What is the expected output? What do you see instead?

The estimated Haste EP values have a contribution from damage dealing
procs.  At the moment all those procs are having the haste bonus applied
when it should really only be applied if the source is a hasteable attack.
 The estimated haste EP value should act as a lower bound on the value of
haste (given you're more or less "guaranteed" the extra attacks).

For the time being it probably makes sense to turn off the haste bonus for
anything that procs off things other than autoattacks.

Longer term the proc testing routine should probably take into account as
to whether the strike is hasteable.

This only effects the estimated haste EP value, so is a low priority.

Original issue reported on code.google.com by quixi...@gmail.com on 11 Apr 2010 at 11:36