Skip to content

Fix stat values for some Tree nodes when using Hulking Form#1414

Merged
LocalIdentity merged 1 commit into
devfrom
ifx_node-scaling
Sep 20, 2025
Merged

Fix stat values for some Tree nodes when using Hulking Form#1414
LocalIdentity merged 1 commit into
devfrom
ifx_node-scaling

Conversation

@LocalIdentity
Copy link
Copy Markdown
Contributor

If Hulking form was allocated we were still scaling Notables by 1x and the gsub to scale the mod value would match multiple values in the line and replace them all with the scaled value
e.g. 1% increased Damage per 15 Strength
1% would be scaled by 1x and then the gsub would replace both numbers in the line with 1 so it would become
1% increased Damage per 1 Strength
Fixes #1413

Build:
https://maxroll.gg/poe2/pob/1c29d0gt

Before:
image
image

After:
image
image

If Hulking form was allocated we were still scaling Notables by 1x and the gsub to scale the mod value would match multiple values in the line and replace them all with the scaled value
e.g. 1% increased Damage per 15 Strength
1% would be scaled by 1x and then the gsub would replace both numbers in the line with 1 so it would become
1% increased Damage per 1 Strength
@LocalIdentity LocalIdentity added the bug: calculation Numerical differences label Sep 18, 2025
@Wires77
Copy link
Copy Markdown
Member

Wires77 commented Sep 18, 2025

Do any stats on the tree scale the second number instead? That could cause an issue here

@LocalIdentity
Copy link
Copy Markdown
Contributor Author

Do any stats on the tree scale the second number instead? That could cause an issue here

Heavy Drinker

20% increased Life Recovery from Flasks
Life Flasks applied to you grant Guard for 4 seconds equal to 8% of the Life Recovery per Second they apply

I checked all the nodes on the tree and the only one I found was a Flask notable that is not in the range of any jewel sockets so can't scale anyway

Copy link
Copy Markdown
Member

@Wires77 Wires77 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably fine for now, especially if this is only visual

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.

Brute Strength node has incorrect description

2 participants