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 Juggernaut Unbreakable not working with Iron Reflexes + evasion #6101

Merged
merged 1 commit into from
Apr 22, 2023

Conversation

LocalIdentity
Copy link
Contributor

The Juggernaut Unbreakable node is meant to double the armour received from having iron reflexes and evasion on your body armour
Fixes #6096

The Juggernaut Unbreakable node is meant to double the armour received from having iron reflexes and evasion on your body armour
@LocalIdentity LocalIdentity added the bug: calculation Numerical differences label Apr 21, 2023
Copy link
Member

@Wires77 Wires77 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I briefly looked at this the other day. This works for now, but someday I think a refactor of these calculations might be in order. It seems like we should base values of each defense type on gear, then converting or calculating totals afterward. There's a heap of caveats to that approach, though, so I'm in no rush to do that work now :)

@LocalIdentity LocalIdentity merged commit 4c595b2 into dev Apr 22, 2023
@LocalIdentity LocalIdentity deleted the fix_unbreakable-evasion branch April 22, 2023 05:46
Dullson pushed a commit to Dullson/PathOfBuilding that referenced this pull request Dec 6, 2023
…athOfBuildingCommunity#6101)

The Juggernaut Unbreakable node is meant to double the armour received from having iron reflexes and evasion on your body armour

Co-authored-by: LocalIdentity <localidentity2@gmail.com>
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.

Juggernaut "Bugfix" from 3.21.0c not working ?
2 participants