Skip to content
This repository has been archived by the owner on Nov 23, 2024. It is now read-only.

Dancer Optimalizations #227

Merged
merged 3 commits into from
Sep 15, 2024
Merged

Dancer Optimalizations #227

merged 3 commits into from
Sep 15, 2024

Conversation

Four-amer
Copy link
Contributor

Won't use abilities right before dancing or while dancing because their animation lock might delay the dance, happened to me with curing waltz.

Sometimes it used FanDance II or FanDance when FanDance III was available, now it shouldn't.
However it's hard to test since it happens rarely, but i didn't notice a single instance of it happening for 2 hours so its probably working as intented now.

The anti overcapping mechanic for feathers now uses feathers before nextGCD that can generate it, instead of instantly when it hits 4, also moved the storing feathers mechanic to when the player hits level high enough for devilment instead of tech step.

It seems that the code that prevents using standard step when it would delay tech step didn't work properly. Fixed that

Tested on Dancer Lv 72 and 60.

Also FYI my discord username is Fouramer .
Happy to contribute!

Shouldn't use abilities that their animation lock can drift dancing right before dance and during it, Just like Curing Waltz

Sometimes it used FanDance II or FanDance when FanDance III was available, now it doesn't.
However it hard to test since it happens rarely but i didn't notice a single instance of it happening for 2 hours so its probably working as intented

The overcapping mechanic for feathers now uses feathers before nextGCD that can generate it, instead of instantly when it hits 4, also moved the storing feathers mechanic to when the player hits level anough for devilment instead of tech step.

It seems that the code that prevented using standard step when technical step cd is less that time required to do the dance didn't work properly. Fixed that
TIL that WillHaveOneCharge returns true if the ability is not avaible to use
@LTS-FFXIV LTS-FFXIV merged commit dec56fb into FFXIV-CombatReborn:main Sep 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants