GTNewHorizons / GT-New-Horizons-Modpack

New Modpack with Gregtech, Thaumcraft and Witchery
https://www.gtnewhorizons.com/
Other
980 stars 299 forks source link

Multiple controller covers on controller block override each other #9469

Closed GTNH-Afx237v7 closed 2 years ago

GTNH-Afx237v7 commented 2 years ago

Your GTNH Discord Username

Afx237v7

Your Pack Version

2.1.2.2

Your Server

SSP

Type of Server

Single Player

Your Expectation

Following setup used to work but is now broken (gregtech version gregtech-1.7.10-5.09.40.16-35-g825ecd1 from jenkins, now including the cover GUI access from the parent block).

Build LCR, put machine controller covers on all of the sides of the controller block, set to "disable with redstone". Place output hatches with fluid detector covers around the controller, set to "normal" mode, threshold to "0". (this is my current use-case to handle sulfuric gas/light fuel/heavy fuel/naphtha in a single LCR).

grafik

The Reality

Noticed that the output hatch on the left had heavy fuel in it which should stop the LCR from operating until the hatch is empty, as no redstone signal is transferred to the controller. However the LCR kept operating, eventually voiding the heavy fuel. Removing the controller covers one after another revealed that, once the cover on the opposite side of the controller block (i.e. on the right side) is removed, the LCR operates as expected. Seems that covers on the opposite side of each other have a mutual influence

Your Proposal

It should work independent of covers being opposite. Question: Do covers operate as AND logic or OR logic?

Final Checklist

Savallator commented 2 years ago

It is just not intended to use more than one machine controller cover anyways... For the lcr, just put the hatches in the middle of each face, with fluid detectors facing inwards (towards the ptfe pipe casing), set them to strong signal with soldering iron, and put the machine controller at the back of the controller block. works fine.

Glease commented 2 years ago

If you put multiple machine controller cover on it, only one will ever be effective. IIRC this has always been the case. Which of them would be the one effective is an implementation detail that we prefer to not disclose, as they are subject to change without notice.

Savallator commented 2 years ago

I mean, it would certainly be nice to have consistent behavior (either AND or OR), but i guess it is not that easy, and most likely not worth implementing, as every possible usecase could be solved easily without.

bombcar commented 2 years ago

Then perhaps we should refuse to allow multiple controllers on a single block, so people don't try it and think they have a working setup.