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

Add BioMove option, update logic, and enhance debug UI #614

Merged
merged 1 commit into from
Feb 9, 2025
Merged

Conversation

LTS-FFXIV
Copy link
Member

This pull request includes several changes to the BasicRotations and RotationSolver modules to enhance functionality and improve the user interface. The most important changes are summarized below:

Enhancements to BasicRotations:

  • Added a new configuration option BioMove to MCH_Default and MCH_HighEnd classes to allow the use of Bioblaster while moving. (BasicRotations/Ranged/MCH_Default.cs, [1]; BasicRotations/Ranged/MCH_HighEnd.cs, [2]
  • Updated the GeneralGCD method in both MCH_Default and MCH_HighEnd classes to respect the new BioMove configuration option. (BasicRotations/Ranged/MCH_Default.cs, [1]; BasicRotations/Ranged/MCH_HighEnd.cs, [2]

Improvements to RotationSolver UI:

  • Enhanced the DrawStatus method in RotationConfigWindow.cs to display the stack count of status effects, converting a stack value of 255 to "N/A" for better readability. (RotationSolver/UI/RotationConfigWindow.cs, RotationSolver/UI/RotationConfigWindow.csL2817-R2819)

Submodule Update:

  • Updated the ECommons submodule to a new commit. (ECommons, ECommonsL1-R1)

Added a new configuration option `BioMove` to the `MCH_Default` and `MCH_HighEnd` classes, allowing Bioblaster use while moving. Updated logic to check `BioMove` setting before using Bioblaster, ensuring it can be used if `BioMove` is enabled or if the player is not moving. Updated subproject commit reference in `ECommons` to `a1df6434ddac33eab306b3f4f605f59f0d0a7279`. Enhanced `RotationConfigWindow` to display stack count of statuses.
@LTS-FFXIV LTS-FFXIV merged commit 439a445 into main Feb 9, 2025
@LTS-FFXIV LTS-FFXIV deleted the MCHBSfix branch February 24, 2025 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant