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

Switching Batch Mode does not update UI values correctly. #16003

Closed
2 of 3 tasks
AbdielKavash opened this issue Apr 18, 2024 · 1 comment
Closed
2 of 3 tasks

Switching Batch Mode does not update UI values correctly. #16003

AbdielKavash opened this issue Apr 18, 2024 · 1 comment
Labels
Bug: Minor Status: Triage Issue awaiting triage. Remove once this issue is processed

Comments

@AbdielKavash
Copy link
Member

Your GTNH Discord Username

Abdiel_Kavash

Your Pack Version

2.6.0-beta-2

Your Server

SP

Java Version

Java 20

Type of Server

Single Player

Your Expectation

Turn on or off batch mode on a multiblock via button in the GUI.

Expect the output amounts to update correctly.

The Reality

Before:

After:

You can see that the processing time updates correctly, but the amount of items is still reported per recipe, not per batch. This results in the items per second being wrong.

Closing and reopening the GUI fixes the issue, and new values are now reported correctly.

This is purely an UI problem, actual amount of items produced is correct.

Your Proposal

Batch mode setting needs to be properly synced to the client when the button is pressed, not only when the UI is opened.

Final Checklist

  • I have searched this issue tracker and there is nothing similar already. Posting on a closed issue saying the bug still exists will prompt us to investigate and reopen it once we confirm your report.
  • I can reproduce this problem consistently by follow the exact steps I described above, or this does not need reproducing, e.g. recipe loophole.
  • I have asked other people and they confirm they also have this problem by follow the exact steps I described above, or this does not need reproducing, e.g. recipe loophole.
@AbdielKavash AbdielKavash added Bug: Minor Status: Triage Issue awaiting triage. Remove once this issue is processed labels Apr 18, 2024
@FourIsTheNumber
Copy link
Contributor

Fixed by GTNewHorizons/GT5-Unofficial#3561

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Minor Status: Triage Issue awaiting triage. Remove once this issue is processed
Projects
None yet
Development

No branches or pull requests

2 participants