osu-web icon indicating copy to clipboard operation
osu-web copied to clipboard

Incorrect displaying of "non-calculated scores"

Open RichardKanshen opened this issue 2 years ago • 6 comments

When playing in lazer, some scores take time to calculate, and I can see that in-game, but the web browser says, that only personal best scores can give pp. image osu_2022-08-21_14-02-40

RichardKanshen avatar Aug 21 '22 12:08 RichardKanshen

They don't take time to calculate. If anything, the in-game one is likely wrong.

Needs further investigation.

peppy avatar Aug 21 '22 15:08 peppy

@nanaya Able to check on this? The pp values do seem to be populating correctly.

root@db-master:osu> select * from solo_scores_performance where score_id = 47472793;
| score_id | pp    |
|----------|-------|
| 47472793 | 12.76 |
1 row in set

11FAF0E7-B82E-4221-ACE9-7403CCBC7DAC

peppy avatar Sep 04 '22 03:09 peppy

score displayed on the web is from osu_scores which for lazer is converted when the score is submitted and thus won't have the pp value

nanaya avatar Sep 04 '22 04:09 nanaya

Hmmmmmmm I see. Did we have something in place previously to make this work? I swear lazer score pp values were showing in recent.

I guess it's safe to say this is resolved with the new table format implicitly.

peppy avatar Sep 04 '22 04:09 peppy

@nanaya with the change to solo_scores, do we have new messaging for the case that pp is not calculated yet? the message in this issue in particular will not be correct going forward, i believe. at least not with how things work currently (every lazer score does get pp calculated, unless a fail i believe).

peppy avatar Sep 08 '22 05:09 peppy

I haven't added it. Shouldn't be too difficult.

nanaya avatar Sep 08 '22 06:09 nanaya

The bug is still present as far as I can tell. lazer web

Swekka avatar Sep 29 '22 17:09 Swekka

It's not fixable until the score storage system is switched to new one. If anything, the best tooltip it can currently give out is "pp value for recent scores from lazer client isn't available on web at the moment" (or something along that line).

The fix in the related PR is for when the system is finally migrated.

Actually, there's probably no hint at all whether or not the score is lazer so the tooltip might not even be possible.

nanaya avatar Sep 29 '22 17:09 nanaya