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

1.21 Comparator retains power from empty shulker box #524

Closed
AlxTray opened this issue Jun 18, 2024 · 18 comments
Closed

1.21 Comparator retains power from empty shulker box #524

AlxTray opened this issue Jun 18, 2024 · 18 comments
Labels
A-vanilla-parity Area: Vanilla Parity F-good-issue Flag: This is an example of a good issue report F-has-workaround Flag: There is a workaround available P-high Priority: High R-pending-release Resolution: Pending release S-fixed Status: Fixed T-bug Type: Bug

Comments

@AlxTray
Copy link

AlxTray commented Jun 18, 2024

Version Information

lithium-fabric-mc1.21-0.12.5

Expected Behavior

The comparator reading the shulker box should not output a signal if the shulker box is empty

Actual Behavior

Sometimes when using a box unloader once the shulker box has been drained of all it's items, the comparator reading the box retains powered with a signal strength of 1. This does not occur without the mod, granted this has not been extensively tested but it usually breaks after 2 or 3 shulkers through the unloader and has not happened yet in Vanilla.

Reproduction Steps

Not exactly 100% sure as I have only tested using a particular box unloader but other than that place a comparator facing away from a shulker box and then empty it, sometimes the empty shulker box will still have a signal strength of 1 read from it.

Other Information

Tested on a private fabric server running Lithium, FerriteCore, C2ME, Bluemap, Jade, No-enderman-grief, and Spark

@AlxTray AlxTray added the T-bug Type: Bug label Jun 18, 2024
@2No2Name 2No2Name added S-info-needed Status: Further information is needed to diagnose this issue S-cannot-repro Status: Developer can't reproduce labels Jun 18, 2024
@2No2Name
Copy link
Member

2No2Name commented Jun 18, 2024

Cannot reproduce. Tested with only lithium installed, placing a bunch of hoppers, comparators reading the block above the hopper, then placing a shulker box with a few items inside already on top of the hopper, allowing the comparator to read it. After a while all comparators turn off.

@2No2Name
Copy link
Member

Maybe test this with and without the other mods to rule out any mod incompatibilities. Otherwise please provide the exact setup (e.g. world download) where the issue shows up

@Shoftie03
Copy link

Shoftie03 commented Jun 19, 2024

I have tested this on both Lithium 1.20.6-0.12.4 and 1.21-0.12.5, running on Prism Launcher Java 21 and can reproduce this issue. The comparator will not update even if the shulker box is empty until a block update is given to it.
https://youtu.be/CQjt4UU92U8 -> where I was able to reproduce this issue
https://youtu.be/9eHcvMi7HUw?si=_d4eJpfItmNGNqeA -> Rapscallion's latest CartMIS sorter, I was able to reproduce this issue in the wdl given: the comparator for the shulker unloader at -2 -46 22 won't update

edit: no other mods aside from lithium present, and issue can be reproduced even without tick sprinting

@2No2Name 2No2Name added P-high Priority: High A-vanilla-parity Area: Vanilla Parity S-confirmed Status: Confirmed F-good-issue Flag: This is an example of a good issue report and removed S-info-needed Status: Further information is needed to diagnose this issue S-cannot-repro Status: Developer can't reproduce labels Jun 19, 2024
@2No2Name
Copy link
Member

Confirmed by using the Rapscalliion world download at the location with 10+ fully filled boxes in the nearby chest

@AlxTray
Copy link
Author

AlxTray commented Jun 19, 2024

That's interesting as the unloader I was facing the issue on was for the CartMIS V2 (https://www.youtube.com/watch?v=LPshwkDboXk). Which the unloader for the CartMIS V3 (the video linked above) uses a very similar or the same unloader for the input. I have just tested on a fresh download of CartMIS V2 and can reproduce again (with only Lithium on my client).

I also had a look at the Storage Tech discord earlier and someone is also facing this issue with the CartMIS V3.

@AlxTray
Copy link
Author

AlxTray commented Jun 19, 2024

Additionally, I went back onto the server with Lithium installed with the unloader in a state with a powered comparator from an empty shulker box and no matter how many times I removed and replaced the comparator it would be powered, this was even retained on a server restart. But after removing Lithium and restarting replacing the comparator is de-powered and was fixed.

@2No2Name
Copy link
Member

After some more testing I think my attempt to reproduce the issue was broken, because I assumed the empty box shouldn't stay there no matter the comparator state. Can't reproduce in further testing

@2No2Name
Copy link
Member

@Shoftie03 is your video recorded in singleplayer with only lithium installed, or is it on a server? If it is on a server, which mods are installed on the server?

@Shoftie03
Copy link

It is in a singleplayer save of the exact world download in rapscallion's video. I deleted the previous save which I had accessed previously with tweakeroo to unzip a new one, and could still reproduce the issue.

@2No2Name
Copy link
Member

What are you putting into the shulker boxes, are you putting them into the storage hall's input chest?

@Shoftie03
Copy link

I used the sample hermitcraft boxes provided in the outskirts of the wdl (I think I used Grian's set). I nbt copied the barrels, placed them down and put them manually into the input chest inside the hall. Initially I observed the issue with carpet creativeNoClip and thought it was something to do with collision but that's probably irrelevant.

@2No2Name
Copy link
Member

Thank you very much, reproduced for sure now

@AlxTray
Copy link
Author

AlxTray commented Jun 19, 2024

Are you two only unloading fully-filled shulkers? As I'm attempting to re-produce using half-filled shulkers and I am not getting the issue. So possibly this only occurs when draining full shulkers to empty?

@2No2Name
Copy link
Member

This probably requires different item types, with some hopper minecarts not being able to pull items due to the different item type

@2No2Name 2No2Name added the R-pending-release Resolution: Pending release label Jun 19, 2024
@2No2Name
Copy link
Member

1f97499

@Shoftie03
Copy link

Shoftie03 commented Jun 19, 2024

I am consistently able to reproduce this issue with a full shulkers of non-stackable, not with 16 or 64 stackables

edit: but seems like 2no2name has already resolved this issue; thank you very much!!

@2No2Name
Copy link
Member

Workaround: Add line to config/lithium.properties file:
mixin.block.hopper=false

@2No2Name 2No2Name added F-has-workaround Flag: There is a workaround available and removed P-high Priority: High S-confirmed Status: Confirmed labels Jun 19, 2024
@2No2Name 2No2Name added P-high Priority: High F-good-pr Flag: This is an example of a good pull request S-confirmed Status: Confirmed and removed F-good-pr Flag: This is an example of a good pull request labels Jun 19, 2024
@AlxTray
Copy link
Author

AlxTray commented Jun 19, 2024

Nice! Thanks for your work in fixing it.

@2No2Name 2No2Name added the S-fixed Status: Fixed label Jun 19, 2024
@jellysquid3 jellysquid3 removed the S-confirmed Status: Confirmed label Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-vanilla-parity Area: Vanilla Parity F-good-issue Flag: This is an example of a good issue report F-has-workaround Flag: There is a workaround available P-high Priority: High R-pending-release Resolution: Pending release S-fixed Status: Fixed T-bug Type: Bug
Projects
None yet
Development

No branches or pull requests

4 participants