Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MNK] Adv. Rota still using solar after even minute phantom rush #189

Closed
Whitemoose78 opened this issue Dec 12, 2024 · 4 comments · Fixed by #192
Closed

[MNK] Adv. Rota still using solar after even minute phantom rush #189

Whitemoose78 opened this issue Dec 12, 2024 · 4 comments · Fixed by #192
Assignees
Labels
Fixes This will fix a current issue

Comments

@Whitemoose78
Copy link

Whitemoose78 commented Dec 12, 2024

Supposedly this issue (same as #174, solar nadi being built after phantom rush in buffs) was fixed, but the behaviour is still the same. Tried it with adv rota with everything ticked, as well as my personal setup (manual PB/buffs), still a solar nadi is being built in both cases.
image
I'm sorry in advance Kage xD

@Kagekazu
Copy link
Member

Got a log where it happens?
I'll doublecheck on my end aswell

@Whitemoose78
Copy link
Author

Damage logging got messed up, but event/cast log is fine.
https://www.fflogs.com/reports/a:GWCPrAMnV1g4T8wB?fight=1&source=4&type=casts&view=events
Longer log of the 2 is all boxes of the adv. rota ticked. Shorter my manual setup.

@Kagekazu
Copy link
Member

image

On my testing build it works, so probably once the new opener system is in, it works how it should!

@Kagekazu Kagekazu self-assigned this Dec 12, 2024
@zbee zbee changed the title [MNK] Adv. Rota still using solar after even minute phantom rush (repeat of issue 174) [MNK] Adv. Rota still using solar after even minute phantom rush Dec 12, 2024
@zbee zbee added the Fixes This will fix a current issue label Dec 12, 2024
@Kagekazu
Copy link
Member

Found a fix with the old openers, will PR it soon

@Kagekazu Kagekazu mentioned this issue Dec 13, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixes This will fix a current issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants