Key press Help Required

Ok trying to-do something real simple and its not working i have a if statement but its not working no error returned now i think its to-do with the os.queueEvent("Key",w)

os.pullEvent("redstone")
if rs.getBundledInput(bottom) == 2048 then
os.queueEvent("key",W)
end

I think you need an "end", or maybe you need to change the 2048 to the color.
Hope this helped.

ah sorry i have an end didn't paste that in as that was irrelevant no it runs fine the main issue is the key isn't working i believe

Deathknight0897, on 04 July 2012 - 10:59 AM said:

Ok trying to-do something real simple and its not working i have a if statement but its not working no error returned now i think its to-do with the os.queueEvent("Key",w)

os.pullEvent("redstone")
if rs.getBundledInput(bottom) == 2048 then
os.queueEvent("key",W)
end

try putting a print after the if to check if thats the error

os.pullEvent("redstone")
if rs.getBundledInput(bottom) == 2048 then
print("debug")
os.queueEvent("key",W)
end

Kolpa, on 04 July 2012 - 11:51 AM said:

Deathknight0897, on 04 July 2012 - 10:59 AM said:

Ok trying to-do something real simple and its not working i have a if statement but its not working no error returned now i think its to-do with the os.queueEvent("Key",w)

os.pullEvent("redstone")
if rs.getBundledInput(bottom) == 2048 then
os.queueEvent("key",W)
end

try putting a print after the if to check if thats the error

os.pullEvent("redstone")
if rs.getBundledInput(bottom) == 2048 then
print("debug")
os.queueEvent("key",W)
end

ok so just debugged the hell out of it and im not even getting the signal checked my wiring and its fine the code looks fine so really stumped now any suggestions

Deathknight0897, on 04 July 2012 - 12:02 PM said:

Kolpa, on 04 July 2012 - 11:51 AM said:

Deathknight0897, on 04 July 2012 - 10:59 AM said:

Ok trying to-do something real simple and its not working i have a if statement but its not working no error returned now i think its to-do with the os.queueEvent("Key",w)

os.pullEvent("redstone")
if rs.getBundledInput(bottom) == 2048 then
os.queueEvent("key",W)
end

try putting a print after the if to check if thats the error

os.pullEvent("redstone")
if rs.getBundledInput(bottom) == 2048 then
print("debug")
os.queueEvent("key",W)
end

ok so just debugged the hell out of it and im not even getting the signal checked my wiring and its fine the code looks fine so really stumped now any suggestions

oh shiz i just noticed the problem

os.pullEvent("redstone")
if rs.getBundledInput("bottom") == 2048 then --better not forget quotes
print("debug")
os.queueEvent("key",W)
end

Kolpa, on 04 July 2012 - 12:31 PM said:

Deathknight0897, on 04 July 2012 - 12:02 PM said:

Kolpa, on 04 July 2012 - 11:51 AM said:

Deathknight0897, on 04 July 2012 - 10:59 AM said:

Ok trying to-do something real simple and its not working i have a if statement but its not working no error returned now i think its to-do with the os.queueEvent("Key",w)

os.pullEvent("redstone")
if rs.getBundledInput(bottom) == 2048 then
os.queueEvent("key",W)
end

try putting a print after the if to check if thats the error

os.pullEvent("redstone")
if rs.getBundledInput(bottom) == 2048 then
print("debug")
os.queueEvent("key",W)
end

ok so just debugged the hell out of it and im not even getting the signal checked my wiring and its fine the code looks fine so really stumped now any suggestions

oh shiz i just noticed the problem

os.pullEvent("redstone")
if rs.getBundledInput("bottom") == 2048 then --better not forget quotes
print("debug")
os.queueEvent("key",W)
end

cant test right now but hope your right will test later and respond

Deathknight0897, on 04 July 2012 - 10:59 AM said:

Ok trying to-do something real simple and its not working i have a if statement but its not working no error returned now i think its to-do with the os.queueEvent("Key",w)

os.pullEvent("redstone")
if rs.getBundledInput(bottom) == 2048 then
os.queueEvent("key",W)
end

I can't believe nobody noticed the obvious. Here is fixed code, annotated with comments.

os.pullEvent("redstone")
if rs.getBundledInput("bottom") == 2048 then
os.queueEvent("key",17) -- key code of w is 17 - you're passing nil as a param by just using W
os.queueEvent("char","w") -- You'll also want to queue the char "w" also to cover any usage scenario.
end

Cloudy, on 04 July 2012 - 12:46 PM said:

Deathknight0897, on 04 July 2012 - 10:59 AM said:

Ok trying to-do something real simple and its not working i have a if statement but its not working no error returned now i think its to-do with the os.queueEvent("Key",w)

os.pullEvent("redstone")
if rs.getBundledInput(bottom) == 2048 then
os.queueEvent("key",W)
end

I can't believe nobody noticed the obvious. Here is fixed code, annotated with comments.

os.pullEvent("redstone")
if rs.getBundledInput("bottom") == 2048 then
os.queueEvent("key",17) -- key code of w is 17 - you're passing nil as a param by just using W
os.queueEvent("char","w") -- You'll also want to queue the char "w" also to cover any usage scenario.
end

hey i did i just never used queue event i thought u can use it like that :/

Cloudy, on 04 July 2012 - 12:46 PM said:

Deathknight0897, on 04 July 2012 - 10:59 AM said:

Ok trying to-do something real simple and its not working i have a if statement but its not working no error returned now i think its to-do with the os.queueEvent("Key",w)

os.pullEvent("redstone")
if rs.getBundledInput(bottom) == 2048 then
os.queueEvent("key",W)
end

I can't believe nobody noticed the obvious. Here is fixed code, annotated with comments.

os.pullEvent("redstone")
if rs.getBundledInput("bottom") == 2048 then
os.queueEvent("key",17) -- key code of w is 17 - you're passing nil as a param by just using W
os.queueEvent("char","w") -- You'll also want to queue the char "w" also to cover any usage scenario.
end

thanks very much cloudy trying it now new to this whole coding malarkey but getting on with it well my day job is a graphical artist so this a real step on the while side for me :P/>/>

Ok so i have redone alot of the code and its still not working could someone have a look and see if any obvious error is screaming out here to why this is refusing to work

local function Control()
-- repeat so function continues
repeat
-- inputs (using bundled cables/pressure plates)
-- green=up, red=left, orange=down, yellow=right,
-- grey=enter
-- white=restart
-- black= shutdown
if rs.testBundledInput("bottom", 8192 ) then--green
  os.queueEvent("key", 200) --up
elseif rs.testBundledInput("bottom", 16384) then--red
  os.queueEvent("key", 203) --left
elseif rs.testBundledInput("bottom", 2) then--orange
  os.queueEvent("key", 205)--down
elseif rs.testBundledInput("bottom", 16) then--yellow
  os.queueEvent("key", 208)--right
elseif rs.testBundledInput("bottom", 128) then--grey
  os.queueEvent("key", 28)--enter
elseif rs.testBundledInput("bottom", 1) then--White
  os.queueEvent("os.reboot()")--restart
elseif rs.testBundledInput("bottom", 32768) then--Black
  os.queueEvent("os.shutdown()")--shutdown
end
-- sleep so parallels doesn't get flooded and error
sleep(0.2)
until a == 1
end

give the variable 'a' a value

try enclosing all of the numbers in os.queueEvent("key",number) with quotes

and why are you calling the event os.reboot() and os.shutdown() thay are not events they are just codes, so try

local function Control()
local a=0
repeat
if rs.testBundledInput("bottom", 8192 ) then
  os.queueEvent("key", "200")
elseif rs.testBundledInput("bottom", 16384) then
  os.queueEvent("key", "203")
elseif rs.testBundledInput("bottom", 2) then
  os.queueEvent("key", "205")
elseif rs.testBundledInput("bottom", 16) then
  os.queueEvent("key", "208")
elseif rs.testBundledInput("bottom", 128) then
  os.queueEvent("key", "28")
elseif rs.testBundledInput("bottom", 1) then
  os.reboot()
elseif rs.testBundledInput("bottom", 32768) then
  os.shutdown()
end
sleep(0.2)
until a == 1
end

maybe I'm wrong about the quotes around the numbers so try without them as well. I probably have a few errors in there as well, I am not the greatest at coding but give it a try

i am pretty sure that code doesn't work it just spams errors sorry

and without the quotes on all of the numbers?

same sadly its really frustrating that this wont work and what even worse it doesn't even report back an error

KaoS, on 05 July 2012 - 01:30 PM said:

and without the quotes on all of the numbers?

Well not sure when a becomes 1 but until then once you call the function it'll just keep adding events to the queue, doing nothing else in your code Oo

ok, time to do this in my format. try this one

local function control()
local function qu(number)
os.queueEvent("key",number)
end
local function chk(colors)
return rs.testBundledInput("bottom",colors)
end
while true do
if chk(colors.green) then
qu(200)
elseif chk(colors.red) then
qu(203)
elseif chk(colors.orange) then
qu(205)
elseif chk(colors.yellow) then
qu(208)
elseif chk(colors.grey) then
qu(28)
elseif chk(colors.white) then
os.reboot()
elseif chk(colors.black) then
os.shutdown()
end
sleep(0.5)
end
end

I can summarise it further if you like but I am at work at the moment with a pissed off client:) give that a try

Pinkishu, on 05 July 2012 - 02:10 PM said:

Well not sure when a becomes 1 but until then once you call the function it'll just keep adding events to the queue, doing nothing else in your code Oo

yes, that is the point, we are sending in the event of pressing a key, thus if you input the code os.queueEvent("key",200) the system thinks you pressed key 200

Yeah but no code will ever process that queue'd event until a becomes 1 cause until then it just keeps looping and queueing doing nothing else

you can use the corouteine api to run this in the background of anouther program. you can run worm with this in the background and use bundled cable inputs to control the worm, take a look at this thread http://www.computercraft.info/forums2/index.php?/topic/1996-questionlua/

lol. that was horrible spelling, sorry

but either way, I have no clue what he is using it for, I just want to get it working for him. then maybe he can tell us how he wants to integrate it

Yeah he could be using that, i doubt he does kinda
and that sleep(0.2) will likely eat all your nicely queue'd up events

Pinkishu, on 05 July 2012 - 02:27 PM said:

and that sleep(0.2) will likely eat all your nicely queue'd up events

true, should build an alternative function for the delay

or sleep before creating the event, not after

Ok so i tried your code from earlier same problem as my code i think the lady may be right about it staying in a queue but never getting release to the rest of the program how would i do that

How are you planning on using this code? then I can write something to do that

basically i have a menu u know what ill send u what i got in a pm for obvious reasons dont want to post it until im finished will be open source once done however

you could just use a loop with pullEvent

Do you realise that you never call the function? So the program won't do anything.
The function:

local function control()
  while true do
	os.pullEvent("redstone") -- wait for redstone input
	if rs.testBundledInput("bottom", 8192) then -- you should use colors.<colorName> instead, so it's more readable
	  os.queueEvent("key", 200)
	elseif rs.testBundledInput("bottom", 16384) then
	  os.queueEvent("key", 203)
	elseif rs.testBundledInput("bottom", 2) then
	  os.queueEvent("key", 205)
	elseif rs.testBundledInput("bottom", 16) then
	  os.queueEvent("key", 208)
	elseif rs.testBundledInput("bottom", 128) then
	  os.queueEvent("key", 28)
	elseif rs.testBundledInput("bottom", 1) then
	  os.reboot()
	elseif rs.testBundledInput("bottom", 32768) then
	  os.shutdown()
  end
end

And now to actually run this, you need to call that function:

control()

But if you'r trying to run it on the background, you need to use parallel:

parallel.waitForAny(control, theOtherFunction)

So it will run both functions, and when it receives a redstone event it checks the input and queues the corresponding key.
Example:

parallel.waitForAny(control, function() shell.run("shell") end)

That would run the shell and the control function at the same time, so you can control the computer with redstone (just the keys you added in the function).

