search

OverlayPlugin / cactbot

FFXIV TypeScript Raiding Overlay
Apache License 2.0
85 stars 34 forks source link

raidboss: R3S Tag Team 1/2, KB towers 2 lariat combo #335

Closed valarnin closed 3 weeks ago

valarnin commented 1 month ago

Removed the phase check on R3S Murderous Mist as the later instance of this ability has a different ID.

I was only able to verify the '9AEA', '9AEB' IDs for Lariat Combo via VOD, because FF14 RNG is like that. I had a single pull that saw 9AE8 but that pull wiped before the second lariat so I couldn't verify the safe spots. (More FF14 RNG fun: Every single pull that got to this point in the fight, checking the VODs, had the boss jump to south. Except one pull where the boss jumped east. I didn't even realize the boss could jump somewhere else until I was reviewing the VODs.)

For TT1/TT2, there are separate IDs for TT1 vs TT2. I was able to verify all four IDs against VODs.

wexxlee commented 1 month ago

Sorry, busy weekend and haven't had a chance to look at this yet. Will check this against some vod/logs tomorrow.

valarnin commented 1 month ago

Tested in-instance for a while tonight. TT1 and TT2 seem to be good. KB towers lariat needs some debugging still.

On Mon, Aug 12, 2024, 00:47 Wexx @.***> wrote:

Sorry, busy weekend and haven't had a chance to look at this yet. Will check this against some vod/logs tomorrow.

— Reply to this email directly, view it on GitHub https://github.com/OverlayPlugin/cactbot/pull/335#issuecomment-2283106392, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABOWBLULYOKFVRXXJJA4MGLZRA47JAVCNFSM6AAAAABMJDPUPKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBTGEYDMMZZGI . You are receiving this because you authored the thread.Message ID: @.***>

JLGarber commented 4 weeks ago

My group will be in there tomorrow. I'll try to remember to test this when we run and see whether I can add anything on the knockback Lariat.

JLGarber commented 3 weeks ago

Yup, nothing further for me to add aside from agreeing with wexxlee's latest comments.

valarnin commented 3 weeks ago

Rebased to main to fix merge issue.