Skip to content

Add support for Petrified Blood EHP - #3981

Closed
deathbeam wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
deathbeam:petrified-blood
Closed

deathbeam wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
deathbeam:petrified-blood

Conversation

@deathbeam

@deathbeam deathbeam commented Jan 29, 2022 •

Copy link
Copy Markdown
Contributor

This adds support for Petrified Blood life loss prevention to EHP calcs
in similar fashion as Mind over Matter. The prevention gain is checked
against being at low life or not, below low life it will use full value
(e.g 40 base) above full life it will use halved value (Pathfinders for
example utilize this mechanic and recover to full only via flasks).

Signed-off-by: Tomas Slusny slusnucky@gmail.com

Fixes #2737

Link to a build that showcases this PR:

https://pastebin.com/TxUzDmsC (full life Petrified Blood Pathfinder)

Before screenshot:

image

After screenshot:

image

@deathbeam
deathbeam force-pushed the petrified-blood branch 2 times, most recently from e649647 to 714d564 Compare January 29, 2022 00:38
@deathbeam deathbeam changed the title Add support for Petrified Blood life loss prevention and recovery dis… Add support for Petrified Blood EHP and recovery disabling Jan 29, 2022
Comment thread src/Data/Skills/act_str.lua Resolved
@deathbeam deathbeam changed the title Add support for Petrified Blood EHP and recovery disabling Add support for Petrified Blood EHP Jan 29, 2022
Comment thread src/Modules/CalcSections.lua Resolved
@Nostrademous Nostrademous added the enhancement New feature, calculation, or mod label Jan 29, 2022
This adds support for Petrified Blood life loss prevention to EHP calcs
in similar fashion as Mind over Matter. The prevention gain is checked
against being at low life or not, below low life it will use full value
(e.g 40 base) above full life it will use halved value (Pathfinders for
example utilize this mechanic and recover to full only via flasks).

Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
@deathbeam

Copy link
Copy Markdown
Contributor Author

Updated the PR and moved Petrified Blood calcuations before guard skill, as based on this post that is the correct order: https://www.pathofexile.com/forum/view-thread/3123794

The actual order btw is (from bottom to top):

Petrified blood, x before life (so mind over matter), energy shield, guard, aegis, frost shield, jinxed juju

So I think also mind over matter is in incorrect position and should be before guard skill, but I am not going to fix that here as that is different issue.

@Regisle

Regisle commented Jan 30, 2022

Copy link
Copy Markdown
Member

@deathbeam the whole thing is being reworked anyway in #2390 and its order is correctly applied there, also your list is missing ward

@deathbeam

Copy link
Copy Markdown
Contributor Author

Ah i see. And yea that post is missing info about ward and i was just going based on that.

@Wires77

Wires77 commented Feb 4, 2022

Copy link
Copy Markdown
Member

@deathbeam The calc defences rework PR has been merged into dev, so can you take a look at the conflicts for me and we'll take another look

@Regisle

Regisle commented Feb 4, 2022

Copy link
Copy Markdown
Member

I am in the process of fixing it to work with the new system, as well as giving people an idea of what "degen" they can expect

@deathbeam

Copy link
Copy Markdown
Contributor Author

Closing in favour of #4060 because of the calc defenses rework

@deathbeam deathbeam closed this Feb 4, 2022
@deathbeam
deathbeam deleted the petrified-blood branch February 4, 2022 08:55
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.

Add support of eHP for petrified blood

4 participants