Skip to content

Conversation

@majochem
Copy link
Contributor

@majochem majochem commented Sep 1, 2025

Fixes #1170 .

Note:

This assumes that the calculation that applies Evasion Rating to Elemental Damage works the same way as the Armour Formula, which has technically never been confirmed by GGG, but does seem like the most logical option.
This has now been confirmed (see screenshot below)

Description of the problem being solved:

  • Add support for "50% of Evasion Rating also grants Elemental Damage reduction"
  • Rework "... and Protect me From Harm" (Invoker Ascendancy Node) - ""Physical Damage Reduction from Armour is based on your combined Armour and Evasion Rating" to be more flexible and also work with self-damage calculation
  • Improved Breakdown for Defences applying to non-physical damage

Steps taken to verify a working solution:

  • Evasion rating properly increases max hit
  • Interacts correctly with "Armour Applies to Elemental Damage" (only tested for 'Blackbraid' Unique since the new mod is not yet supported)
  • Works correctly with Iron Reflexes (no more reduction)
  • "... and Protect Me From Harm" still works correctly

Link to a build that showcases this PR:

https://maxroll.gg/poe2/pob/lgwrt0q5

After screenshot:

image

Just Evasion applies example
image

Combined with 'Blackbraid Unique' (and 10k Armour)
image

Viperesque confirming the mechanic:
image

- This also reworks how "... and protect me from Harm" works, by
  replacing the boolean `FLAG` "EvasionAddsToPdr" to use a more generic
  and numerical "EvasionAlsoAppliesToPhysicalDamageTaken" `BASE` mod
  that works similarly to the Armour mods that apply to Elemental damage
This was previously not done for "EvasionAddsToPdr"
@majochem majochem marked this pull request as ready for review September 1, 2025 14:03
@LocalIdentity LocalIdentity added the enhancement New feature, calculation, or mod label Sep 1, 2025
@LocalIdentity LocalIdentity merged commit 59fd6a8 into PathOfBuildingCommunity:dev Sep 2, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature, calculation, or mod

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pathfinder "Sustainable Practices" ascendancy point

2 participants