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

[BUG] Capacitor Bank energy limit #302

Closed
1 task done
gabrieldn opened this issue Jul 14, 2023 · 0 comments · Fixed by #330
Closed
1 task done

[BUG] Capacitor Bank energy limit #302

gabrieldn opened this issue Jul 14, 2023 · 0 comments · Fixed by #330
Labels
Area-Conduits Everything involving conduits MC-1.20.1 P-2-Low Low priority. Not hurting anyone, just annoying. Type-Bug There is a problem.

Comments

@gabrieldn
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

Currently can't fill a Capacitor Bank multiblock with more than 2,147,483,647 energy and the "percentage gauge" does not represent its actual capacity when over that value.

Reading the energy of the multiblock works fine in the UI.
But if you connect to another capacitor bank that contains more energy, it will increase the total in the UI but the "percentage gauge" will keep its behavior, limited to the mentioned value.

Just to be clear what I refer as "percentage gauge" is the capacitor bank face that can be changed by crouching + right clicking it once on an empty face.

Expected Behavior

  1. Allow the Capacitor Bank multiblock to be filled to its entire capacity and the "percentage gauge" to actually represent its capacity
  2. Or prevent multiblocks that can't be fully functional to be formed

Steps To Reproduce

  1. Build a Capactior Bank multiblock with at least 2,147,483,647 energy capacity
  2. Try to fill it
  3. Observe the "percentage gauge" as it fills

Environment

- OS: Windows
- Java version: Temurin 17.0.7+7 hotspot
- Minecraft version: 1.20.1
- Version: 6.0.5-alpha-all

Anything else?

No response

@gabrieldn gabrieldn added Status-Triage A Triage Member should take a look at this Type-Bug There is a problem. labels Jul 14, 2023
@Rover656 Rover656 added MC-1.20.1 Area-Conduits Everything involving conduits P-2-Low Low priority. Not hurting anyone, just annoying. and removed Status-Triage A Triage Member should take a look at this labels Jul 17, 2023
@Rover656 Rover656 removed this from the The Great Squashening (v6.0.x) milestone Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Conduits Everything involving conduits MC-1.20.1 P-2-Low Low priority. Not hurting anyone, just annoying. Type-Bug There is a problem.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants