Logic Programming

[DesT]

I just upgrade my z-way from 3.0.0 to 3.0.6 and of course I reload openLuup to reconnect it to zway and since that, OpenSprinkler is in a loop!

[therealdb]

Can you please provide some logs?

[therealdb]

Also, enable verbose mode. Thanks.

[DesT]

@therealdb

The only thing that came out just before the reload it's

2020-05-24 08:02:00.939   luup_log:813: VeraOpenSprinkler[1.3.0](findChild@213): Cannot find child: 813 - "OS-WL-0"
2020-05-24 08:02:00.939   luup_log:813: VeraOpenSprinkler[1.3.0](discovery@407): Device to be added
2020-05-24 08:02:00.941   luup.chdev.sync:: [813] OpenSprinkler, syncing children
2020-05-24 08:02:00.942   openLuup.luup:: device 813 'OpenSprinkler' requesting reload

[therealdb]

@DesT ok, working on a fix. I will provide it soon.

[therealdb]

So, the problem is that it's missing the Water Level Device. @DesT, can you tell me if the there's a device, and if present, report its variables/attributes?

Also, can you report Configured variable on the master? This should be 1 and the aforementioned code should not run. Maybe there's something setting it to 0 again. I can provide a new version using a proprietary variable service in order to prevent the Vera/Openluup subsystem from overriding it.

[therealdb]

I just pushed a new version to GitHub, with just this change. Try it and let me know if it's fixed.

[DesT]

Sorry for the delay, got a little more "Work"

Will try to update today!

[DesT]

I just updated... looks like the "reload" issue is OK now!

Why my device for S08 is not "ON" ?

[DesT]

Another thing...
I try to open S02 from openLuup and it works, the OFF command works BUT in openLuup i'm still "ON"

2020-06-01 08:01:16.040   luup_log:813: VeraOpenSprinkler[1.3.2](deviceMessage@223] OpenLuup detected
2020-06-01 08:01:16.040   luup.task:: status=2 VeraOpenSprinkler : Unable to send command to controller
2020-06-01 08:01:16.040   luup_log:813: VeraOpenSprinkler[1.3.2] Switch power error: 2285 - false - nil

[therealdb]

What's your opensprinkler version? It seems to have a very different kind of replies. In fact, I have a "Unable to send command to controller" error. As with previous efforts, I really need you to start it again with verbose debug enabled, because I'm getting an empty response, so there's something not working.

[DesT]

I'm using
App. 2.1.10
Firmware: 2.1.9 (3)

[therealdb]

@DesT ok, so same fw version. can you send me a ja output? I have an open sprinkler emulator and I can test it easily with anyone's output. thanks.

[therealdb]

looking at previous reply, it seems to differ from my responses, where station names are included. so strange...

[therealdb]

So, I finally, properly resolved the reload part. Version 1.4 is on GitHub, with a couple of other fixes (as the ability to have the children in the same room as the master, via the standard ChildrenSameRoom variable.

@DesT, can you please test it? Your controller is giving different responses, so I'm interested in getting it supported. Thanks.

[DesT]

yeah will try to update later today

[DesT]

I upgraded this morning and I got this in the log

2020-06-03 09:04:16.311   openLuup.scheduler:: [813] OpenSprinkler device startup
2020-06-03 09:04:16.311   luup_log:813: VeraOpenSprinkler Controller Plug-In starting...
2020-06-03 09:04:16.313   luup_log:813: VeraOpenSprinkler[1.4.0]: Plugin starting: "VeraOpenSprinkler" - "1.4.0"
2020-06-03 09:04:16.313   luup_log:813: VeraOpenSprinkler[1.4.0](nil@707): Running on OpenLuup: true
2020-06-03 09:04:16.314   luup_log:813: VeraOpenSprinkler[1.4.0](updateFromController@558): updateFromController started: true
2020-06-03 09:04:16.314   luup_log:813: VeraOpenSprinkler[1.4.0](sendDeviceCommand@245): sendDeviceCommand("ja",nil,813)
2020-06-03 09:04:16.314   luup_log:813: VeraOpenSprinkler[1.4.0](sendDeviceCommand@268): sendDeviceCommand - url: "http://10.99.99.16/ja?&pw=63df27a19d8f8c6669cc50ca042920cc"
2020-06-03 09:04:16.317   luup_log:813: VeraOpenSprinkler[1.4.0](nil@191): HttpGet: "http://10.99.99.16/ja?&pw=63df27a19d8f8c6669cc50ca042920cc" - 1 - "200" - "{\"settings\":{\"devt\":1591175056,\"nbrd\":2,\"en\":1,\"sn1\":0,\"sn2\":0,\"rd\":0,\"rdst\":0,\"sunrise\":310,\"sunset\":1240,\"eip\":1167972197,\"lwc\":1591162809,\"lswc\":1591162809,\"lupt\":0,\"lrbtc\":99,\"lrun\":[3,4,600,1591155601],\"loc\":\"45.55855,-73.89608\",\"jsp\":\"https://ui.opensprinkler.com/js\",\"wsp\":\"weather.opensprinkler.com\",\"wto\":{\"key\":\"\"},\"ifkey\":\"\",\"wtdata\":{\"wp\":\"Manual\"},\"wterr\":0,\"sbits\":[128,0,0],\"ps\":[[0,0,0],[0,0,0],[0,0,0],[0,0,0],[0,0,0],[0,0,0],[0,0,0],[1,24945,1591167601],[0,0,0],[0,0,0],[0,0,0],[0,0,0],[0,0,0],[0,0,0],[0,0,0],[0,0,0]]},\"programs\":{\"nprogs\":5,\"nboards\":2,\"mnp\":40,\"mnst\":4,\"pnsize\":32,\"pd\":[[65,127,0,[420,-1,-1,-1],[0,0,0,0,0,0,0,32400,0,0,0,0,0,0,0,0],\"Daily Cedars\"],[65,2,0,[180,-1,-1,-1],[0,1200,7200,900,0,0,0,0,0,0,0,0,0,0,0,0],\"St-Eustache Rules AM\"],[64,127,0,[1170,-1,-1,-1],[0,0,0,0,0,0,0,0,600,0,0,0,0,0,0,0],\"Daily Garden\"],[1,125,0,[180,0,0,0],[0,600,1200,600,0,0,0,0,0,0,0,0,0,0,0,0],\"Quick daily\"],[65,2,0,[1200,-1,-1,-1],[10800,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0],\"St-Eustache Rules PM\"]]},\"options\":{\"fwv\":219,\"tz\":32,\"hp0\":80,\"hp1\":0,\"hwv\":64,\"ext\":1,\"sdt\":0,\"mas\":0,\"mton\":0,\"mtof\":0,\"wl\":100,\"den\":1,\"ipas\":0,\"devid\":0,\"uwt\":0,\"ntp1\":50,\"ntp2\":97,\"ntp3\":210,\"ntp4\":169,\"lg\":1,\"mas2\":0,\"mton2\":0,\"mtof2\":0,\"fwm\":3,\"fpr0\":100,\"fpr1\":0,\"re\":0,\"sar\":1,\"ife\":0,\"sn1t\":1,\"sn1o\":0,\"sn1on\":0,\"sn1of\":0,\"wimod\":169,\"reset\":0,\"dexp\":-1,\"mexp\":24,\"hwt\":255},\"status\":{\"sn\":[0,0,0,0,0,0,0,1,0,0,0,0,0,0,0,0],\"nstations\":16}"
2020-06-03 09:04:16.318   luup_log:813: VeraOpenSprinkler[1.4.0](updateStatus@443): Update status in progress...
2020-06-03 09:04:16.318   luup_log:813: VeraOpenSprinkler[1.4.0](updateStatus@448): Controller status: 1, "1"
2020-06-03 09:04:16.318   luup_log:813: VeraOpenSprinkler[1.4.0](setVar@120): setVar("urn:upnp-org:serviceId:SwitchPower1","Status","1",813) old value "1"
2020-06-03 09:04:16.318   openLuup.context_switch::  ERROR: [dev #813] ./L_VeraOpenSprinkler1.lua:452: attempt to index local 'jsonResponse' (a nil value)

[therealdb]

your responses are different. I'm not sure that's because you have more than 8 stations. I will add a special support for this. It's strange that it's happening.

All that said, v 1.4.1 is on GitHub. Just upoad L_VeraOpenSprinkler1.lua. This should prevent this kind of situation, I had the update before the device creation.
Thanks!

[therealdb]

@akbooer I found that, on some system, both Vera and openluup, the HTTP call gets a truncated response. but not every time. I'll switch to CURL to prevent this, but do you have an idea why it's happening? Thanks!

[therealdb]

anyway, @DesT , I published a new version using CURL instead of socket.http/ssl.https, if you want to try it and see if the json truncation is over (it should be). Thanks!