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

Additional Effects on weapons missing or not implemented properly #1487

Closed
SBC-Nyxai opened this issue Apr 7, 2019 · 28 comments
Closed

Additional Effects on weapons missing or not implemented properly #1487

SBC-Nyxai opened this issue Apr 7, 2019 · 28 comments
Assignees

Comments

@SBC-Nyxai
Copy link

I've been using a Stun Kukri and Bloody Blade for some time now across a range of different mob difficulties (too weak to VT). Neither of these weapons have ever proc'd their additional effect. When a bloody bolt proc's even for 0 hit points there is still an animation and a text message saying it did so. Bloody Blade has never given an animation or text for any HP drain. Additionally, the Stun Kukri has simply never proc'd. I know some lower level added fire damage procs do work but seeing as these two weapons have never worked for me I'm concerned that some of the higher level added effect weapons do not work at all.

@Tjonke
Copy link

Tjonke commented Apr 8, 2019

It's the same for ALL the elemental staves, they just don't have an additional dmg effect procc.

@gweivyth
Copy link

gweivyth commented Apr 8, 2019

Holy Lance and Webcutter also fall under this, #1283

@greyenvy
Copy link

greyenvy commented Apr 8, 2019

I have never seen any of the Great Axe additional effects proc (Plantbane's fire dmg, Raifu's lightning dmg) but I have seen katana added effects proc, specifically Busuto +1's poison. It wasn't often but I got a few in Zitah the other day on NIN. That might help with tracking down code I guess.

@AnkleAssassinz
Copy link

There are weapons additional effect that do work as intended:
-Also Confirmed Busuto's poison effect works.

  • Garuda's dagger silence effect also works as intended. It seems to processed more often on too weak mobs and less on xp-giving mobs; as expected.
  • Cruel Spear(lvl 20: Additional effect Impairs Evasion) also works. Not tested Cruel Scythe.

@gweivyth
Copy link

gweivyth commented Apr 8, 2019

Since we are compiling a list of added effects, Yoto +1 works as intended, although the proc rate seems embarrassingly low. (Not the 5%~ advertised on Wiki.)

@Volkaru
Copy link

Volkaru commented Apr 8, 2019

Hellfire Axe works, albeit rarely.

@DrfistMD
Copy link

DrfistMD commented Apr 8, 2019

Dark Mezraq works, albeit like the others seems fairly low.

@Lan-var
Copy link

Lan-var commented Apr 24, 2019

Dainslaif also doesn't work sadly.

@Lan-var
Copy link

Lan-var commented Apr 26, 2019

Wing Sword doesn't work either.

@AkumoBunni
Copy link

Is there any plan to have this working? Had a bloody blade made and a bit disappointed it doesn't work..

@Volkaru
Copy link

Volkaru commented May 21, 2019

Someone is currently working on them, yes. It's a lot to do. They have to pretty much code for every single weapon.

@gweivyth
Copy link

Is there any plan to have this working? Had a bloody blade made and a bit disappointed it doesn't work..

I've completely finished coding all of the added effect weapons up to level 75, I just need to finish data gathering proc rates/damage rates on retail, which has been taking some time.

@bkesecker
Copy link

I am presently reviewing a pull request that partially resolves some of these.

@gweivyth
Copy link

gweivyth commented May 24, 2019

I am presently reviewing a pull request that partially resolves some of these.

I'm about 80% done with data gathering if you just want to wait a few days. I'm going on vacation the last week of the month and plan on finishing this project while I'm on the beach drinking Corona.

For anyone who is curious, I've found that weapon added effects have a pretty static formula that they all use. It appears that their ranges are actually relative to their weapon damage, and all weapons had static rates at which they proc. (Unless otherwise noted on things like Ifrit's Blade, Shiva's Shotel, etc.) Every weapon I've tested so far has fallen into the 9% to 12% range, so I think they're all just 10% flat on retail.

Pretty cool findings so far.

@andimandi1
Copy link

Hi there, just tried Bloody Blade and additional effect still doesn't seem to work. Have you already had time to implement the updates? Thanks

@bkesecker
Copy link

Hi there, just tried Bloody Blade and additional effect still doesn't seem to work. Have you already had time to implement the updates? Thanks

When this, and some other GitHub issues become closed, these would be fixed live after about a week of being closed. The order is roughly: issue created > staff research > prioritized > assigned > researched > coded > local testing > pull request > review > adjust > merge > build > deploy > smoke test > live. This cycle can take a long time.

@bkesecker bkesecker self-assigned this Jun 27, 2019
@andimandi1
Copy link

Appreciate the update BK. Hadn't seen any more comments for the past month so was curious where it stands.

@gweivyth
Copy link

I have all of these completely written and I even tested most of them against retail values and have them corrected, but I got banned so I'm not going to waste any more time testing the values. If someone else wants them they can reach out to me and I'll share what I've got.

@bkesecker
Copy link

I have all of these completely written and I even tested most of them against retail values and have them corrected, but I got banned so I'm not going to waste any more time testing the values. If someone else wants them they can reach out to me and I'll share what I've got.

Thank you for the heads up Gweivyth. If you still want to share your data, let me know via Discord, name is Cygon.

@Volkaru
Copy link

Volkaru commented Jan 22, 2020

Suggesting to link all reports on broken additional effects to this bug report. And not close it until every single one is fixed. There are still many out there that aren't functioning.

@Volkaru Volkaru reopened this Jan 22, 2020
@ShiyoKozuki
Copy link

Sacrifice torque latent doesn't work
https://www.bg-wiki.com/bg/Sacrifice_Torque

@Slamm21
Copy link

Slamm21 commented Jan 22, 2020

Tred Scythe doesn't work, should be best scythe right now outside of apoc
(#2296)

@gweivyth
Copy link

gweivyth commented Jan 24, 2020

Tred Scythe doesn't work, should be best scythe right now outside of apoc
(#2296)

I'll look into this one tonight. I probably missed it on my big batch of weapons cause it doesn't have a listed hidden effect. I didn't mess with any hidden effect weapons.

@gweivyth
Copy link

#2441 has lua pasted into ticket that should fix it.

@gweivyth
Copy link

Also did #931 on its respective ticket as well. That one may need extra code review, I didn't have the chance to run it thru a live server test.

@gweivyth
Copy link

Also added #2309

@gweivyth
Copy link

#1955 added.

@KnowOne134 KnowOne134 self-assigned this Feb 6, 2020
@bkesecker bkesecker removed their assignment Apr 30, 2020
@KnowOne134
Copy link

believe we have added all, make new tickets for any that are not working

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