S_HVAC device on vera doesnt set set point to arduino

[leeoz]

Hey.
Im using vera edge with latest firmware.
Arduino nano with nRF24l01 as my gateway. 1.5 API version. On the other side I have Arduino uno with DHT22 and IR led to turn on and off the AC.
Every thing works ok but sending a new set point to the UNO.
I recive the Off and ColdOn from vera and sending the room temp from the UNO to vera also works (i can see the inside temperature chnages). But when I set new temperature in Vera to be sent to the UNO, i see that the UNO recive nothing (i use the serial to monitor it).
Any known bug? Or mybe I missing somthing in my code (can u direct me to an example of using S_HVAC)?
Tnx

[hek]

Might very well be a bug in the Vera plugin as this is untested ground.
Would be nice to see if plugin even try to send anything. Do you have access to the Vera log?

[leeoz]

No, how can I access it? And I will send you all the info needed to debug.
BTW when I send the command it say "command fail" under the temperature set point field.

[hek]

@leeoz, Could you try a thing?

Change the two occurrences of SetCurrentSetpoint to NewCurrentSetpoint in I_Arduino1.xml and report back.

[leeoz]

Its already set to NewCurrentSet point.
???

[hek]

No it isn't...
https://github.com/mysensors/Vera/blob/UI7/I_Arduino1.xml#L67

But I doubt that this is the problem...

@akbooer, need some help on this. Do you see something obvious?

[akbooer]

I notice that code has "NewCurrentSetpoint" and wonder whether it should not, instead, be "newCurrentSetpoint" ?

[blacey]

@akbooer and @hek checkout - http://wiki.micasaverde.com/index.php/Luup_UPnP_Variables_and_Actions#TemperatureSetpoint1

Note, screenshot from UI5 below but it seems that this has changed to the above as of UI7....

[leeoz]

Sorry, I'm new to Lua and vera stuff so I don't understand what I need to do in order to fix it, or to find the error? I know c++ and c# so I familiar with programming but this is something I never done before.
my vera version: 1.7.1320
is this UI7?

[akbooer]

@leeoz said:

Sorry, I'm new to Lua and vera stuff so I don't understand what I need to do in order to fix it, or to find the error? I know c++ and c# so I familiar with programming but this is something I never done before.
my vera version: 1.7.1320
is this UI7?

Yes, it is.

The issue under discussion here is something which definitely needs fixing once and for all by a plugin developer familiar with Lua, and Vera, and UI5, and UI7, and thermostats. I fail on the last point (but can learn) and am taking a look at the issue now, although that certainly doesn't stop anyone else from giving it a go.

[akbooer]

@blacey
You are, of course, right. The problem is that the plugin does not implement the TemperatureSetpoint1 service. Requires a small change to the I_Arduino1.xml file and possibly the Lua file too.

[leeoz]

R u guys going to update the GitHub files and publish it? or you want to give me some instructions on where to change what and test it?
Let me know if I can be any help (mybe sending you my log or somthing...)
tnx
Leeoz

[akbooer]

The point is that no one actually has a solution right now.

I see two ways forward, described here

http://forum.micasaverde.com/index.php/topic,27712.msg246892.html#msg246892

And here

http://forum.micasaverde.com/index.php/topic,28943.msg207265.html#msg207265

The second one requires us to do nothing but wait for a month (so they say) at which time the Vera firmware will be changed to reinstate the missing command.

The first approach is a pragmatic workaround which could be relatively easily coded in the modules I mentioned previously.

[leeoz]

L_Arduino.lua I_Arduino1.xml MyMessage.h

So I fixed it with simple fix.
After reading this:
http://forum.mysensors.org/topic/1337/request-new-sensor-type-thermostatically-controlled-switch/2
Attached are the latest lua+xml+h files with the fix I've made.
Leeoz

[korttoma]

Hi @leeoz would you mind sharing the arduino sketch for the HVAC device from this project?

[korttoma]

I finally put together a thermostat device sketch. Unfortunately it got a bit out of hand and I implemented lots of features that most people probably not need.

Not sure if I'm using the V_HVAC_FLOW_STATE and V_HVAC_SETPOINT_HEAT correctly but it seems to work with my Vera UI5 presented as a S_HEATER device.

Here is the sketch in case someone needs it.

https://codebender.cc/sketch:169733

[korttoma]

Is anyone using Heater device in Vera UI7? Just moved my thermostat to a VeraPlus but it is not quite working correctly. Cant send setpoint value to the device. Sending Heat and Off command works as expected and also status changes are received in Vera. I need to get my z-Wave thermostat moved from my UI5 Veralite to the VeraPlus to do some comparison of the variables used. Then I'll look in to the MySensors GW plugin files if I can find something in need of fixing.

[petewill]

@korttoma
Coincidentally I am working on a thermostat prototype using your code now. I actually just found an error in the Vera files while I was doing some testing. Try uploading the I_Arduino1.xml and L_Arduino.lua to your Vera from here: https://github.com/mysensors/Vera/tree/UI7

[korttoma]

@petewill I actually tried the new files yesterday but I could not get the termostat to work. For some reason the window cover device allso stoped working so I went back to the old file. I dont know why but I suspect that I should update the GW since I dont remember when I updated it an not sure of the version running on it.

[petewill]

@korttoma Wow, that's strange that your window coverings stopped working! I have window coverings and they are working after the update. Maybe updating the gateway is a good idea :)

Are you able to get the logs from the heater device? The window covering messages are 29-31 which are no where near the heater messages which is why this is so strange (to me at least).