RCVolus / league-prod-toolkit

Toolkit for League Productions with overlays for champion select, ingame events, end of game stats, and more
MIT License
138 stars 24 forks source link

[ Bug ] Gold / XP Leaderboards don't work in custom games #538

Open sdomi4 opened 1 year ago

sdomi4 commented 1 year ago

Original Issue: https://github.com/RCVolus/league-observer-tool/issues/28

Describe the bug When using this tool with prod-toolkit the Gold and Level leaderboard works when spectating a Ranked Game. When spectating a custom game both of those are blank.

System:

OS: Windows Version: 6.7.2 (or whatever the latest which we had to manually install is)

Himyu commented 1 year ago

We added some changes to the code of the ingame module to fix this and some other issues

FireFoXz09 commented 1 year ago

Gold and XP seem to work now in custom games (as they show something), but the XP + Level seems off, as some player don't have the right level displayed sometimes (off by 1 level). Maybe related to all the changes with the Riot ID or something else (again)

Himyu commented 1 year ago

Probably related to #552 as well

Himyu commented 1 year ago

As mentioned in #522 there was an issue with the tool trying to save - player levels multiple times which also caused this issue and should also be fixed with the newest patch to the in-game module.

JustSxm commented 12 months ago

I will confirm this in my next cast which might be after christmas

FireFoXz09 commented 11 months ago

Tested it today in custom games and it was buggy, giving wrong levels for players (some were at lvl "0" with 0 XP)

chinzchi commented 11 months ago

I found that if I don't select Level Display, it won't show that level on the leader board and it will display incorrectly. For example, I set the Level Display to be 6, 11, 16 every time they have an ultimate skill. If they're not at those levels, It will be 0 early in the game.

Himyu commented 11 months ago

I found that if I don't select Level Display, it won't show that level on the leader board and it will display incorrectly. For example, I set the Level Display to be 6, 11, and 16 every time they have an ultimate skill. If they're not at those levels, It will be 0 early in the game.

Based on that comment, I know what the problem is and can supply a fix for that later today.

Himyu commented 11 months ago

We added a patch that should fix this issue 👍

chinzchi commented 11 months ago

We added a patch that should fix this issue 👍

How can I update it without affecting the CSS file I previously edited?

sdomi4 commented 11 months ago

If you edited the CSS directly in the module, you will need to back up your changes and reapply them. Ideally, you apply CSS changes through a theme which will not be affected by updates.

chinzchi commented 11 months ago

Gold Leader Board in Custom game (Tournament Draft) Displayed incorrect. It's like pulling in old gameplay data of players in the custom room to display. It's based on someone playing Renekton while playing Thresh in room. https://youtu.be/xLxaUaM1zRg?t=10831 on time 3:00:33

sdomi4 commented 11 months ago

This is probably due to champions being carried over from the previous game being spectated. Resetting the game state (ideally before/at the start of the game) should fix this, but adding better automatic resets once a game ends is something we'll look into.

chinzchi commented 11 months ago

This is probably due to champions being carried over from the previous game being spectated. Resetting the game state (ideally before/at the start of the game) should fix this, but adding better automatic resets once a game ends is something we'll look into.

Do I have to press this button? Is it necessary? ![Uploading Screenshot 2023-12-15 162908.png…]()

chinzchi commented 11 months ago

Reset Game State Only use this when really needed, all game data of the current game will be lost!

Furthermore, what effect will it have?

FireFoXz09 commented 11 months ago

Reset Game State Only use this when really needed, all game data of the current game will be lost!

Furthermore, what effect will it have?

If you press it DURING a game, some data won't be synced, like the number of turrets destroyed, for example. By pressing it before or at the beginning, you don't miss any major events so all stats resets acordingly.