Git Product home page Git Product logo

xivslothcombo's People

Contributors

akineechan avatar augporto avatar chrismellor avatar codemned avatar cupcake121 avatar daemeous avatar damolitionn avatar ddd726 avatar ele-starshade avatar genesis-nova avatar gibxiv avatar grimgal avatar hazado avatar iaotle avatar jundong321 avatar k-kz avatar kagekazu avatar lunamidori5 avatar nik-potokar avatar nikkoxiv avatar nyuuuwu avatar random-stack-random-day avatar s13xiii avatar sauronshotdogs avatar shoryukenpizza avatar spazzze avatar steviecoops avatar tartarga avatar taurenkey avatar tsusai avatar

Stargazers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

Watchers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

xivslothcombo's Issues

[Suggestion] SMN Mountain Buster Feature

When Titan's Favor is up (during the Titan summon), can Gemshine turn into Mountain Buster seeing as it's a GCD into an oGCD cast?

I see the benefit of having it on Ruin III for the Titan Unique Feature but this would streamline the cast even more I feel.

Warrior issue

Hi,
Currently testing your fork in game on warrior and noticed that storm's path combo chain does not change to storm's path but to storms eye, just a heads up :)

ty for the amazing fork tho!

[NINJA] Please remove the transition of Armor Crush in Aeolian Edge combo when Huton buff below 30 sec

image

Hi Nik. picture above is my ninja setting. the current main combo logic is that Aeolian edge will be replaced with armor crush automatically when huton buff less than 30 sec,which in order to extend buff to 60 sec. I acknowledge that setting can make combo more convenient, but meanwhile it comes with two disadvantage:

  1. Armor Crush is lv54 skill. it would becomes 'invalid' when the player in dungeon which below lv54 (for example: Main Scenario Quests ). it make main combo stuck while huton buff less than 30 sec.

  2. Armor Crush and Aeolian Edge both are positional skills. it will cause potency loss when the plugin change aeolian edge to armor crush suddenly and player does not aware of the change .

Is it possible to take the armor crush out of Aeolian edge combo or make it as an individual feature (people can enable or disable by themselves) . Thanks in advance.

Fix DRK overcap?

Noticed that DRK overcap on Souleater gives me Bloodspiller straight away (on Hard Slash) instead of on the 3rd GCD (Souleater). Is this intended behavior? Wouldn't it be better to get Bloodspiller only on overcap?

Ninja Huton/Armor Crush Combo at <40s | How to change?

Hello,

I love the Ninja Huton/Armor crush combo feature. It works very well!

I've looked through your Combo files and found this on line #78

if (lastComboMove == NIN.GustSlash && level >= 45 && huton.HutonTimer < 30000)

Would it be possible to raise that timer to <4000 aka. do Armor Crush if Huton is less than 40s?

The problem, I don't know how to compile the changed file, so I could try it out for myself.
This is why I am asking you for your help.

How can I make this work in the actual Plugin, or how do I compile it after I changed the File myself?

Would be awesome if you could help me out with that.

Regards

Logic for MNK Rotation simplified

Hey, this is not exactly an issue but i would love to provide you with the logics and thought processes behind the MNK rotation to make the single button rotation work. If you're interested please let me know of a way to contact!

[GNB] WickedTalon combo not work

oGKD capacities are not used.
(More precisely, they are used incorrectly, only the last oGCD ability in a combo is used)
This works correctly in the base combo plugin. Just take the code from there =)

