Logic Programming

[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!

[therealdb]

New version with a lot of fixes, sensors and disabled stations support. @DesT maybe give it a try

[DesT]

Is there a place I can set the zone I would like to have exactly? I'm not using all the 16 zones and I would like to specify somewhere to only took 1,2,3,4,8,16 for example!

[therealdb]

Not at the moment, but I think we can add this feature.

[DesT]

@therealdb When updating my code using your latest version, it make openLuup to crash with

Service XML root element name is not 'scpd'

[therealdb]

@dest very strange. I’m out at the moment, but I’ll take a look tomorrow.

[akbooer]

This is a syntax check from the XML reader. Are you sure that you have specified the correct service/device files for the device?

[DesT]

I got this issue just after I upgraded my files from his repo....

[therealdb]

@dest I've just tried to install it via AppStore on openLuup and it's working for me. AppStore is fetching files from GitHub, so I don't how to help you more, besides suggesting to try to update it again.

EDIT: SPC is coming from a Service file, but I'm just emulating lights here, so it's definitely coming from somewhere else. Can you please look at the complete logs (maybe startup logs)?

[DesT]

Yeah but I think I will delete my devices and recreate them from the AppStore... I was fetching manually your file from github since the beginning..