timhul / ClassicSim

An event-driven simulation tool written in C++ for World of Warcraft Classic.
Other
116 stars 70 forks source link

bear dps soon? #194

Open che0522 opened 4 years ago

che0522 commented 4 years ago

just mcp maul. hoj triggers double maul. thank you for creating this tool.

timhul commented 4 years ago

Do you happen to know what the baseline melee damage is for bears and how does it scale? The cat implementation equips hidden claw items to modify the weapon range. I'm imagining bears should do the same.

yellowmia11 commented 4 years ago

I'm working on bear dps at the moment. Might create a pull request within the next few days.

yellowmia11 commented 4 years ago

I have implemented Bearform, Maul and the Ferocity talent to work but I'm facing some issues:

  1. I think Catform Claw Damage is not calculated properly for level ranges 20 to 59. Since we assume the damage increase is linear I assumed the same in the following experiment: I took DPS values from the damage tooltips of two druids (lvl 60 and lvl 53), both without any buffs and an attack speed of 1.0 in cat form and subtracted the dps gain caused by attack power: lvl 60 dps: 63.2 lvl 53 dps: 49.06 I tried to fit a linear function over {(53, 49.06), (60, 63.2)} which resulted in 2.02 level - 58 which can't be right since it would result in a negative dps value for lvl 20 cats. Linear Fit on Wolfram Alpha I think we need more data on this. For now I will probably settle for the current solution in CSIM and choose 2.5 60 for base paw damage and the same factors that were used for the cat claws. My druid on classic has a paw damage of 146.5 which is close but doesn't match it (2.5 * 60=150) either. The correct claw and paw damage values would change the range factors (which are currently 0.85 and 1.25) to 0.8 and 1.2.

Any ideas/comments would be appreciated :)

I created a pull request. I think it's in a usable state right now.