EDIT:
I just noticed you just copied the code from here, and you did it wrong, cause you left some parts out (the part that actually runs the program).

im confused why are people using repeats for infinite loops o.O thats stupid and anyways why use elseif if u dont need to

local function Control()
local a=0
while true do
if rs.testBundledInput("bottom", 8192 ) then
  os.queueEvent("key", "200")
end
if rs.testBundledInput("bottom", 16384) then
  os.queueEvent("key", "203")
end
if rs.testBundledInput("bottom", 2) then
  os.queueEvent("key", "205")
end
if rs.testBundledInput("bottom", 16) then
  os.queueEvent("key", "208")
end
if rs.testBundledInput("bottom", 128) then
  os.queueEvent("key", "28")
end
if rs.testBundledInput("bottom", 1) then
  os.reboot()
end
if rs.testBundledInput("bottom", 32768) then
  os.shutdown()
end
sleep(0.2)
end
end

Kolpa, on 05 July 2012 - 09:13 PM said:

im confused why are people using repeats for infinite loops o.O thats stupid and anyways why use elseif if u dont need to

i'm confused why people loop over redstone test with a sleep()
just wait for a redstone event.. seriously


And yes in this case If would probably be better as more than 1 color can turn on at once

You'r right, the elseif's wouldn't allow multiple inputs.
It should be:

local function control()
  while true do
	os.pullEvent("redstone") -- wait for redstone input
	if rs.testBundledInput("bottom", 8192) then
	  os.queueEvent("key", 200)
    end
    if rs.testBundledInput("bottom", 16384) then
	  os.queueEvent("key", 203)
    end
    if rs.testBundledInput("bottom", 2) then
	  os.queueEvent("key", 205)
    end
    if rs.testBundledInput("bottom", 16) then
	  os.queueEvent("key", 208)
    end
    if rs.testBundledInput("bottom", 128) then
	  os.queueEvent("key", 28)
    end
    if rs.testBundledInput("bottom", 1) then
	  os.reboot()
    end
    if rs.testBundledInput("bottom", 32768) then
	  os.shutdown()
    end
  end
end

well im here for probably 3 hours redownloading tf2 :/ damm i hate my internet so just ask

ok mystic t i will try tht im going away for the weekend with my girlfriend so will be back Monday and try that out

Give this a try

local function check(color)
return rs.testBundledInput("back",color)
end
local function control(main,one,two,three,four,select)
while true do
  os.pullevent("redstone")
  if check(colors.white) then
   main(one)
  end
  if check(colors.orange) then
   main(two)
  end
  if check(colors.magenta) then
   main(three)
  end
  if check(colors.lightBlue) then
   main(four)
  end
  if check(colors.black) then
   return main(select)
  end
end
end

then if you call control(myfunctionname,"up","right","back","left","select")

every time it gets a change in redstone input it checks again and calls myfunctionname with the text string in the 2-6th parameter as a parameter to it (eg. if you give it white input it will call myfunctionname("up"))

PLEASE NOTE THAT THE FUNCTION NAME SHOULD NOT BE IN INVERTED COMMAS, IT IS NOT A STRING!!

I used this to create a good menu, you just make myfunctionname handle the menu, it automatically stops looping when you select a menu item and it actually returns the output of myfunctionname to the selecting command (that way you can make myfunctionname always return true if it succeeds in selecting something and you can then see if it failed by using the control function in a line like this 'if control(myfunctionname,"up","right","back","left")==false then print("ERROR!!!") return end')

you can use it for multiple menus as you just change myfunctionname

you can adapt that in a few ways, you can make it accept a table as a last parameter and make it automatically compile the menu from that or rather than using one function make it call a separate one each time etc etc etc

if there are any problems with it let me know, I have tested it

another thing to keep in mind. using pressure plates with this is not such a great idea as while you step on the second one the first one is still on so it gets called again (because it got a redstone event and looped again while it was still on)

you can add functionality to store the previous input and ignore it the next time if you like but consider yourself warned, that's a whole other story with hidden problems 8o

@Deathknight0897 If you would like me to adapt it for your purpose let me know, a pm would be fine