-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Incorrect calculations for certain input #13
Comments
Crystal armour strength multiplier was being derived incorrectly from the attack bonus (was Fixed in 512046b. 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: |
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: Max Hit: Let me know if I made any mistakes. |
Well I just realized I got strength and attack backwards so I have to push out another update 😞. Fixed in ac3c693. 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: 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. |
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:
In order of default, slayer helmet(i), crystal body, and crystal legs:
which matches the blog's values. |
Closing for now, since the original crystal armour bonuses were resolved. Residual changes needed for gear bonus additivity can be tracked in #17. |
The below input results in incorrect output:
Input:
![image](https://user-images.githubusercontent.com/18340303/124339815-f18f6180-db7e-11eb-8be5-23aeef7292dd.png)
![image](https://user-images.githubusercontent.com/18340303/124339803-e1778200-db7e-11eb-9db5-12211552c632.png)
![image](https://user-images.githubusercontent.com/18340303/124339823-04a23180-db7f-11eb-89e4-db91612b5ebf.png)
![image](https://user-images.githubusercontent.com/18340303/124339827-0b30a900-db7f-11eb-8079-af1e922c4533.png)
Output:
![image](https://user-images.githubusercontent.com/18340303/124339834-14217a80-db7f-11eb-89c3-10cd23a61470.png)
The text was updated successfully, but these errors were encountered: