Incorrect calculations for certain input

[ThePharros]

The below input results in incorrect output:

Input:

Output:

[ThePharros]

And these are the values I got in a custom DPS calculator for the above input:

[LlemonDuck]

Crystal armour strength multiplier was being derived incorrectly from the attack bonus (was 2 * attBonus instead of 2 * (attBonus - 1) + 1, to double 30% => 60% instead of 130% => 260%).

Fixed in 512046b7. Will push out immediately as a quick fix.

We still had some discrepancies in our calculations, but I didn't see anything else that would be incorrect. Here are the new values I got:

[ThePharros]

Ah that would do it lol, thanks for the quick fix. Yeah I'm not sure where the discrepancies arise. Seems like mine is calculating higher accuracy with lower damage by comparison. Would you mind sharing some behind the scenes calculations? Since the Max Hit gets calculated early we could start from there. calculation I'm getting:

Effective Ranged Strength: ERS = ⌊(⌊(RangedLvl + Boost) PrayBonus⌋ + AttStyle + 8) VoidModifier⌋ = ⌊(⌊(99 + 13) 1.23⌋ + 0 + 8) 1⌋ = ⌊(⌊137.76⌋ + 8)⌋ = ⌊137 + 8⌋ = 145

Max Hit: Gear Ranged Strength = 113 Gear Bonus = 1.275 Max = ⌊0.5 + (ERS (GRS + 64) GB) / 640⌋ = ⌊0.5 + (145 (113 + 64) 1.275) / 640⌋ = ⌊0.5 + (145 (113 + 64) 1.275) / 640⌋ = ⌊0.5 + 32,722.85 / 640⌋ = ⌊51.6294921875⌋ = 51

Let me know if I made any mistakes.

[LlemonDuck]

Well I just realized I got strength and attack backwards so I have to push out another update 😞. Fixed in ac3c693b.

You do have the order a little off there: I apply the gear and crystal bonuses after the +320 /640, which matches the wiki page here: https://oldschool.runescape.wiki/w/Damage_per_second/Ranged#Step_two:_Calculate_the_maximum_hit

The only part I'm not sure about is whether the two gear bonuses should be applied multiplicatively then rounded, or rounded separately. I wasn't able to find good resources on this aspect.

[ThePharros]

lol no worries.

and that order shouldn't matter as it's just associative property of multiplication; the calculation will come out the same.

According to the ranged rebalance blog, the crystal armor and slayer helm bonuses stack together additively:

For example, we expect using an imbued Slayer Helmet with the Crystal Body and Legs to be popular, as it’ll grant a total bonus of 40% Ranged accuracy and 27.5% Ranged damage while on any Slayer task.

In order of default, slayer helmet(i), crystal body, and crystal legs:

Gear Bonus Damage = (1 + 0.15 + 0.075 + 0.05) = 1.275
Gear Bonus Accuracy = (1 + 0.15 + 0.15 + 0.1) = 1.4

which matches the blog's values.

[LlemonDuck]

Closing for now, since the original crystal armour bonuses were resolved.

Residual changes needed for gear bonus additivity can be tracked in #17.