[ZwaveJS] Position and cover not working for Roller Shutter 1/2/3
-
@toggledbits sorry. node 148 is working as expected. this is a RS3. node 4 is a RS2.
here's node 4:
cover.state=true energy_sensor.units="kWh" energy_sensor.value=6.58 position.value=null power_sensor.units="W" power_sensor.value=0 power_switch.state=true x_debug.dt={"0":769,"1":4097,"entity_class":"Cover","match":"deviceClass.generic.key=17;deviceClass.specific.key=6","capabilities":["cover","toggle","position"],"primary_attribute":"cover.state","manufacturerId":271,"productType":770,"productId":4096} x_zwave_values.Binary_Switch_currentValue=true x_zwave_values.Binary_Switch_targetValue=null x_zwave_values.Configuration_Delay_Time_After_S2=10 x_zwave_values.Configuration_Energy_Reports=10 x_zwave_values.Configuration_Forced_Roller_Shutter_Calibration=0 x_zwave_values.Configuration_Lamellas_Positioning_Mode=1 x_zwave_values.Configuration_Managing_Lamellas_in_Response_to_Alarm=0 x_zwave_values.Configuration_Motor_Operation_Detection=0 x_zwave_values.Configuration_Motor_Operation_Time=240 x_zwave_values.Configuration_Periodic_Power_or_Energy_Reports=3600 x_zwave_values.Configuration_Power_Reports=10 x_zwave_values.Configuration_Reports_Type=0 x_zwave_values.Configuration_Response_to_General_Alarm=0 x_zwave_values.Configuration_Response_to_Smoke_CO_CO2_Alarm=0 x_zwave_values.Configuration_Response_to_Temperature_Alarm=0 x_zwave_values.Configuration_Response_to_Water_Flood_Alarm=0 x_zwave_values.Configuration_Roller_Shutter_Operating_Modes=1 x_zwave_values.Configuration_Scenes_Associations_Activation=1 x_zwave_values.Configuration_Self_Measurement=0 x_zwave_values.Configuration_Switch_Type=0 x_zwave_values.Configuration_Turning_Time_Delay_Time=150 x_zwave_values.Manufacturer_Proprietary_fibaro_venetianBlindsPosition=99 x_zwave_values.Manufacturer_Specific_manufacturerId=271 x_zwave_values.Manufacturer_Specific_productId=4096 x_zwave_values.Manufacturer_Specific_productType=770 x_zwave_values.Meter_reset=null x_zwave_values.Meter_value_65537=6.58 x_zwave_values.Meter_value_66049=0 x_zwave_values.Multilevel_Sensor_Power=0 x_zwave_values.Multilevel_Switch_Close=null x_zwave_values.Multilevel_Switch_Open=null x_zwave_values.Multilevel_Switch_currentValue=99 x_zwave_values.Multilevel_Switch_duration=null x_zwave_values.Multilevel_Switch_restorePrevious=null x_zwave_values.Multilevel_Switch_targetValue=99 x_zwave_values.Protection_exclusiveControlNodeId=null x_zwave_values.Protection_local=0 x_zwave_values.Protection_rf=0 x_zwave_values.Protection_timeout=null x_zwave_values.Version_firmwareVersions=["25.25"] x_zwave_values.Version_libraryType=3 x_zwave_values.Version_protocolVersion="3.52" zwave_device.capabilities=[37,38,49,50,112,114,117,134,145] zwave_device.endpoint=0 zwave_device.failed=false zwave_device.generic_class="Multilevel Switch" zwave_device.impl_sig="24233:1:24233:1" zwave_device.is_beaming=false zwave_device.is_listening=true zwave_device.is_routing=true zwave_device.is_secure=false zwave_device.manufacturer_info=[271,770,4096] zwave_device.max_data_rate=null zwave_device.node_id=4 zwave_device.specific_class="Motor Control Class B" zwave_device.status=4 zwave_device.status_text="alive" zwave_device.version_info=[null,"25.25"]
-
Here's node 148 for reference:
cover.state=true energy_sensor.units="kWh" energy_sensor.value=0.41 position.value=0.92 power_sensor.units="W" power_sensor.value=0 x_debug.dt={"entity_class":"Cover","match":"deviceClass.generic.key=17;deviceClass.specific.key=6","capabilities":["cover","toggle","position"],"primary_attribute":"cover.state","description":"Roller Shutter 3","model":"FGR223","default_name":"Roller Shutter 3","values":[{"match":{"endpoint":[1,2],"commandClass":38,"property":"currentValue"},"entity_class":"Cover","primary_attribute":"cover.state","capabilities":{"cover":{"attributes":{"state":{"expr":"value !== 0"}},"actions":{"open":{"valueId":"+targetValue:#","value":99},"close":{"valueId":"+targetValue:#","value":0},"stop":{"valueId":"+Up:","value":false}}},"position":{"attributes":{"value":{"expr":"round( value / 99, 2 )"}},"actions":{"set":{"valueId":"+targetValue:#","value":{"expr":"max( 0, min( 99, round( 100 * ( parameters.value ?# 0 ), 0 ) ) )"}},"increase":{"valueId":"+targetValue:#","value":{"expr":"max( 0, min( 99, round( 100 * ( ( entity.attributes.position?.value ?# 0 ) + ( parameters.amount ?# 0.10 ) ), 0 ) ) )"}},"decrease":{"valueId":"+targetValue:#","value":{"expr":"max( 0, min( 99, round( 100 * ( ( entity.attributes.position?.value ?# 0 ) - ( parameters.amount ?# 0.10 ) ), 0 ) ) )"}},"relative":{"valueId":"+targetValue:#","value":{"expr":"max( 0, min( 99, round( 100 * ( ( entity.attributes.position?.value ?# 0 ) + ( parameters.amount ?# 0.10 ) ), 0 ) ) )"}}}},"toggle":{"actions":{"toggle":{"valueId":"+targetValue:#","value":{"expr":"entity.attributes.cover.state ? 0 : 99"}}}}}}],"manufacturerId":271,"productType":771,"productId":4096} x_zwave_values.Meter_reset=null x_zwave_values.Meter_value_65537=0.41 x_zwave_values.Meter_value_66049=0 x_zwave_values.Multilevel_Switch_Down=null x_zwave_values.Multilevel_Switch_Up=null x_zwave_values.Multilevel_Switch_currentValue=91 x_zwave_values.Multilevel_Switch_duration="unknown" x_zwave_values.Multilevel_Switch_restorePrevious=null x_zwave_values.Multilevel_Switch_targetValue=91 x_zwave_values.Notification_Power_Management_Over_current_status=0 x_zwave_values.Notification_System_Hardware_status=0 x_zwave_values.Notification_alarmLevel=null x_zwave_values.Notification_alarmType=null zwave_device.capabilities=[38,50,113] zwave_device.endpoint=1 zwave_device.failed=null zwave_device.impl_sig="24233:1:24233:1" zwave_device.manufacturer_info=null zwave_device.node_id=148 zwave_device.version_info=null
-
OK. That's easy to fix, I think. Pull the updated
zwavejs_data.zip
file from the server and put in yourconfig
directory, restart Reactor, and let's see if 4 behaves better (I just applied the exceptions for the 223 to the 222, adjusted for a single endpoint). -
Still not working. position.value is still null.
Logs:
[latest-24212]2024-08-20T15:19:14.892Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs performing position.relative on Cover#zwavejs>4-0 with [Object]{ "amount": 0.5 } [latest-24212]2024-08-20T15:19:20.349Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs performing position.relative on Cover#zwavejs>4-0 with [Object]{ "amount": 0.5 } [latest-24212]2024-08-20T15:19:29.900Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs performing cover.close on Cover#zwavejs>4-0 with [Object]{ } [latest-24212]2024-08-20T15:19:29.901Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs action cover.close set node 4 endpoint 0 value 38:targetValue:(undefined) = 0 [latest-24212]2024-08-20T15:19:29.903Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs setting value [Object]{ "command": "node.set_value", "nodeId": 4, "valueId": { "endpoint": 0, "commandClass": 38, "property": "targetValue" }, "value": 0 } [latest-24212]2024-08-20T15:19:32.172Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs performing cover.close on Cover#zwavejs>4-0 with [Object]{ } [latest-24212]2024-08-20T15:19:32.173Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs action cover.close set node 4 endpoint 0 value 38:targetValue:(undefined) = 0 [latest-24212]2024-08-20T15:19:32.174Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs setting value [Object]{ "command": "node.set_value", "nodeId": 4, "valueId": { "endpoint": 0, "commandClass": 38, "property": "targetValue" }, "value": 0 } [latest-24212]2024-08-20T15:19:34.602Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs performing cover.stop on Cover#zwavejs>4-0 with [Object]{ } [latest-24212]2024-08-20T15:19:36.567Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs performing cover.open on Cover#zwavejs>4-0 with [Object]{ } [latest-24212]2024-08-20T15:19:36.568Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs action cover.open set node 4 endpoint 0 value 38:targetValue:(undefined) = 99 [latest-24212]2024-08-20T15:19:36.570Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs setting value [Object]{ "command": "node.set_value", "nodeId": 4, "valueId": { "endpoint": 0, "commandClass": 38, "property": "targetValue" }, "value": 99 } [latest-24212]2024-08-20T15:19:45.548Z <ZWaveJSController:INFO> ZWaveJSController#zwavejs performing position.relative on Cover#zwavejs>4-0 with [Object]{ "amount": 0.5 }
-
toggledbitswrote on Aug 20, 2024, 3:26 PM last edited by toggledbits Aug 20, 2024, 11:26 AM
OK. You're talking about an attribute, but showing me actions in a log. On my system, the FGR222 is now showing position.value correctly for any value in range.
Here's what my log looks like (position.set to 0.45 on device 4, using your configuration):
[###BUILDVERSION###]2024-08-20T15:25:04.586Z <ZWaveJSController#zwstatic:INFO> ZWaveJSController#zwstatic performing position.set on Cover#zwstatic>4-0 with [Object]{ "value": 0.45 } [###BUILDVERSION###]2024-08-20T15:25:04.587Z <ZWaveJSController#zwstatic:5:ZWaveJSController.js:2135> ZWaveJSController#zwstatic def [Object]{ "endpoint": 0, "commandClass": 38, "property": "currentValue", "propertyKey": --undef--, "impl": { "valueId": "+targetValue:#", "value": { "expr": "max( 0, min( 99, round( 100 * ( parameters.value ?# 0 ), 0 ) ) )" } } } [###BUILDVERSION###]2024-08-20T15:25:04.591Z <ZWaveJSController#zwstatic:INFO> ZWaveJSController#zwstatic action position.set set node 4 endpoint 0 value 38:targetValue:(null) = 45 [###BUILDVERSION###]2024-08-20T15:25:04.593Z <ZWaveJSController#zwstatic:5:ZWaveJSController.js:1760> ZWaveJSController#zwstatic _set_value(4,0,38,targetValue,(null),45) [###BUILDVERSION###]2024-08-20T15:25:04.594Z <ZWaveJSController#zwstatic:INFO> ZWaveJSController#zwstatic setting value [Object]{ "command": "node.set_value", "nodeId": 4, "valueId": { "endpoint": 0, "commandClass": 38, "property": "targetValue" }, "value": 45 } [###BUILDVERSION###]2024-08-20T15:25:04.596Z <ZWaveJSController#zwstatic:5:ZWaveJSController.js:335> ZWaveJSController#zwstatic sending #1724167504595<8/20/2024, 11:25:04 AM>: [Object]{ "command": "node.set_value", "nodeId": 4, "valueId": { "endpoint": 0, "commandClass": 38, "property": "targetValue" }, "value": 45, "messageId": 1724167504595 }
So both are working for me. I would try deleting the entity in MSR and restarting to recreate it. Then test again.
-
I don't know what to say. I've deleted all the ZWaveJS entities, restarted and still no luck.
-
What is the
impl_sig
attribute on device 4? -
zwave_device.impl_sig="24233:1:24233:2"
-
Are you using position.set directly, or using the relative actions?
-
position.set.
-
That's wonderful. I have a FGR222 (previous fw) that's working. Here's node 3:
cover.state=false energy_sensor.units="kWh" energy_sensor.value=0 position.value=0 power_sensor.units="W" power_sensor.value=0 power_switch.state=true x_debug.dt={"entity_class":"Cover","match":"deviceClass.generic.key=17;deviceClass.specific.key=6","capabilities":["cover","toggle","position"],"primary_attribute":"cover.state","description":"Roller Shutter","model":"FGRM222","default_name":"Roller Shutter","values":[{"match":{"commandClass":38,"property":"currentValue"},"capabilities":{"cover":{"attributes":{"state":{"expr":"value !== 0"}},"actions":{"open":{"valueId":"+targetValue:#","value":99},"close":{"valueId":"+targetValue:#","value":0},"stop":{"valueId":"+Up:","value":false}}},"position":{"attributes":{"value":{"expr":"round( value / 99, 2 )"}},"actions":{"set":{"valueId":"+targetValue:#","value":{"expr":"max( 0, min( 99, round( 100 * ( parameters.value ?# 0 ), 0 ) ) )"}},"increase":{"valueId":"+targetValue:#","value":{"expr":"max( 0, min( 99, round( 100 * ( ( entity.attributes.position?.value ?# 0 ) + ( parameters.amount ?# 0.10 ) ), 0 ) ) )"}},"decrease":{"valueId":"+targetValue:#","value":{"expr":"max( 0, min( 99, round( 100 * ( ( entity.attributes.position?.value ?# 0 ) - ( parameters.amount ?# 0.10 ) ), 0 ) ) )"}},"relative":{"valueId":"+targetValue:#","value":{"expr":"max( 0, min( 99, round( 100 * ( ( entity.attributes.position?.value ?# 0 ) + ( parameters.amount ?# 0.10 ) ), 0 ) ) )"}}}},"toggle":{"actions":{"toggle":{"valueId":"+targetValue:#","value":{"expr":"entity.attributes.cover.state ? 0 : 99"}}}}}}],"manufacturerId":271,"productType":769,"productId":4097} x_zwave_values.Binary_Switch_currentValue=true x_zwave_values.Binary_Switch_targetValue=null x_zwave_values.Configuration_Delay_Time_After_S2=10 x_zwave_values.Configuration_Energy_Reports=10 x_zwave_values.Configuration_Forced_Roller_Shutter_Calibration=0 x_zwave_values.Configuration_Lamellas_Positioning_Mode=1 x_zwave_values.Configuration_Managing_Lamellas_in_Response_to_Alarm=0 x_zwave_values.Configuration_Motor_Operation_Detection=1 x_zwave_values.Configuration_Motor_Operation_Time=240 x_zwave_values.Configuration_Periodic_Power_or_Energy_Reports=3600 x_zwave_values.Configuration_Power_Reports=10 x_zwave_values.Configuration_Reports_Type=0 x_zwave_values.Configuration_Response_to_General_Alarm=0 x_zwave_values.Configuration_Response_to_Smoke_CO_CO2_Alarm=0 x_zwave_values.Configuration_Response_to_Temperature_Alarm=0 x_zwave_values.Configuration_Response_to_Water_Flood_Alarm=0 x_zwave_values.Configuration_Roller_Shutter_Operating_Modes=0 x_zwave_values.Configuration_Scenes_Associations_Activation=1 x_zwave_values.Configuration_Self_Measurement=0 x_zwave_values.Configuration_Switch_Type=0 x_zwave_values.Configuration_Turning_Time_Delay_Time=150 x_zwave_values.Manufacturer_Proprietary_fibaro_venetianBlindsPosition=99 x_zwave_values.Manufacturer_Specific_manufacturerId=271 x_zwave_values.Manufacturer_Specific_productId=4097 x_zwave_values.Manufacturer_Specific_productType=769 x_zwave_values.Meter_reset=null x_zwave_values.Meter_value_65537=0 x_zwave_values.Meter_value_66049=0 x_zwave_values.Multilevel_Sensor_Power=0 x_zwave_values.Multilevel_Switch_Close=null x_zwave_values.Multilevel_Switch_Open=null x_zwave_values.Multilevel_Switch_currentValue=0 x_zwave_values.Multilevel_Switch_duration=null x_zwave_values.Multilevel_Switch_restorePrevious=null x_zwave_values.Multilevel_Switch_targetValue=0 x_zwave_values.Protection_exclusiveControlNodeId=null x_zwave_values.Protection_local=0 x_zwave_values.Protection_rf=0 x_zwave_values.Protection_timeout=null x_zwave_values.Version_firmwareVersions=["24.24"] x_zwave_values.Version_libraryType=3 x_zwave_values.Version_protocolVersion="3.52" zwave_device.capabilities=[37,38,49,50,112,114,117,134,145] zwave_device.endpoint=0 zwave_device.failed=false zwave_device.generic_class="Multilevel Switch" zwave_device.impl_sig="24233:1:24233:2" zwave_device.is_beaming=false zwave_device.is_listening=true zwave_device.is_routing=true zwave_device.is_secure=false zwave_device.manufacturer_info=[271,769,4097] zwave_device.max_data_rate=null zwave_device.node_id=3 zwave_device.specific_class="Motor Control Class B" zwave_device.status=4 zwave_device.status_text="alive" zwave_device.version_info=[null,"24.24"]
-
OK. I think I see it. #4 has a different type and ID, and the alias has an error. Pull the new data ZIP and try again.
-
That did the trick. Two things.
position.value
is now incosistent, whereas dimming.level was ok before. I tried to set node 4 to .9, and it's now reporting .31. A refresh is updating the position (to .91), but nowpower_switch.state=null
.power_sensor.value
is still 0. -
What are you comparing it to? For node 4, the last data you show have the Meter class value (meterType 1, property
value
, propertyKey 66049) at 0, and the Multilevel Sensor withsensorType
=4 (Power) also 0. What are you expecting the value to be? Again, can you show more context, too? Post all the attributes? These things are moving targets, so more data is better than less. -
I'm getting the values from ZWaveJS UI and checking on the reported values from MSR:
Watts reading seems to be more related to the device not refreshing itself (or refreshing randomly), as it was doing on Vera. I'll keep debugging the ZWaveJS part, since it seems that if I refresh the device, the values is passed onto Reactor.
Thanks for the help!
-
Sorry, but the image looks like you're showing position, not watts. I would like to understand which of the two power values provided by the device you are trying to reconcile.
That said, a position of 0.91 would be correct, as the scale of the
position.value
attribute in Reactor is 0.0 to 1.0, computed from the device range of 0 to 99 (90/99 = 0.91 rounded to two digits). -
@toggledbits said in [ZwaveJS] Position and cover not working for Roller Shutter 1/2/3:
Sorry, but the image looks like you're showing position, not watts. I would like to understand which of the two power values provided by the device you are trying to reconcile.
no screenshot. As I've said, I'm finding that the device is not reporting that well. I'm 100% sure it was OK under Vera, and maybe it was part of the custom code they wrote for the device, Idk.
@toggledbits said in [ZwaveJS] Position and cover not working for Roller Shutter 1/2/3:
That said, a position of 0.91 would be correct, as the scale of the position.value attribute in Reactor is 0.0 to 1.0, computed from the device range of 0 to 99 (90/99 = 0.91 rounded to two digits).
ok, but I've asked for 0.90 via Reactor and I got 0.91 inside Reactor and 0.90 inside ZWaveJS. Imagine something that tries to position the device until it is OK...
-
toggledbitswrote on Aug 21, 2024, 3:26 PM last edited by toggledbits Aug 21, 2024, 11:27 AM
The rounding is a function of the scaling.
I believe I mentioned in another thread that comparing equality of floating point numbers is a well-known Bad Idea in pretty much all programming languages.
The raw value is always available in the
x_
extended attributes. -
so, the correct way is to look for
x_zwave_values.Multilevel_Switch_currentValue
and divide by 100?EDIT: anyway, I still think that this behavior is not OK, because value are not consistent, but the call is yours.
-
toggledbitswrote on Aug 21, 2024, 11:47 PM last edited by toggledbits Aug 21, 2024, 7:47 PM
I can change it, but it may be a breaking change for all users at this point. Not something to be taken lightly. I have considered it.
I still don't have enough information from you to do anything about the power issue, assuming there still is one.
20/37