Skip to content

Berzerk / Psychedelic duration stacking customization - #2191

Open
Starkku wants to merge 6 commits into
Phobos-developers:developfrom
Starkku:feature/psychedelic-stacking
Open

Berzerk / Psychedelic duration stacking customization#2191
Starkku wants to merge 6 commits into
Phobos-developers:developfrom
Starkku:feature/psychedelic-stacking

Conversation

@Starkku

@Starkku Starkku commented May 3, 2026

Copy link
Copy Markdown
Contributor

Berzerk (Psychedelic) duration stacking customization

  • By default Psychedelic warheads override the current duration of the berzerk effect regardless of if the new duration is higher or lower than the current one. This can now be customized with Psychedelic.StackingMode, with both global setting under [CombatDamage] and per-Warhead customization.

In rulesmd.ini:

[CombatDamage]
Psychedelic.StackingMode=override  ; Stacking mode enum (override|setifzero|min|max|add|subtract|multiply|divide)

[SOMEWARHEAD]                      ; WarheadType
Psychedelic.StackingMode=          ; Stacking mode enum (override|setifzero|min|max|add|subtract|multiply|divide)

Dev note: The enum was generalized for potential (future) use elsewhere but is also possibly over-engineered for these kind of purposes and mixes two things together (when and how) which might not be ideal, can simplify it if need be I suppose.

@github-actions

github-actions Bot commented May 3, 2026

Copy link
Copy Markdown

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@DeathFishAtEase DeathFishAtEase added Needs testing ⚙️T1 T1 maintainer review is sufficient ❓Unhardcoding / Customization Make something more tweakable labels May 3, 2026
@Starkku
Starkku force-pushed the feature/psychedelic-stacking branch from a15f909 to 4ab5de1 Compare May 8, 2026 13:12
@Coronia

Coronia commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

'add' option doesn't work in our test for some reason, which still behaves like override

@Starkku
Starkku force-pushed the feature/psychedelic-stacking branch 2 times, most recently from 612bfbd to d8e5761 Compare July 30, 2026 11:59
@fagonghaiwo

Copy link
Copy Markdown

all are override

@Starkku
Starkku force-pushed the feature/psychedelic-stacking branch from d8e5761 to d35a77c Compare July 31, 2026 14:15
@Starkku

Starkku commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

Fixed the [CombatDamage] default not being correctly applied.

@fagonghaiwo

Copy link
Copy Markdown

works, but min|subtract|multiply|divide need unit been Berzerk, maybe shouldnt in [CombatDamage]

@Coronia

Coronia commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

works, but min|subtract|multiply|divide need unit been Berzerk, maybe shouldnt in [CombatDamage]

this case should be mentioned in the doc that it'll make Berzerk can't be applied normally if the global tag is set to these values

Coronia added 2 commits August 3, 2026 10:02
# Conflicts:
#	CREDITS.md
#	docs/Whats-New.md
@Coronia

Coronia commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

there's still one remaining thing: even if we use these stacking values to make the duration of berzerk becomes 0, it'll still retint for a frame. This make it look bad when we want to use it to clear berzerk effect

@Coronia

Coronia commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

there's still one remaining thing: even if we use these stacking values to make the duration of berzerk becomes 0, it'll still retint for a frame. This make it look bad when we want to use it to clear berzerk effect

done and tested to work, and since 0 Damage can't make it enter berzerk already, I made it simply bail out if the duration is not positive. I don't think a customizable tag is needed here but correct me if I'm wrong

@Coronia Coronia added the Will be merged in 24h This PR will be merged in 24 hours if no one has further instructions. label Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚙️T1 T1 maintainer review is sufficient Tested ❓Unhardcoding / Customization Make something more tweakable Will be merged in 24h This PR will be merged in 24 hours if no one has further instructions.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants