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

[v3.9.7+forge-1.20.1] Incompatibilities with Upgrade Aquatic in multiplayer mode #119

Open
2 tasks done
ghost opened this issue May 31, 2024 · 1 comment
Open
2 tasks done
Assignees
Labels
Bug Something isn't working

Comments

@ghost
Copy link

ghost commented May 31, 2024

What happened?

Both mods work in singleplayer, on multiplayer servers or in lan scrolling or filter menu breaks in statistics screen. It mostly appears on 1.20.1 versions, but some problems exist on 1.19.2 (for example filter menu doesn't close or "Time Since Last Rest" updates when menu is opened, when it should not)

1.20.1 BSS runs with Sinytra Connector, 1.19.2 I used forge version, so it's not a problem with Connector.

Steps to reproduce

  1. Enter singleplayer world or join server
  2. (in singleplayer only) open game to lan
  3. open stats screen

Relevant log output or crash report

No response

Other installed mods

  1. Better Statistics Screen
  2. Upgrade Aquatic https://www.curseforge.com/minecraft/mc-mods/upgrade-aquatic

Code of Conduct

  • I agree that if the issue is regarding a crash, I gave the logs and the crash report.
  • I agree that I will be available later for any follow-up questions to help diagnose and resolve the issue.
@ghost ghost added the Bug Something isn't working label May 31, 2024
@TheCSDev
Copy link
Member

TheCSDev commented Aug 19, 2024

Hello, and sorry for taking so long (months) to respond. It appears I somehow managed to miss this issue.
After some testing, I encountered the same issue, after which, I immediately knew what exactly is going on and causing the issue.

The issue lies in the fact that another mod (in this case likely Update Aquatic or its API mod), is for some reason sending frequent "statistics" packets from the server to the client.

Normally, this is how BSS client and the server are supposed to communicate:

  1. The client open the BS screen, and BSS requests the statistics from the server
  2. The server recieves the request, and sends a single payload to the client, containing the statistics
  3. The client recieves the payload, after which BSS is programmed to refresh the entire screen

Now, because something from the server keeps sending statistics payloads over and over on repeat, several times per second (what a waste of bandwidth if you ask me...), BSS being programmed to refresh the screen on payload, keeps refreshing itself on repeat, causing weird glitchy behavior.

In other words, recieving a payload spam from the server was never accounted for in BSS. I'll likely patch that in the future, just so the mod can tackle servers that act weird and do that kinda stuff.

Thanks a lot for the suggestion (edit: i meant bug report), and sorry again for taking so long. Have a great day!

@TheCSDev TheCSDev self-assigned this Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant