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

[DRK] Simple Modes, In-Rotation Mit expansion, Rework #320

Merged
merged 36 commits into from
Feb 8, 2025

Conversation

zbee
Copy link
Member

@zbee zbee commented Jan 27, 2025

Tip

See this discord post for a more visual list of what is being added in this PR, and for any feedback about this PR.

Log Snippet comparisons:

  • Rework most action logic into TryGet methods like RDM uses

    • oGCDs
    • GCDs
      • Blood Overcap
      • Delirium Combo
      • Pre-Burst Blood maintenance
      • Blood Burst usage (low level)
      • 1-2-3 combos
  • Expand in-rotation mitigation options

    • Oblation
    • Reprisal
    • Dark Missionary
    • Rampart
    • Arms Length
  • Add Simple Modes

    • Single-Target
    • AoE
  • Fix [DRK] advanced mode options #89, Fix [DRK] quietus on advanced mode delirium #205

    • Separate Blood abilities/Delirium combo from Delirium
    • Make sure oGCD feature is up to date
    • Add slider for AoE Blood overcap option
      • And for the ST overcap option
  • Reorganize all ST and AoE presets

  • Reset DRK options

  • Add HP% sliders

@zbee zbee added Optimization Similar to suggestion, but solely for increasing performance New Feature This adds a new option or feature Fixes This will fix a current issue Adjustments This will adjust something inside a rotation, feature, or core function labels Jan 27, 2025
@PunishXIV PunishXIV deleted a comment from semanticdiff-com bot Jan 27, 2025
@moreime
Copy link

moreime commented Feb 2, 2025

would it be possible to separate shadowbringers from disesteem as well?

@zbee
Copy link
Member Author

zbee commented Feb 2, 2025

@moreime I'm fairly certain it never was, but it certainly is detached now, yes.

@moreime
Copy link

moreime commented Feb 2, 2025

it was when i tried it a few weeks ago, i disabled disesteem option and it wouldnt use shadowbringers until disesteem was used. but thankies, great work on drk in general

@zbee
Copy link
Member Author

zbee commented Feb 3, 2025

@moreime this is effectively already done with the new Shadowbringer logic. There was a bug with it before.

However, the Shadowbringer Burst Option will still wait for Disesteem to be used before using Shadowbringer.
There's not a good way to change this logic, without overengineering it, I believe, so it will stay that way, but I will make that more clear.

@zbee zbee marked this pull request as ready for review February 4, 2025 06:28
@zbee
Copy link
Member Author

zbee commented Feb 4, 2025

@Kagekazu would appreciate your review, but mostly pinging you because I think you might like to pull this branch and see how I made DRK similar to RDM, with all the logic moved out of the actual rotations to be easier to maintain (hopefully).

For example, on the left is a preview of the new Simple Mode, and the right is the code for "cooldown" actions:

@Kagekazu
Copy link
Member

Kagekazu commented Feb 4, 2025

@Kagekazu would appreciate your review, but mostly pinging you because I think you might like to pull this branch and see how I made DRK similar to RDM, with all the logic moved out of the actual rotations to be easier to maintain (hopefully).

I will have a look, thanks!

@zbee zbee marked this pull request as draft February 4, 2025 22:41
@zbee zbee marked this pull request as ready for review February 5, 2025 22:42
@Taurenkey Taurenkey merged commit 65f552e into PunishXIV:main Feb 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Adjustments This will adjust something inside a rotation, feature, or core function Fixes This will fix a current issue New Feature This adds a new option or feature Optimization Similar to suggestion, but solely for increasing performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DRK] quietus on advanced mode delirium [DRK] advanced mode options
4 participants