HomeGenie Forum
Automation Program Plugins and Wizard Scripting => Help => Topic started by: nolio on December 16, 2014, 11:27:28 PM
-
Hi,
I am starting a script that could launch a z-wave command and check the response (for this : http://www.homegenie.it/forum/index.php?topic=230.msg1207#msg1207 (http://www.homegenie.it/forum/index.php?topic=230.msg1207#msg1207)).
I try something like :
Modules.WithName("xxxxx").Command("Basic.Get").Execute();And use it with a try/catch :
try {
Modules.WithName("xxxxx").Command("Basic.Get").Execute();
} catch(Exception ex) {
///// Send alert //////
}
But if there is no module or no response, it's not be catch by the try/catch ..
So is there a way to have a status of "Execute" command ?
Bye
-
You don't have a result from a module command. This is because the module itself is supposed to store all up to date values in its parameters.
So when you execute a Z-Wave basic get, you're just asking to update the value of "Status.Level" in the selected module(s).
g.
-
Hi Gene,
Ok that's clear.
So is there another way like ?
- Request a node and see last time this node has a status update
- Request a node and verify with "ModuleParameterChanged" that this node is really updated
- Other way ...
My aim is to test my z-wave network is working well (no jamming by a thief).
Bye
-
It could be added some method to test the freshness of a parameter or wait it to get updated:
var module = Module.InDomain("Whatever.Domain").WithAddress("1234").Get();
var parameter = module.Parameter("Status.Level");
var idleTime = parameter.IdleTime; // the parameter is not being update since "idleTime" milliseconds
// ....
module.Command("Basic.Get").Execute();
if (parameter.Wait(5000))
{
// ok value was updated in the meantime
var freshValue = parameter.Value;
}
else
{
// parameter was not updated in the last 5 seconds
Program.Notify("Warning", "No updates from Status.Level within latest 5 seconds");
}
if you like this, please file a issue on github.
Cheers,
g.
-
Hi Gene,
I update in r453 and see the add "parameter.wait" was added. So i try this code (by replacing 5000 to 5, because it seem to be in second instead milliseconds), but it use the "else" part (therefore i see the pop-up in HG UI of the device level updated).
Perhaps "parameter.wait" is launch after the system have receive the result of "module.Command("Basic.Get").Execute();" ?
Any idea ?
I don't use :
var idleTime = parameter.IdleTime; // the parameter is not being update since "idleTime" milliseconds
Bye
-
I was expecting an issue like this.
I think it might be correct to have the .Wait method consider a parameter is updated whenever its timestamp is within a range of 1 second before the .Wait is invoked.
Cloud this be a good solution?
Eventually file a issue to github, also regarding IdleTime property that I totally forget to implement.
Cheers,
g.
-
Hi nolio,
I think I will change things like this:
parameter.RequestUpdate(); // this will store current parameter timestamp
module.Command("Basic.Get").Execute();
if (parameter.WaitUpdate(5)) // will check against previously stored timestamp
{
// ok value was updated in the meantime
var freshValue = parameter.Value;
}
else
{
// parameter was not updated in the last 5 seconds
Program.Notify("Warning", "No updates from Status.Level within latest 5 seconds");
}
I did reopen the issue.
https://github.com/genielabs/HomeGenie/issues/72
Cheers,
g.
-
Gene,
By this change, i am not sure that you can avoid so quick response of basic command. But it probably depend on how you imagine to work for .WaitUpdate.
.IdleTime, do you mean something that return that the return value is the last time the device has been reuqested ? Or something else ?
Bye
-
By calling parameter.RequestUpdate(); before sending the command, it will be stored the current timestamp (before the command is sent so...).
Then the parameter.WaitUpdate(); will use that timestamp to find out if the parameter has really been updated.
It will work ;)
g.
-
Oki great :D
-
Hi Gene,
I try this in r466 but it's weird because i have a lot of lost. I didn't count the ok and ko response. But i have too much lost.
My code :
var capteur = Program.InputField("Test.Capteur").Value;
var module = Modules.WithName(capteur).Get();
var parameter = module.Parameter("Status.Level");
var idleTime = parameter.IdleTime; // the parameter is not being update since "idleTime" milliseconds
while (Program.IsEnabled)
{
parameter.RequestUpdate(); // this will store current parameter timestamp
module.Command("Basic.Get").Execute();
if (parameter.WaitUpdate(5)) // will check against previously stored timestamp
{
// ok value was updated in the meantime
var freshValue = parameter.Value;
Program.Notify("Ok", "Updates from Status.Level within latest 5 seconds");
}
else
{
// parameter was not updated in the last 5 seconds
Program.Notify("Warning", "No updates from Status.Level within latest 5 seconds");
}
}
Can i optimize it ?
Bye
-
Hi,
I found a work-around by :
* Delete the following line :
module.Command("Basic.Get").Execute();* Add a polling on the device (use of the native HG function "Level Poll"
* Replace 5 seconds by 60 seconds
=> No error again.
So the command :
module.Command("Basic.Get").Execute();must be run asynchronously or by another program ... Perhaps problem with parameter.RequestUpdate(); command ?
Bye
-
My script run 24hours without any errors. Tonight, i shutdown my testing device and i got error.
So it works fine ! I have my anti-jamming system !!
I can now optimize my script ... :)
Thanks
-
Hi,
Each time my script is running, my mono process is almost at 100%.
I have no idea how can i reduce the cpu use ...
Usually in script i use the :
Program.Setup( () => {
Program.AddInputField("Test.Capteur", "", "Nom du capteur à tester");
});
return true;---
When.ModuleParameterChanged( (module, parameter) =>But in that case, i want to do that all the time. More over, if the zwave signal is jamming i will probably have no "ModuleParameterChanged" so it's useless ..
Someone as an idea ?
My script :
while (Program.IsEnabled)
{
Program.RunAsyncTask(()=>
{
parameter.RequestUpdate(); // this will store current parameter timestamp
if (parameter.WaitUpdate(60)) // will check against previously stored timestamp
{
var freshValue = parameter.Value;
Program.Notify("Ok", "Updates from Status.Level within latest 60 seconds");
}
else
{
Program.Notify("Ko", "Update failed from Status.Level within latest 60 seconds");
}
});
}Bye
-
you program do a restless loop because you're using runasynctask that launches a new thread inside a while loop.
this means that your program will spawn infinite threads one after the other.
I think you don't need runasynctask or either put a reasonable pause inside the loop.
is there any reason you're using runasynctask?
g.
-
I think you don't need runasynctask or either put a reasonable pause inside the loop.
is there any reason you're using runasynctask?
It was just a try (a bad one;)) to reduce the usage of cpu, but with or without "runasynctask" the cpu is around 100%. :(
parameter.WaitUpdatealready do a "pause" but i try to add one to test, but no more chance.
-
How do you monitor cpu usage? I've used "top" but not sure if there's something better that's already included. I haven't reviewed utilization, but it would probably be worth checking out to make sure my codes don't cause issues.
-
"top" is a good way. And i use the following program to have an history : http://www.homegenie.it/forum/index.php?topic=76.msg317#msg317 (http://www.homegenie.it/forum/index.php?topic=76.msg317#msg317)
-
Hi,
@Gene, do you think there is a way to do without "charging" so much the cpu ?
Bye