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

GNB and return data.role == 'tank' not working #453

Closed
AmenneHolelane opened this issue Jul 10, 2019 · 3 comments
Closed

GNB and return data.role == 'tank' not working #453

AmenneHolelane opened this issue Jul 10, 2019 · 3 comments

Comments

@AmenneHolelane
Copy link
Contributor

hello, i was running inno EX and the tank specific triggers do not seem to be triggering when i was on GNB. log is attached
Network_20190709.log

@AmenneHolelane
Copy link
Contributor Author

upon further testing, using the test.HTML, i'm reported as being on job 60 GNB when on Dancer and job 80 DNC when on GNB

@KhloeAliapoh
Copy link

From what i noticed in FFXIVProcess.cs line 121 and 122 the job IDs for Dnc and Gnb are swapped, Gnb should be 37 and Dnc should be 38, not the other way around.

@quisquous
Copy link
Owner

WHOOPS. Thanks for this catch. I swear I looked at the tables from Saint Coinach to write this, but I should have tested this properly. Sorry!

There's not really a way for me to fix this until I get back this next weekend, but I'll put out a new release then.

@quisquous quisquous mentioned this issue Jul 15, 2019
SiliconExarch pushed a commit to SiliconExarch/cactbot that referenced this issue Oct 31, 2024
This series includes several strategy options for the m4s fight that fit
the
standard NA PF strategies.

- **raidboss: add support for DN strategy for Ion Cluster in m4s**
- **raidboss: add support for DN strategy during Witch Hunt in m4s**
- **raidboss: add uptime sunrise sabaath callouts in m4s**

These are based off the shabin pastebin and hector guide, with the
uptime
waymarks strategy for sunrise sabaath.

I made all of these configuration options, so that existing default
callouts
which do not use a specific strategy are not affected. I tried to put
good
brief explanations, and used the standard strat names I see going
around.

I've tested variations of this code in my runs, though it did not use a
strategy option at the time. Thus, this exact code is not yet battle
tested
outside of the simulator.

---------

Signed-off-by: Jacob Keller <[email protected]>
github-actions bot pushed a commit to SiliconExarch/cactbot that referenced this issue Oct 31, 2024
…us#453)

This series includes several strategy options for the m4s fight that fit
the
standard NA PF strategies.

- **raidboss: add support for DN strategy for Ion Cluster in m4s**
- **raidboss: add support for DN strategy during Witch Hunt in m4s**
- **raidboss: add uptime sunrise sabaath callouts in m4s**

These are based off the shabin pastebin and hector guide, with the
uptime
waymarks strategy for sunrise sabaath.

I made all of these configuration options, so that existing default
callouts
which do not use a specific strategy are not affected. I tried to put
good
brief explanations, and used the standard strat names I see going
around.

I've tested variations of this code in my runs, though it did not use a
strategy option at the time. Thus, this exact code is not yet battle
tested
outside of the simulator.

---------

Signed-off-by: Jacob Keller <[email protected]> 2d8a434
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

No branches or pull requests

3 participants