search

CivClassic / FactoryMod

Configurable factories for automating item production - Built for Paper 1.16.5
Other
2 stars 17 forks source link

redstone factory activation fails #40

Open TheOrangeWizard opened 3 years ago

TheOrangeWizard commented 3 years ago

previously factories required two redstone pulses to activate - first would "half" activate it, second would start the recipe, third would cancel

presently first pulse "half" activates the factory, second pulse cancels. recipe never starts from redstone

Diet-Cola commented 3 years ago

Have you tested this with the factoy /ctinsecure'd? With 1.14+ the citadel redstone stuff changed a bit which I'm guessing is the culprit here

TheOrangeWizard commented 3 years ago

i believe the furnace etc were unreinforced. i'll check what happens with /ctin

TheOrangeWizard commented 3 years ago

seems to be the same issue when everything is set to insecure unless i'm missing something really obvious

TheOrangeWizard commented 3 years ago

further testing reveals that redstone DUST does not work, however, other redstone blocks (observers, levers, torches, etc.) activate the factory on a single pulse

leaving this issue open as i'm not sure if this is intended behaviour

Maxopoly commented 3 years ago

You need to input it directly, a straight line of redstone will work

caucow commented 2 years ago

this should also be fixed along with #51 in the IO PR. triggering still requires dust/a redstone power source pointed into the factory but should work consistently now