Skip to content

Fix "x to Total Mana Cost" mods not working correctly with Life / ES cost conversion - #10262

Merged
LocalIdentity merged 1 commit into
devfrom
fix_cost-conversion-calc
Aug 23, 2026
Merged

LocalIdentity merged 1 commit into
devfrom
fix_cost-conversion-calc

Conversation

@LocalIdentity

@LocalIdentity LocalIdentity commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

Description of the problem being solved:

Fix added Total Mana cost when converting to Energy Shield or Life cost.

For some reason the game can only add to the base cost but not subtract from it. So modifiers such as Non-Channelling Skills have -7 to Total Mana Cost do not reduce Blood Magic life cost but Non-Channelling Skills have +7 to Total Mana Cost would increase it.

Added tests to cover multiple scenarios

Link to a build that showcases this PR:

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

Before screenshot:

image image

After screenshot:

image image

Fix added Total Mana cost when converting to Energy Shield or Life cost.

For some reason the game can only add to the base cost but not subtract from it. So modifiers such as `Non-Channelling Skills have -7 to Total Mana Cost` do not reduce Blood Magic life cost but `Non-Channelling Skills have +7 to Total Mana Cost` would increase it.

Added tests to cover multiple scenarios
@LocalIdentity LocalIdentity added the bug: calculation Numerical differences label Aug 21, 2026
@LocalIdentity LocalIdentity changed the title Convert positive late flat skill costs with their resource Fix "x to Total Mana Cost" mods not working correctly with Life / ES cost conversion Aug 23, 2026
@LocalIdentity
LocalIdentity merged commit 8734d29 into dev Aug 23, 2026
3 checks passed
@LocalIdentity
LocalIdentity deleted the fix_cost-conversion-calc branch August 23, 2026 09:53
LocalIdentity added a commit to PathOfBuildingCommunity/PathOfBuilding-PoE2 that referenced this pull request Aug 30, 2026
…h Life / ES cost conversion (#2478)

* Apply changes from PathOfBuildingCommunity/PathOfBuilding#10262

* Fix port

---------

Co-authored-by: LocalIdentity <LocalIdentity@users.noreply.github.com>
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.

1 participant