Ivaar / Skillchains

Active Battle Skillchain Display FFXI Windower Lua addon.
42 stars 19 forks source link

Jumps don't skillchain #13

Closed klikevil closed 4 years ago

klikevil commented 4 years ago

The plugin is counting jumps as a weaponskill and says you can close with relic ws or cam torment... You can't

Ivaar commented 4 years ago

This was likely caused by junk added effect data in the action packet and should now be fixed. I noticed this in my data collection logs and tried to reproduce it over a week ago. Since it was caused by a random passerby and I wasn't recorded packets, I could not verify what I needed. Can you confirm your version number and see if it still occurs in the latest version?

klikevil commented 4 years ago

Current:


Administrator@period ~/Administrator/addons/Skillchains
$ grep -i version Skillchains.lua 
_addon.version = '2.20.08.08'
Administrator@period ~/Administrator/addons/Skillchains
$ git log --color=always|& head
commit dd7a7c084be7e99c713c8ecfb9a0f8a78410451e
Author: Ivaar <7287342+Ivaar@users.noreply.github.com>
Date:   Sat Aug 15 04:15:30 2020 -0700

    Skillchains: all trust attributes verified, blue spells amended

Just pulled the most recent


Administrator@period ~/Administrator/addons/Skillchains
$ git pull
remote: Enumerating objects: 9, done.
remote: Counting objects: 100% (9/9), done.
remote: Compressing objects: 100% (3/3), done.
remote: Total 6 (delta 3), reused 6 (delta 3), pack-reused 0
Unpacking objects: 100% (6/6), done.
From https://github.com/Ivaar/Skillchains
   dd7a7c0..7d68a2c  master     -> origin/master
Updating dd7a7c0..7d68a2c
Fast-forward
 Skillchains.lua |  4 ++--
 skills.lua      | 49 +++++++++++++++++++++----------------------------
 2 files changed, 23 insertions(+), 30 deletions(-)

Will test with this one now:


Administrator@period ~/Administrator/addons/Skillchains
$ !grep
grep -i version Skillchains.lua 
_addon.version = '2.20.08.17'
klikevil commented 4 years ago

It is not fixed in this most recent version

Ivaar commented 4 years ago

You mentioned relic weaponskill, does it happen without Gugnir equipped? Can you give any additional information/steps to reproduce?

Ivaar commented 4 years ago

If you're able to get a packet capture, it would really help. Load packetviewer and use this tracking command //pv t i 0x28 Category 3;pv rec perform a jump and screenshot it.

klikevil commented 4 years ago

img_20200818_230631 Hope this is good enough, appears to be caused by relic reforge body +3 that does additional damage 100% of wyvern's hp; still occurs with relic weapon unequipped but only shows camlann's torment seems to unequivocally be caused by this Ptero. Mail +3's additional effect appearing like a weapon skill I suppose?

Ivaar commented 4 years ago

That rules out junk effects, which should all be filtered. I couldn't find anything on the wikis that gave jump an added effect and actually displayed a message. Thank you for verifying that.

klikevil commented 4 years ago

Line 365 displays a runtime error when i perform jumps now saying it got a null value and sometimes it is ignoring weaponskills after the first one. This is the link to the armor Not sure if I should make a new issue for that or leave this one open because it's likely still related due to the runtime error in skillchain_ids

img_20200819_100056

Ivaar commented 4 years ago

Pushed a fix, was a late night typo. Sorry for any inconvenience.

klikevil commented 4 years ago

That got it, and no problem I appreciate your hard work and quick turn around time, thanks!