Skip to content
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

Fix Calculation of Ignite/Bleed when at low stacks + show min/max DPS for Ignite/Bleed/Poison #6321

Merged
merged 8 commits into from
Aug 14, 2023

Conversation

LocalIdentity
Copy link
Contributor

The previous formula used for Ignite and Bleed stacking was incorrect and was off by around 15% at low stacks and gradually got closer the higher the stack value
This PR also adds support for showing the min and max DPS values for Ignite and Bleed skills.
It is much more useful for Bleed imo as you can low roll a bleed that deals 20% of your max damage
image
image

LocalIdentity added 4 commits August 13, 2023 19:35
Changes the formula for stacking ailment calculations as it overvalued lower hit rates too much
Also shows a DPS range for Ignites and Bleeds so people can see how
@LocalIdentity LocalIdentity added enhancement New feature, calculation, or mod bug: calculation Numerical differences labels Aug 13, 2023
LocalIdentity added 2 commits August 14, 2023 00:20
The roll range would not appear if the baseVal was unmodified, now it always shows up
@LocalIdentity LocalIdentity changed the title Fix Calculation of Ignite/Bleed when at low stacks + show min/max DPS for Ignite/Bleed Fix Calculation of Ignite/Bleed when at low stacks + show min/max DPS for Ignite/Bleed/Poison Aug 13, 2023
@LocalIdentity LocalIdentity merged commit eb000fb into dev Aug 14, 2023
@LocalIdentity LocalIdentity deleted the fix_ailment-calculations+min-max branch August 14, 2023 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug: calculation Numerical differences enhancement New feature, calculation, or mod
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant