Darker Reactor style

[Tarkus]

@cw-kid yes 0.1.21062 but it seems that my scenes are not getting invoked by MSR today as well. I did test when I initially installed the other day and they did work.

[cw-kid]

@tarkus Right I see, we are both in the same boat then. Its a bug on todays MSR build.

Fingers crossed for a hot fix !

[toggledbits]

Both, and yes, I see the PR. I hadn't looked at my email or Mantis yet this morning, still getting off the ground here, so I didn't see it get opened. All good and thank you. Discussion on that thread shortly.

[toggledbits]

FYI, I'm also trying to implement the "run lua" action for the (vera-specific) system device. That does not mean Lua will run in MSR -- it does not, can not, and will not. But Vera does give me the ability to pass it Lua and have it execute it. That fills me with dread, frankly, but if it's what you need, it can be done. It will also need a UI change to support it, since typing Lua into a small entry field won't due. If it works, it should make invoking functions in a Lua module (built as @therealdb has shown you to do) pretty easy.

[toggledbits]

Update, if you haven't already been notified by Mantis: patch is up on PR#80.

[cw-kid]

@toggledbits said in Running Lua Code ? And watching device properties?:

FYI, I'm also trying to implement the "run lua" action for the (vera-specific) system device. That does not mean Lua will run in MSR -- it does not, can not, and will not. But Vera does give me the ability to pass it Lua and have it execute it. That fills me with dread, frankly, but if it's what you need, it can be done. It will also need a UI change to support it, since typing Lua into a small entry field won't due. If it works, it should make invoking functions in a Lua module (built as @therealdb has shown you to do) pretty easy.

I'm sure this will be very handy for some users.

I am good at the moment however, using the method therealdb showed me. Its working currently.

[cw-kid]

@toggledbits said in Running Lua Code ? And watching device properties?:

Update, if you haven't already been notified by Mantis: patch is up on PR#80.

Confirmed as working again thanks.

[toggledbits]

@tarkus said in Running Lua Code ? And watching device properties?:

I now am trying the CG plugin that Patrick sorted out for me. The small issue with that is I need to convert all my Global Cache codes to pronto hex. I only mention so others know it does not support CG codes.

Were you not using the GC100 plugin before? Because if you were, it should not work any differently through MSR. MSR only passes the string in whatever format. If GC100 understood other code types before (even though the action has "Pronto" in the name), it would still now. MSR doesn't modify or interpret that data.

[Tarkus]

@toggledbits No I was using Lua in Vera Reactor to pass the Global Cache codes. I have played around with the GC Plugin in the past but I have not used it in years up until MSR.

[toggledbits]

@cw-kid said in Running Lua Code ? And watching device properties?:

I am good at the moment however, using the method therealdb showed me. Its working currently.

I think you're missing what I'm saying. You don't need change your library structure or not do what @therealdb has suggested. It's a good, clean way to go. But if you do this from MSR right now as a Request action (that's what I assume you are doing) to run the function:

http://ip-address:3480/data_request?id=lu_action&serviceId=urn:micasaverde-com:serviceId:HomeAutomationGateway1&action=RunLua&Code=VeraScenes.turnLightsOn%28nil%29

You can (21063 and after) just do this, assuming you've loaded the library in startup Lua:

And if you haven't loaded it in startup Lua, you can just bring it in with the code, like this:

[kfxo]

@toggledbits This looks great. So much easier then having to write out the http request.

[Tarkus]

This post is deleted!

[cw-kid]

@toggledbits

I thought you meant we could actually put the LUA code itself in to a MSR action "Set Reaction" and some how you were going to pass that to Vera.

But I see what you meant now, so we still need to edit the VeraScenes.lua file and add our functions / LUA code etc.

I can confirm the Play button works and the code is executed on my Vera Plus

So instead of sending a HTTP Request to Vera to initiate the function we can now just specify the function in the MSR rule actions.

[toggledbits]

@cw-kid said in Running Lua Code ? And watching device properties?:

I thought you meant we could actually put the LUA code itself in to a MSR action "Set Reaction" and some how you were going to pass that to Vera.

You can, and that's exactly what's happening. Your call to your preloaded function is Lua code. You can also put this in and it will run on the Vera:

function later()
    luup.log("I'm running five seconds later!", 2)
end

luup.log("I'm running now!")
luup.call_delay( "later", 5 )

You can put in any Lua you want. You can open a socket, send data, and close it. You can reload Luup (intentionally). You can write a file. No limits.

The code structure proposed by @therealdb makes good sense and is just a better way to encapsulate those functions than to bury them in Reactor rules. It will make your maintenance easier to structure your code that way.

[cw-kid]

Even better then ! Best of both worlds.

[cw-kid]

@toggledbits said in Running Lua Code ? And watching device properties?:

That said, this won't work today, because I haven't yet connected the expressions to the action parameters (the {master_level} value in the dimming action won't work yet)

Hi

Is this related to PR 0000005 ? "Set Variable action not yet implemented" Which is now Closed.

Thanks

[Tarkus]

@therealdb said in Running Lua Code ? And watching device properties?:

@cw-kid here we go: https://github.com/dbochicchio/vera/tree/master/Library

I've added a sample to RunLua as well, so this should serve well our future us when in need of it

Does this work with lua that is using functions? The following runs as standalone code but will not run from the library. I tested another block of lua without functions and that works in the library.

module("VeraScenes", package.seeall) -- TODO: personalize it

-- Function / Samsung_tv_power_on
function matrixtest() 
    luup.log('VeraScenes.matrixtest')
local socket = require("socket")

function tcpclose(tcp)
 tcp:close()
end

function Delay1()
--[[HDMI Matrix B1--]]
tcp = assert(socket.connect("192.168.255.204", 4998))
tcp:send("sendir,1:3,1,38000,1,69,341,171,22,21,22,21,22,21,22,21,22,21,22,21,22,21,22,21,22,64,22,64,22,64,22,64,22,64,22,64,22,64,22,64,22,21,22,21,22,64,22,21,22,64,22,21,22,21,22,21,22,64,22,64,22,21,22,64,22,21,22,64,22,64,22,64,22,1493,341,85,22,3669" .. "\r\n")
luup.call_delay("tcpclose", 1050, tcp)
end

luup.call_delay("Delay1", 1)

function Delay2()
--[[HDMI Matrix B2--]]
local socket = require("socket")
tcp = assert(socket.connect("192.168.255.204", 4998))
tcp:send("sendir,1:3,1,38000,1,69,343,171,21,21,22,21,22,21,22,21,21,21,22,21,22,21,21,21,22,63,22,63,22,63,22,63,22,63,22,63,22,63,22,63,22,63,22,63,22,63,22,21,22,63,22,21,21,21,22,21,22,21,21,22,21,21,22,63,22,21,22,63,22,63,22,63,22,1528,342,85,22,1528" .. "\r\n")
luup.call_delay("tcpclose", 1050, tcp)
end

luup.call_delay("Delay2", 10)
end	

[cw-kid]

@tarkus I have also seen the same issue / limitation.

Seems if your LUA code has functions within it already, you can't use that code in the VeraScenes.lua file as it doesn't run the code.

I've had to run from MSR some Vera scenes with that LUA code instead for now.

Perhaps we are both doing it incorrectly which is quite possible.

[toggledbits]

It's not a limitation. It's the way Lua works, combined with one way Luup is broken.

The problem is that you are declaring your delay callback/handler function in a module. It is therefore not a global function, it is a module function -- it is scoped within the module, not globally. To access functions inside a module, you have to use modulename.functionname(arguments), as you know, and the same is true for the delay handler/callback.

But the way Vera implemented luup.call_delay(), it takes a function name as a string, rather than a function reference. The function, therefore, has to be global. And it's not global, it's in a module.

One way to solve this problem (which all plugins experience, too, by the way) is to make a global alias. That means in your startup Lua, you do something like this:

mymodule = require( "mymodule" )         -- this line loads your module

-- Create a global alias for my delay handler
mymodule_delayhandler = mymodule.delayhandler

Then, in your module, instead of doing luup.call_delay( 'delayhandler', 5 ) you have to do luup.call_delay( 'mymodule_delayhandler', 5 ). This makes the delay use your global alias instead of the function name directly (which it can't).

[Tarkus]

So further testing seems to indicate that it is not the function causing the issue but rather either the multiple delays or just something in the structure of my code.