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

Another attempt at correct Doom Blast calculations #6676

Merged
merged 21 commits into from
Sep 14, 2023

Conversation

Paliak
Copy link
Contributor

@Paliak Paliak commented Sep 13, 2023

Fixes #6481

Description of the problem being solved:

Steps taken to verify a working solution:

  • Compared the values to the Doom Blast spreadsheet
  • Consulted with author of spreadsheet and user from linked issue

Link to a build that showcases this PR:

https://pobb.in/gF8sKhHrauwf

Before screenshot:

obraz

After screenshot:

obraz

@LocalIdentity LocalIdentity merged commit 0b79956 into PathOfBuildingCommunity:dev Sep 14, 2023
@LocalIdentity LocalIdentity added the bug: calculation Numerical differences label Sep 14, 2023
shafouz pushed a commit to shafouz/PathOfBuilding that referenced this pull request Dec 8, 2023
…munity#6676)

* FIX: misc fulldps issues

* FIX: cleanup dead code

* FIX: doom blast calcs

* FIX: spellcheck

* FIX: use curse cast rate in curse replacement mode

* FIX: apply min mod on source rate

* FIX: move Doom Blast source config to eff dps

* FIX: modify Doom Blast to better match spreadsheet

https://docs.google.com/spreadsheets/d/1J-yVLmDhKqKNE8QecsPnZgNR2TDGDm41f-1tdC_BxQA/edit#gid=1230763965

* FIX: make vixen's mode defualt

* FIX: move doom blast options to skill options

* Move to alphabetical order

* FIX: another attempt at correctly implementing doom blast

* FIX: remove duplicate warn

* FIX: implement the suggested formula

* FIX: incorrect matching if doom blast not coming from a gem

* Revert "FIX: implement the suggested formula"

This reverts commit e65e1f5.

---------

Co-authored-by: LocalIdentity <[email protected]>
@Paliak Paliak deleted the doomBlastTweaks branch January 20, 2024 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug: calculation Numerical differences
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New trigger rate rework does not work correctly for Impending Doom
2 participants