Skip to content

Fix incorrect max resist calculation when using Saffell's Frame unique#1535

Merged
Wires77 merged 2 commits into
PathOfBuildingCommunity:devfrom
raxkin:fix-max-resist-saffells-frame-unique
Dec 6, 2025
Merged

Fix incorrect max resist calculation when using Saffell's Frame unique#1535
Wires77 merged 2 commits into
PathOfBuildingCommunity:devfrom
raxkin:fix-max-resist-saffells-frame-unique

Conversation

@raxkin
Copy link
Copy Markdown
Contributor

@raxkin raxkin commented Dec 6, 2025

Fixes #1505

Description of the problem being solved:

It was adding the max block to the already calculate max res without taking into acound the max res cap. I just added it inside the m_min, so it can't never go over ResistMax

Steps taken to verify a working solution:

Imported the same build that had the problem and verify that the max res can't go over 90

Link to a build that showcases this PR:

Before: https://pobb.in/7VIVf1JBz-d5
After: https://pobb.in/HF0qCGlUL9ry

Before screenshot:

imagen

After screenshot:

imagen

@Wires77 Wires77 added the bug: calculation Numerical differences label Dec 6, 2025
@Wires77 Wires77 merged commit 79fc9cb into PathOfBuildingCommunity:dev Dec 6, 2025
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.

Maximum resistances can incorrectly exceed 90% via increases to max block chance with the unique shield Saffell's Frame equipped

2 participants