Skip to content

Fix negative eHP and NaN Max hit values for some builds#9772

Merged
LocalIdentity merged 1 commit intodevfrom
fix_inf-ehp
Apr 17, 2026
Merged

Fix negative eHP and NaN Max hit values for some builds#9772
LocalIdentity merged 1 commit intodevfrom
fix_inf-ehp

Conversation

@LocalIdentity
Copy link
Copy Markdown
Contributor

Description of the problem being solved:

Some builds could show up having negative eHP values or nan max hit when they have single digit life values
numHits could become negative which would make eHP show up as negative too
Enemy damage conversion could also result in max hit becoming 0 in one part of the code and that would then display as NaN after it tries to divide by 0

Link to a build that showcases this PR:

https://maxroll.gg/poe/pob/qvznz0rl

Before screenshot:

image

After screenshot:

image

Some builds could show up having negative eHP values or nan max hit when they have single digit life values
numHits could become negative which would make eHP show up as negative too
Enemy damage conversion could also result in max hit becoming 0 in one part of the code and that would then display as NaN after it tries to divide by 0
@LocalIdentity LocalIdentity added bug: calculation Numerical differences pob2 Label for features that should be ported over to PoB-PoE2 labels Apr 17, 2026
@LocalIdentity LocalIdentity merged commit cd1d28e into dev Apr 17, 2026
3 checks passed
@LocalIdentity LocalIdentity deleted the fix_inf-ehp branch April 17, 2026 19:10
github-actions bot pushed a commit to PathOfBuildingCommunity/PathOfBuilding-PoE2 that referenced this pull request Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug: calculation Numerical differences pob2 Label for features that should be ported over to PoB-PoE2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant