Skip to content

raidboss: DMU P5 initial triggers - #1105

Open
Legends0 wants to merge 60 commits into
mainfrom
p5-initial-triggers
Open

Legends0 wants to merge 60 commits into
mainfrom
p5-initial-triggers

Conversation

@Legends0

@Legends0 Legends0 commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

This PR adds triggers for P5 mechanics:

  • Ultima Repeater AoEs
  • Fell Forces Role Stacks
  • Flood Start Location and Rotation
  • Maddening Orchestra Triggers
  • Celestriad Debuff and Tower Triggers, including initial tower direction, this is also configurable if strategy is CW or CCW from the vulnerable tower
  • Catastrophic Choice In/Out Trigger
  • Exaflare locations (Contains safe rows, but player may want to alter output to be shorter as it lasts 3.1s)
  • Forsaken AoEs and Forsaken Bonds Stack with number

@Legends0 Legends0 mentioned this pull request Jul 10, 2026
20 tasks
@github-actions github-actions Bot added the resources /resources label Jul 21, 2026
@github-actions github-actions Bot removed the resources /resources label Jul 23, 2026
Legends0 added 7 commits July 27, 2026 07:35
For the players that have a debuff, we can call the general area ~2s prior
Wrapper outputString provided that would let player alter string based on debuff.
@Legends0

Legends0 commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator Author

I'm not sure how helpful the Exa trigger location can be. This mechanic may be too fast for the output to be meaningful and there may not be a predictable pattern to merge every other one. Also the output could remove NW/NE and instead use something like a number count as odds are always nw and evens are always ne. I guess ideally to make any actual use of it, the player would need to override the pattern matching with the markers they are using.

was seeing emulator skip first one and timings be off in emulator
@Legends0

Legends0 commented Aug 15, 2026

Copy link
Copy Markdown
Collaborator Author

Example outputs (this is with the P4/P5 ultima upsurge split) from the other PR #1117:
Tank:
image

Healer:
image

DPS:
image

@Legends0
Legends0 marked this pull request as ready for review August 15, 2026 02:20
@github-actions github-actions Bot added the needs-review Awaiting review label Aug 15, 2026
Comment thread ui/raidboss/data/07-dt/ultimate/dancing_mad.ts Outdated
Comment thread ui/raidboss/data/07-dt/ultimate/dancing_mad.ts
Comment thread ui/raidboss/data/07-dt/ultimate/dancing_mad.ts
Comment thread ui/raidboss/data/07-dt/ultimate/dancing_mad.ts Outdated
@Legends0

Legends0 commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator Author

ah, that comment is actually just wrong for Clockwise Example 2, I see it's just a copy of the x,y from previous line so will need to go back and check it.

EDIT 2: I see there is also another typo in Counterclockwise Example.

@Dnguye1393

Copy link
Copy Markdown

Is there an update to when this could be merged, if the issues are resolved?

@Legends0
Legends0 requested a review from valarnin September 8, 2026 15:22

@Legends0 Legends0 left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I reviewed the requested changes.

The ternary operator in Fell Forces trigger was changed to if statements.

The triggers for Flood needed more than heading data and I found the x, y data to not be correct in the StartsUsingExtra. However, in my review of it I realized the second output could be updated to be more accurate than just the intercardinal start and so it now outputs the cardinal position which will be closer to the second flood and direction to move. This change matches the kefkabin guide for starting position.

The comments on Suprise Flare/Holy have been corrected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants