-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Added Support for automatically calculating Storm Rain DPS #6782
base: dev
Are you sure you want to change the base?
Conversation
This is technically implemented as the config for num of overlapping beams, but if you wanted to calculate that automatically (which you cant really as not all the beams will overlap and hit the enemy), there are better ways of doing this. If you just dont apply it as a hittime override it will already factor in the attack speed, allowing you to do this entirely within the predamage func rather than having its own function setting Also as there are other uses of dps multiplier this breaks support for some of the stuff so if you wanted to keep this layout I recommend doing edit: you also need to fix the radius reliant on skill part 2, you also need to cap it at 100 arrows out at a time, as each beam will hit up to 4 times, and there can only be a maximum of 100 arrows out at a given time, so you need to take those into account, this means your damage numbers are off by atleast 4 times, for a low arrows/second count (due to getting 4 beam hits per arrow shot), and more if you go over the cap (eg beams fire every 0.4 seconds, so it takes 1.6 seconds to consume all the beams, so at even 62 arrows/second your already at the cap) |
DPS formula will now be DPS= x * z * a where x = min(100/arrowDuration, projectilesBeforeFalloff) Some math examples:
|
…nt for arrow cap and duration
Implements #3158
I followed how explosive arrow fuse count was implemented since this seemed a very similar application. First PR so let me know if I am doing anything wrong.
Description of the problem being solved:
Calculates the DPS for storm rain with the below formula
a=average beam damage
x=arrow count (amount of arrows that are fired)
y=attack speed (how many attacks are performed in a second)
z=frequency (how many times beams fire per second)
DPS=(x * y * z) * a
Steps taken to verify a working solution:
Used two different gearsets to prove the dps calculation is correct
DPS=(6 * 3 * 2.5) * 100k ~ 4.5 mil
DPS=(4 * 3.5 * 3.75) * 200k ~ 10.5 mil
Link to a build that showcases this PR:
https://pobb.in/vuxKXMZ0sQpA
After screenshot:
https://prnt.sc/kZMFmKsombof
https://prnt.sc/YNJMnmkCIA0S