And another humble question, do you have any plans to combine two GNB combos into one? (Yes, I'm lazy =)

Reaper Death's design main combo

Is it possible to add shadow of death to slice combo, and shorl of death to scythe combo, so it works like warrior AIO storms path/eye, meaning once buff/debuff gets low, it gets added to combo again.

DRG AoE + Draconian Fury

Coerthan Torment combo seems to be ignoring Draconian Fire completely and making Doom Spike usable instead of Draconian Fury despite the fact that under the buff the latter replaces the former

PLD Confiteor Low Level

Before Pld gets Confiteor using requiescat results in your skill turning to Confiteor without a way to use the last stack from requiescat since you don't actually gain Confiteor till level 80. This effects leveling or lower level dungeons between 68-80. Workaround for now is keeping a separate bind of holy spirit but having a conditional set to only change your holy spirit to Confiteor after level 80 would be nice :)

Another issue with War

Noticed another issue with war. When using Infuriate to gain Nascent Chaos your main combo button/fell cleave isn't being replaced by Inner Chaos. I assume it is supposed to change since with the AOE button (overpower/tempest) either will swap into Chaotic Cyclone as soon as you pop infuriate. Its also a bit confusing since if your main combo is already on a fell cleave for not overcapping... so say you were at 50 and used infuriate to hit 100 your main combo switches to a fell cleave instead of a Inner Chaos. Which shouldn't even be on your bar since fell cleave during Nascent Chaos should automatically switch to Inner Chaos. Hope I explained this well enough.

[GitHub Actions] Need a token and branch renaming.

Aki needs to make a secret token for this repo. Refer to this commit: c3a3811

Additionally, I suggest we decide on master or main as the branch name. Currently this repo uses master, and MyDalamudPlugins uses main. I suggest master for both.

GNB bulletguage cap increase

With EW the bullet guage cap increased to 3 at level 88, so the solid barrel combo should be updated for the overcap protection to happen at 3 rather than 2.

With regards to Fixed SageDPSFeature to include Dosis at all levels...

Mind you I haven't played sage yet, I was just looking at the code.
Assuming Dosis 1, 2, and 3, are upgraded to their higher levels (1>2, 2>3)
Would it not be possible trim down a lot of the checks to just check for the status effect(s) and instead of return SGE.Eukrasia; do return OriginalHook(SGE.Eukrasia);? as that should return the original ability (or its upgrades if you're high enough level automatically)

PLD Royal Authority combo broken at <60 content

PLD Royal Authority combo breaks before lvl60. The combo should replace Royal Authority with its lower level version (Rage of Halone) in sub-60 content, but instead swaps to Royal Authority on the bar. In dungeons it's possible to hit Rage of Halone if timed correctly, however if you miss the CD for it, it will not function.

[SMN] Simple Summoner Feature

Anyone that can help test this in-game and report back here or directly to me on discord (Aki#9251)
it would be greatly appreciated.

Added a simple summoner combo similar to Samurai one but way more condensed.

Features
Everything is on one button (Currently only for Single Target).
Fester, ED, Deathflare, Enkindles, EGI Skills, Buff...

What i want to improve (Need Help):

  • Smoother oGCD Weaving solution
  • Bug Reports

What is coming:

  • AoE Version
  • Multiple EGI Summoning combinations (Order)
  • Low Level Checks

internal class SummonerMainComboFeature : CustomCombo

Any feedback is greatly appreciated!!! <3

[PLD] Requiescat Feature

In its current state, this actually is nearly useless due to Req. showing up early in our rotations now. You may want to make a check that Fight or Flight is not up as well so it doesn't get wasted.

PLD Goring Blade Combo issue.

Paladin when synced under Lv 54 keeps getting an unusable Goring Blade into the Rage of Halone/Royal Authority combo.

[GNB] Sonic break suggestion

Can we get sonic break on the main combo within no mercy instead of tied to the no mercy button. It doesn't make much sense the way it is currently since Sonic Break is a weaponskill and requires a GCD. Changing Sonic break to behave like double down would be much better. This would also clean it up more when doing dungeons and trash pulls... i wanna no mercy and AOE spam... but I keep seeing sonic break on my no mercy button which I would never use in this situation. I know I could turn it off but I do like having bow shock linked to no mercy.

Monk combo ignores True Strike

Figured the MNK single button rotation would ignore True Strike, probably due to looking for the wrong buff.
Could try encorporating a buff check for DisciplinedFist = 3001;
and return TrueStrike when true, or vice versa and go TwinSnakes when false

Zadnor Buff Limit

Fate Looks to Die For. On boss Ayida rotation stuck on Shadow of Death. Infinity reapeat Shadow of Death.

Samurai st and aoe one button test

hey, this is what I have so far for one button rotation for single target rotation on samurai. Think we could optimize?

ST:

internal class SamuraiYukikazeCombo : CustomCombo
{
    protected override CustomComboPreset Preset => CustomComboPreset.SamuraiYukikazeCombo;

    protected override uint Invoke(uint actionID, uint lastComboMove, float comboTime, byte level)
    {
        if (actionID == SAM.Yukikaze)
        {
            if (HasEffect(SAM.Buffs.MeikyoShisui))
            {
                var gauge = GetJobGauge<SAMGauge>();
                if (level >= SAM.Levels.Kasha && gauge.Sen.HasFlag(Sen.KA) == false)
                    return SAM.Kasha;
                if (level >= SAM.Levels.Gekko && gauge.Sen.HasFlag(Sen.GETSU) == false)
                    return SAM.Gekko;
                if (level >= SAM.Levels.Yukikaze && gauge.Sen.HasFlag(Sen.SETSU) == false)
                    return SAM.Yukikaze;
            }

            if (comboTime > 0 && level >= SAM.Levels.Shifu)
            {
                var gauge = GetJobGauge<SAMGauge>();
                if (lastComboMove == SAM.Hakaze)
                {
                    if (level >= SAM.Levels.Shifu && gauge.Sen.HasFlag(Sen.KA) == false && FindEffect(SAM.Buffs.Shifu)?.RemainingTime < 10)
                        return SAM.Shifu;
                    if (level >= SAM.Levels.Jinpu && gauge.Sen.HasFlag(Sen.GETSU) == false && FindEffect(SAM.Buffs.Jinpu)?.RemainingTime < 10)
                        return SAM.Jinpu;
                    if (level >= SAM.Levels.Yukikaze && gauge.Sen.HasFlag(Sen.SETSU) == false)
                        return SAM.Yukikaze;
                    if (level >= SAM.Levels.Shifu && FindEffect(SAM.Buffs.Shifu)?.RemainingTime < 10 || gauge.Sen.HasFlag(Sen.KA) == false)
                        return SAM.Shifu;
                    if (level >= SAM.Levels.Jinpu)
                        return SAM.Jinpu;
                }

                if (lastComboMove == SAM.Shifu && level >= SAM.Levels.Kasha)
                    return SAM.Kasha;
                if (lastComboMove == SAM.Jinpu && level >= SAM.Levels.Gekko)
                    return SAM.Gekko;
            }

            return SAM.Hakaze;
        }

        return actionID;
    }
}

AOE:

internal class SamuraiOkaCombo : CustomCombo
{
    protected override CustomComboPreset Preset => CustomComboPreset.SamuraiOkaCombo;

    protected override uint Invoke(uint actionID, uint lastComboMove, float comboTime, byte level)
    {
        if (actionID == SAM.Oka)
        {
            if (HasEffect(SAM.Buffs.MeikyoShisui))
                return SAM.Oka;

            if (comboTime > 0 && lastComboMove == SAM.Fuga && level >= SAM.Levels.Mangetsu)
            {
                var gauge = GetJobGauge<SAMGauge>();
                if (level >= SAM.Levels.Oka && gauge.Sen.HasFlag(Sen.KA) == false && FindEffect(SAM.Buffs.Shifu)?.RemainingTime < 10)
                    return SAM.Oka;
                if (level < SAM.Levels.Oka || gauge.Sen.HasFlag(Sen.GETSU) == false || FindEffect(SAM.Buffs.Jinpu)?.RemainingTime < FindEffect(SAM.Buffs.Shifu)?.RemainingTime)
                    return SAM.Mangetsu;
                if (level >= SAM.Levels.Oka)
                    return SAM.Oka;
            }

            return SAM.Fuga;
        }

        return actionID;
    }
}

[Suggestion] WAR Overcap

Decided to make another issue for this, for better visibility:

For Warrior, can we split up the features for Storm's Path? Having Inner Beast / Fell Cleave automatically swapped into the Storm's Path combo at 70 is a bit strange to me, should be an Overcap Feature?

I also personally prefer it to only show up when you would overcap on Beast Gauge (e.g. when you have >80 gauge for Storm's Path or >90 gauge for Maim), since I might be saving up my gauge for Berserk/Inner Release window.

[PLD] Circle of Scorn+Spirits/Expiacion

There's a feature in Daemitus' fork that puts these on one button and just uses whichever isn't on CD. Would recommend incorporating them into this addon too.

[Suggestion] A couple of suggestions.

  1. Dark Knight:
  • Would it be possible to apply the overcap feature to dark knight's single target as well, similar to warrior and gunbreaker? There was only one other plugin that did this but it is no longer updated.
  1. Warrior
  • Changing your 1-2-3 to use Storm's Eye if it is missing is a great idea. The only issue right now is that it seems to only trigger if Storm's Eye is not active, which can lead to situations where it falls of for several GCDs. An alternative would be to check if the duration of Storm's Eye is 15 seconds or less similar to ninja.
  • The warrior overcap feature seems to trigger too early, as it triggers fell cleave as soon as you have 50 rage. A cut off of 70 rage or greater would be better as that would ensure you always have 20 rage available for Uphheaval

Thank you!

[WAR] Warrior AOE Combo and related function is not working

Hi Nik-Potokar

Thanks for you great plugin. Please take a look below function in Warrior? those features are not working on lv90 warrior . I am using the japanese client.

145: Mythril tempest combo
151: primal rend feature
152 Orogeny Feature

I assumed that those problems should only happen to me. So I installed the original XIV Combo plugin. AOE combo and Primal Rend feature are working well. but they are not working when I change back to your plugin.

Thank you for your patience with me

Clarification on naming and repo

Since ?yesterday I started seeing the repo getting updated, but the updates not happening in game. Now I notice a "new" mod in the repo called XIVSlothCombo, akso by aki, but with an updated version number.

Am I to believe that this new one, is the same, but with a namechange, and the old "XIVComboWayTooAkiiified" is deprecated?

When changes are made to the XIVComboWayTooAkiiified, when can we expect the changed to be shipped to MyDalamudPlugins where they appear in game?

If I have misunderstood anything, feel free to correct me. Much appreciate the effort put in here.

Warrior Upheaval

Warrior is using upheaval in opener before inner release or storms eye. It is set to only use it during IR or while inner release is on CD but it uses it on cd so it gets popped instantly in opener before you can apply storms eye.

Fix Raise feature

SCH and WHM raise features are polar opposites - why? Homogenize them so they work the same way, or rather just make it one tweak for all Healers + SMN.

In my opinion - Swiftcast should always change into Raise/Resurrection/Verraise/Ascend/Egeiro.

Appreciation and RDM suggestion.

Hi,

First of all, I wish to express my greatest appreciation to your work here. Greatly improved my enjoyment of the game. Thank you.

I am posting a suggestion as I cannot find a place to post in your suggested link. My fault here because this is not an issue.

I was wondering if you can add rdm's new skill (Resolution) to redoublement combo plus.

Thanks again for the wonderful job you are doing.

Name change

I like the name. We got the cool icon as well. SlothCombo sounds kinda lame to me. Maybe XIVComboWayTooAfkiiified?

[WAR] Can't have a separate storms path combo

Its currently not possible to have storms path and storms eye combos be separated, instead of the all-in-one button. being able to separate them and still have the overcap feature work would be nice

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    ๐Ÿ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. ๐Ÿ“Š๐Ÿ“ˆ๐ŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google โค๏ธ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.