Skip to content

Conversation

@majochem
Copy link
Contributor

@majochem majochem commented Sep 2, 2025

Adds support for:

  • Chaos Damage from Fire Spells Contributes to Flammability and Ignite Magnitudes
  • Fire Spells Convert 100% of Fire Damage to Chaos Damage
  • Ignite inflicted with Fire Spells deals Chaos Damage instead of Fire Damage

Technically also supports other elemental versions of those same mods. "Lightning Spells convert 50% of cold damage to fire damage" would also work for example.

Limitations:

"Flammability" mechanic is not yet supported and therefore not specifically addressed. Should be covered by ChaosCanIgnite flag if introduced later

Fixes #1262 .

Steps taken to verify a working solution:

  • Fire damage is converted to chaos
  • Only for fire spells
  • Chaos damage can ignite
  • Ignite deals chaos damage to enemies / is taken as chaos damage
  • Follows correct conversion priority, i.e. conversion is additive with other non-gem related conversion (I tested this in game with the keystone and "Painter's Servant" unique gloves to verify it's supposed to be that way)

Link to a build that showcases this PR:

https://maxroll.gg/poe2/pob/qczkp0q1 - Check Configuration for testing individual mod line interactions

After screenshot:

image image image image

Adds support for:
- Chaos Damage from Fire Spells Contributes to Flammability and Ignite
  Magnitudes
- Fire Spells Convert 100% of Fire Damage to Chaos Damage
- Ignite inflicted with Fire Spells deals Chaos Damage instead of Fire
  Damage

Technically also supports other elemental versions of those same mods.
"Lightning Spells convert 50% of cold damage to fire damage" would also
work for example.

Limitation:
"Flammability" mechanic is not yet supported and therefore not
specifically addressed. Should be covered by `ChaosCanIgnite` flag if
introduced later
@LocalIdentity LocalIdentity marked this pull request as ready for review September 2, 2025 13:55
@LocalIdentity LocalIdentity added the enhancement New feature, calculation, or mod label Sep 2, 2025
@LocalIdentity LocalIdentity merged commit 8ea878c 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.

Blackflame Covenant

2 participants