Skip to content

raidboss: DMU P3-P5 Timeline Updates and Voicelines#1091

Open
Legends0 wants to merge 50 commits into
mainfrom
p3-timeline-more-updates
Open

raidboss: DMU P3-P5 Timeline Updates and Voicelines#1091
Legends0 wants to merge 50 commits into
mainfrom
p3-timeline-more-updates

Conversation

@Legends0

@Legends0 Legends0 commented Jun 20, 2026

Copy link
Copy Markdown
Collaborator

Places BAEA Shocking Impact and BAEB Shockwave on same line. This could later be branched out.
Adds ignore for BCCD Black Spark, triggered by standing in a black hole.
C4BA changed to sync. This seems to be a mismatch of what fflogs was naming it to what it is in the network log.
Timing adjustments up to end of P4 and into Celestriad of P5.

Adds voiceline syncs for all phases and P1 => P2 fake/true ending.

Cleanup some failure related ids in P4, update the ids with spell variances in P4.
P5 Flood cleanup, Celestriad tower times.

@github-actions github-actions Bot added raidboss /ui/raidboss module needs-review Awaiting review labels Jun 20, 2026
@Legends0 Legends0 changed the title raidboss: DMU P3 Timeline Updates raidboss: DMU P3 Timeline Updates and Voicelines Jun 25, 2026
@Legends0 Legends0 mentioned this pull request Jul 10, 2026
20 tasks
@Legends0 Legends0 changed the title raidboss: DMU P3 Timeline Updates and Voicelines raidboss: DMU P3-P4 Timeline Updates and Voicelines Jul 12, 2026
@Legends0 Legends0 changed the title raidboss: DMU P3-P4 Timeline Updates and Voicelines raidboss: DMU P3-P5 Timeline Updates and Voicelines Jul 24, 2026
@Legends0
Legends0 requested a review from valarnin July 25, 2026 02:04
@Legends0

Copy link
Copy Markdown
Collaborator Author

This may be something to merge in before a P3/P4 are in the next release?
It will have network-based timings up to P5, and better transition syncs.

I'll be able to finish up the P5 portion once I have completed the P5 triggers.

Comment thread ui/raidboss/data/07-dt/ultimate/dancing_mad.txt Outdated
Comment thread ui/raidboss/data/07-dt/ultimate/dancing_mad.txt Outdated
1446.5 "Fell Forces 2" Ability { id: ["C654", "C653", "C654"], source: "Kefka" }

1452.9 "Celestriad" Ability { id: "BB42", source: "Kefka" }
1455.9 "--1 towers--" ActorControlExtra { category: "019D", param1: "10", param2: "20" }

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
1455.9 "--1 towers--" ActorControlExtra { category: "019D", param1: "10", param2: "20" }
1455.9 "--1 tower--" ActorControlExtra { category: "019D", param1: "10", param2: "20" }

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 wasn't certain of how this should be labelled there are 4 towers that spawn each time. The label is more for sets than the tower count.

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've adjusted it to be 1st tower, 2nd tower, 3rd tower. Alternatively, tower 1, tower 2, tower 3 could also work.

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

Labels

needs-review Awaiting review raidboss /ui/raidboss module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants