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

raidboss: Increase Alerts for Getting Orbs in E5S #1115

Merged
merged 5 commits into from
Feb 27, 2020

Conversation

panicstevenson
Copy link
Contributor

Raise infoText level alerts for getting an orb to alertText if you don't
already have one.

@panicstevenson
Copy link
Contributor Author

Since we already have working triggers, this pull request is meant to be more of a working proposal.

Raise infoText level alerts for getting an orb to alertText if you don't
already have one.
@valarnin
Copy link
Collaborator

Orbs aren't required for chain lightning. They make it easier, but 100% not needed. Not sure if it makes sense to raise the alert level for orbs when they're an optional mitigation for the mechanic?

@Jaehyuk-Lee
Copy link
Contributor

If you grab an orb on chain lightning, you will have less orbs on field that could be bad for party members.

@panicstevenson
Copy link
Contributor Author

Wait, really? I always thought it was necessary. I can remove that then.

}
return {
en: 'Big Knockback',
};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E5S Centaur\'s Charge is always Big Knockback.
It happens only after fury's fourteen.
I think you were thinking about the add pushing party members from the center to sides, which is completely not dealt with e5s trigger now.
Think of that, tank moves Ramuh during first pushing.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Understood; yea, I had those confused then.

@panicstevenson
Copy link
Contributor Author

I wonder if it makes more sense to instead have a timelineTrigger for Stepped Leader that specifies that it's next, and on-cast wipe that information.

@Jaehyuk-Lee
Copy link
Contributor

Jaehyuk-Lee commented Feb 26, 2020

I agree with making timeline trigger with Stepped Leader, too. I made it Trigger because there were no timeline when I was making this. + I don't know how to make timeline. 🤣

@Jaehyuk-Lee
Copy link
Contributor

I think

data.furysBoltEnum = {
          1: 'Judgment Volts',
          2: 'Tribunal Summons',
          3: 'Thunderstorm',
          4: 'Stepped Leader',
          5: 'Thunderstorm',
          6: 'Judgment Volts',
          7: 'Judgment Volts',
          8: 'Judgment Volts',
        };

is better to run only once, by using Timeline Trigger.
It's not a big deal, but just.

@valarnin
Copy link
Collaborator

Wait, really? I always thought it was necessary. I can remove that then.

Yeah, for chain lightning the orbs reduce the damage taken by the lightning pass by I think 25% or something? It's really not needed unless the group has no coordination, the only risky parts are the tankbuster (just shield it) and the raidwide (reprisal+any two other mitigations, e.g. temperance/shields/soil/etc). I've cleared e5s twice now and both times we just didn't bother with orbs at all unless they were directly in the way of positioning for chain lightning.

Aside from the offtank add kite, Orbs are important for the following mechanics:

  • Judgement Volts (only if fury bolt is up)
  • Stormcloud Summons (melee needs orb while clearing debuff, orb always required)
  • Tribunal Summons/whatever the charge ability is (only if fury bolt is up)
  • Thunderstorm (orb always required for blue lightning aoe)
  • Fury's Fourteen (orb always required for knockback)

That's all of the mechs that require orb I think, unless I'm forgetting one.

Since we're on the subject of orbs though, I wonder if anything can be done for offtank orb count? Can we detect main tank, and have off tank's get orb text be dependent on the max stack size of the rest of the party (e.g. 6 people have 1 stack, 1 person has 2, off tank has 2, so off tank gets red text to grab extra orb)? Might be a bit beyond the scope here, I'm not sure.

@panicstevenson
Copy link
Contributor Author

I wonder if anything can be done for offtank orb count?

You can track all of the gains, the Fury's Bolts, and any losses. Unfortunately, since 'gains effect' only has the name of the status, they're all Surge Protection (unless you were to go over). Does Raiden's tank buster remove a stack?

run: function(data) {
data.fury = false;
data.surgeProtection = false;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do these effect messages only show up when you go from 0 to 1? Is there any message for more than that? Also half wondering if pro level triggers should look at times to figure out if this will run out before the mechanic you need it for too...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The gain messages show up on every orb pickup.
The loss messages show up every time you lose the entirety of the buff or if you overcharge.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah interesting, so if we wanted to track timers, you could just reset the timer on pickup.

@valarnin
Copy link
Collaborator

Does Raiden's tank buster remove a stack?

If I remember correctly, no. It shouldn't matter either way, IIRC, based on how the fight is laid out the offtank is going to require the extra stack to ensure the add is on them, and there's never a case where two mechanics requiring it would hit before the buff wears off.

@quisquous quisquous merged commit 71935ac into quisquous:master Feb 27, 2020
@quisquous
Copy link
Owner

I used this during reclears tonight and it was great! Thanks!

@panicstevenson panicstevenson deleted the e5s-alerts branch February 27, 2020 16:34
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.

4 participants