weirdgloop / osrs-dps-calc

Web-based DPS calculator for Old School RuneScape
GNU General Public License v3.0
25 stars 25 forks source link

Enchanted bolt fixes #364

Closed jmyaeger closed 1 month ago

jmyaeger commented 1 month ago

Opal and pearl bolts were, in effect, rolling for accuracy and damage and then rolling for the bolt effect. The result is there was a higher proportion of hits that were equal to the effect bonus damage because they were inaccurate hits to which the bonus damage was added. Below is an illustration of the result I was getting for opal bolts (pearls were similar): image

Hiding inaccurate hits makes it clear that a lot of inaccurate hits were being boosted to 9s: image

I wanted to make sure that it wasn't like this in-game, so I collected some hit data: Opal bolts (e)

While it's not enough data for the distribution to be as smooth as I'd like, it's pretty clear that the number of 9s is not significantly higher than other hits. Based on this, I think it's implemented like diamond bolts, where it rolls for the bolt proc and, if successful, bypasses the accuracy roll entirely. I did not test pearl bolts, but I think it's a reasonable assumption that they are both implemented the same way. I changed both to be implemented like diamond bolts are in the calc.

I also changed onyx bolts to boost the max hit by 20% or 32% rather than adding 20% or 32% of the player's visible ranged level to the max hit. The wording in-game and on the wiki only refers to boosting damage by that amount and makes no mention of ranged level, and this Ash tweet supports that as well. image

LlemonDuck commented 1 month ago

Weird, I wouldn't have expected opals to do that but this seems correct. Thanks for the deep dive