wise-old-man icon indicating copy to clipboard operation
wise-old-man copied to clipboard

Base X stats achievement progress incorrectly calculated

Open discomethod opened this issue 4 years ago • 5 comments

The achievement partial progress towards base X stats is improperly calculated. On an account with all except 3 skills at 90+ (and those skills are 88+), the achievement progress reads "Base 90 Stats 55.22% completed". The account is probably more accurately 90%+ of the way to base 90s. Other base X achievement partial progress may also be improperly calculated, although there seems to be no issue with actually awarding the achievement when base X is reached.

discomethod avatar Jun 04 '21 04:06 discomethod

Ah, I see the issue, I don't exactly remember why I did this at the time, but the way the progress is being calculated is

PROGRESS_TO_90_BASE = LOWEST_SKILL'S_EXP / EXP_AT_LVL_90

And this causes issues if you have everything 90+ except 1 def, for example, as you should be nearly 95% (22/23) to the goal, but you'd be around 0% on the website.

I guess the right way to fix this would be to sum the exp of all skills (capped at 5.4m for lvl 90), divided by (23 * 5.4m)

psikoi avatar Jun 13 '21 13:06 psikoi

I wouldn't mind taking this one. Seems easy enough.

jkinequon avatar Sep 01 '21 01:09 jkinequon

Great thanks, feel free to message me on discord if you need any help 👍

psikoi avatar Sep 01 '21 01:09 psikoi

Just double checking if you want to see if it's good to merge in. Been open for a while :p

jkinequon avatar Oct 05 '21 17:10 jkinequon

Just double checking if you want to see if it's good to merge in. Been open for a while :p

Yes I do, apologies, reviewing now.

psikoi avatar Oct 05 '21 17:10 psikoi