Skip to content

Medical - Add configurable armor pain system#303

Open
fedoraman737 wants to merge 4 commits intoacemod:masterfrom
fedoraman737:armorPainSystem
Open

Medical - Add configurable armor pain system#303
fedoraman737 wants to merge 4 commits intoacemod:masterfrom
fedoraman737:armorPainSystem

Conversation

@fedoraman737
Copy link
Contributor

When merged this pull request will:

  • This a near duplicate of Medical - Add configurable armor pain system with threshold #266 due to merge conflicts
  • Adds pain effects when armor stops incoming kinetic/melee damage
    • First significant armor hit (above threshold) sets pain to MODERATE + additional damage based on multiplier
    • Subsequent hits while already in pain (at/beyond MODERATE) add no additional pain
    • Per maintainer's feedback: prevents armor spam from continuously increasing pain
  • Adds configurable thresholds and modifiers for damage to pain hitzone from ArmorHitEvent()

Question... should first hit be capped at exactly MODERATE, or allowed to exceed based on impact severity (current implementation)?

Note: the configuration is at the prefab level.

@Kexanone Kexanone added the kind/feature Release Notes: **ADDED:** label Nov 9, 2025
@Kexanone Kexanone added this to the 1.5.0 milestone Nov 9, 2025
@Kexanone
Copy link
Member

Kexanone commented Nov 10, 2025

Question... should first hit be capped at exactly MODERATE, or allowed to exceed based on impact severity (current implementation)?

I think the implementation is fine, but I forgot that MODERATE (=0.8 scaled health) is exactly the lowest possible pain threshold.
I would propose introducing a custom attribute as threshold instead and set the default to something like 0.7 scaled health.

@Kexanone Kexanone added branch/dev Available on dev branch for preview and removed branch/dev Available on dev branch for preview labels Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feature Release Notes: **ADDED:**

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants