sandalle / minecraft_bigreactor_control

Minecraft BigReactor Computercraft Control Program
MIT License
75 stars 41 forks source link

Cannot seem to enable/disable rod automation nor offline the reactor or turbine #62

Open chaos234 opened 9 years ago

chaos234 commented 9 years ago

Hello,

you've forgotten to update you old version to the newer one. Please do this as fast as you can do it!

sandalle commented 9 years ago

What do you mean? lolmer_bigreactor_monitor_prog.lua is at version 0.3.17, which matches the latest release. What's in master is still buggy or not tested, so it's not made into a release and posted to pastebin.

chaos234 commented 9 years ago

Well, but on 0.3.17 I can't click at the monitor to enabel/disabel the rod automation and sometimes I can't take down (offline) the reactor or the turbine ... the debug mode doesn't give me a feedback why the click will not be recognized and after I looked at you script I saw the newer version which contains a bugfix and I was wondering why it's not released because actually it works fine for me (tested with 1 rector (passivly) and 1 reactor (activly) + 1 turbine --- more BRs are setup at the moment to test it with many reactors/turbines as I can build ^^).

If you want, I can "stress test" the new version until 30th May 2015 but I think there should be any error, but I we will see it after my "stress test".

Am 29.05.2015 um 02:23 schrieb Eric Sandall:

What do you mean? lolmer_bigreactor_monitor_prog.lua http://pastebin.com/fguScPBQ is at version 0.3.17, which matches the latest release https://github.com/sandalle/minecraft_bigreactor_control/releases. What's in master is still buggy or not tested, so it's not made into a release and posted to pastebin.

— Reply to this email directly or view it on GitHub https://github.com/sandalle/minecraft_bigreactor_control/issues/62#issuecomment-106639665.

sandalle commented 9 years ago

If you're talking about #61, that'll be fixed in 0.3.18 (not released yet) and only applies for the offline/online functionality after converting a passively cooled reactor to an actively cooled one. The enable/disable worked last I tested it, but my testing is with NST: Diet running ComputerCraft 1.63 and Minecraft 1.6.4. I generally test with the Soartex Fanver Modded Universal and JSTR Modded resource/texture packs. What are you using for mods, Minecraft version, and resource pack(s)?

If you want to see if that fix works for you, I have a TEST_lolmer_bigreactor_monitor_prog.lua pastebin up with the 0.3.18 code for testing (you can modify the startup script to download this version for testing).

chaos234 commented 9 years ago

I play on an FTB Infinity 1.5.0 under 1.7.10 MC Server with BR-0.4.3a and CC-1.6 I think and the error occours on one display, if I useing one display foreach reactor/turbine. The effected one was the reactor controlled ones and stands at the ground. Actually I setting up one reactor (first passivly and then activly colled) with 20 turbines connected to a CC-Network with one debug monitor, one master monitor and one for each other multiblocks. There is only one problem to fix, I think ... how can I setup the monitors correctly, so that I have on the ground the master and debug monitor, above of them the reactor controlled one and on the right and on the left all turbines in correct orders? Is it possible to make a small setup? Because you send requests to all periphals connected trough the network so why not asking the user what the specified monitor should display? E. g.: Found 1 Reactor, 5 Turbines and 10 monitors. Please choose now the monitor for the debug screen (if enabeled), then for the master monitor and after that for each multiblock. This can be easily done over an input from 1 to n where n will be the max number of founded monitors so that the user must only type in the number for the correct monitor so he only must know which number and displayed mintorname is which. Should be possible or not? Well, if my Setup is finished, I'll send you screens from them. Von Samsung-Tablet gesendet

-------- Ursprüngliche Nachricht -------- Von: Eric Sandall notifications@github.com Datum: 29.05.2015 03:12 (GMT+01:00) An: sandalle/minecraft_bigreactor_control minecraft_bigreactor_control@noreply.github.com Cc: Chaos234 info@deltanet-production.de Betreff: Re: [minecraft_bigreactor_control] Different Versions between github and pastebin (#62)

If you're talking about #61, that'll be fixed in 0.3.18 (not released yet) and only applies for the offline/online functionality after converting a passively cooled reactor to an actively cooled one. The enable/disable worked last I tested it, but my testing is with NST: Diet running ComputerCraft 1.63 and Minecraft 1.6.4. I generally test with the Soartex Fanver Modded Universal and JSTR Modded resource/texture packs. What are you using for mods, Minecraft version, and resource pack(s)?

If you want to see if that fix works for you, I have a TEST_lolmer_bigreactor_monitor_prog.lua pastebin up with the 0.3.18 code for testing (you can modify the startup script to download this version for testing).

— Reply to this email directly or view it on GitHub.

sandalle commented 9 years ago

@thetaphi added functionality to control what each monitor displays what you tell it, even duplicate info on multiple monitors if you wish, see #34 .

I'll keep this bug open to try to replicate and fix not being able to offline/online or modify rod control on a reactor from a monitor.

chaos234 commented 9 years ago

Thanks. I'll try out my best to help you and if it's possible, I'll try to change the code to a working one for cc. But at first I must look at the cc-source which was talked about on #34. I'll respond after testing it ;).

Von Samsung-Tablet gesendet

chaos234 commented 9 years ago

I must correct my cc-version, it's 1.7 because CraftOS ist 1.7 ;).

sandalle commented 9 years ago

@Chaos234 , Just FYI, the current stable version supports ComputerCraft 1.58+, 1.63+, and 1.73+ as noted in the Requirements section.