[OH3] Z-Wave - Is it possible to force-reinitialise devices without re-pairing?

chris · November 2, 2020, 10:32pm

Correct - it was not included in the M1 version of he binding - hence my point to use the newer version (I’m not sure if it has been released yet - but if not, hopefully tomorrow).

Bruce_Osborne · November 2, 2020, 10:47pm

M2 was released a few hours ago.

rogierhofboer · November 3, 2020, 9:29pm

3.0.0-M2 has the exact same problem.

I am now setting up everything so I can build & debug openHAB myself on Linux, so I can really see what’s going on.

Why Linux?

Building on Windows gives problems with line endings on the spotless checks, how and why exactly is a great mystery at the moment.
The Eclipse IDE setup on Windows fails with thousands of warning and errors.
It is that the current maintainers must be using

chris · November 3, 2020, 9:33pm

Well, the issue will still be what I said - it’s not in the binding you’re running. Either you’re not running M2, or M2 still doesn’t have this device.

No - at least not me - I use a Mac. Others use Windows, and some use Linux - is doesnt really matter.

rogierhofboer · November 3, 2020, 10:01pm

My guess it was in the database, but still not recognised in a way.
But you are right…It is not in the database… not anymore that is…

Okay,.It is in the openSmartHouse database.

But not anymore in the openHAB database:

It was introduced a long time ago:

github.com

openhab/org.openhab.binding.zwave/blob/2.2.0/ESH-INF/thing/eurotronic_spirit_0_0.xml

<?xml version="1.0" encoding="UTF-8"?>
<thing:thing-descriptions bindingId="zwave"
  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
  xmlns:thing="http://eclipse.org/smarthome/schemas/thing-description/v1.0.0"
  xsi:schemaLocation="http://eclipse.org/smarthome/schemas/thing-description/v1.0.0
                      http://eclipse.org/smarthome/schemas/thing-description/v1.0.0">

  <thing-type id="eurotronic_spirit_00_000" listed="false">
    <label>Spirit Thermostatic Valve</label>
    <description><![CDATA[
Thermostatic Valve<br /><h1>Overview</h1><h1>Intelligenter Energiesparregler für Z-Wave mit wechselbaren Designblenden.</h1> <p>Die neue Energiesparreglerserie von EUROtronic steht für ein völlig neues Designkonzept. Mit wechselbare Designblenden (als Zubehör erhältlich) kann der smarte Heizkörperthermostat an den individuellen Einrichtungsgeschmack angepasst werden.</p> <p>Nutzern von Z-Wave bietet der neue Spirit-Z-Wave eine Vielzahl neuer Funktionen.</p> <p>Ausgestattet mit FLiRS (Frequently Listening Receiver Slave) reagiert der Energiesparregler mit einer besonders kurzen Latenzzeit auf eingehende Befehle (z.B. Temperaturänderungen). <br />Sein farbiges Display ist besonders gut ablesbar und hat eine extragroße Temperaturanzeige. <br />Durch die per LED mehrfarbig beleuchtete Mitteltaste ist eine intuitive Nutzerführung möglich, da der Nutzer durch farbige Blinkfolgen zur Interaktion mit dem Gerät aufgefordert wird.</p> <br /><h2>Inclusion Information</h2><p>Start Inclusion mode of your primary Z-Wave Controller. Press the Boost-Button. Spirit Z-Wave Plus will show the assigned NodeID.</p> <br /><h2>Exclusion Information</h2><p>Start Exclusion mode of your primary Z-Wave Controller. Now press and hold the boost button of the Spirit Z-Wave Plus for at least 5 seconds.</p>
    ]]></description>

    <!-- CHANNEL DEFINITIONS -->
    <channels>
      <channel id="switch_dimmer" typeId="switch_dimmer">
        <label>Dimmer</label>
        <properties>
          <property name="binding:*:PercentType">SWITCH_MULTILEVEL</property>
          <property name="binding:Command:OnOffType">SWITCH_MULTILEVEL</property>

This file has been truncated. show original

It also was in 3.0.0.M1:

github.com

openhab/org.openhab.binding.zwave/blob/3.0.0.M1/src/main/resources/OH-INF/thing/eurotronic/spirit_0_0.xml

<?xml version="1.0" encoding="UTF-8"?>
<thing:thing-descriptions bindingId="zwave"
  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
  xmlns:thing="https://openhab.org/schemas/thing-description/v1.0.0"
  xsi:schemaLocation="https://openhab.org/schemas/thing-description/v1.0.0
                      https://openhab.org/schemas/thing-description/v1.0.0">

  <thing-type id="eurotronic_spirit_00_000" listed="false">
    <label>Spirit Thermostatic Valve</label>
    <description><![CDATA[
Thermostatic Valve<br /><h1>Overview</h1><p>Spirit Z-Wave Plus is a Z-Wave radio standard compatible energy-saving radiator thermostat.</p> <p>FLiRS (Frequently Listening Receiver Slave): Spirit Z-Wave uses FLiRS to provide short latency and short responding times.</p> <br /><h2>Inclusion Information</h2><p>Start Inclusion mode of your primary Z-Wave Controller. Press the Boost-Button. Spirit Z-Wave Plus will show the assigned NodeID.</p> <br /><h2>Exclusion Information</h2><p>Start Exclusion mode of your primary Z-Wave Controller. Now press and hold the boost button of the Spirit Z-Wave Plus for at least 5 seconds.</p>
    ]]></description>
    <category>RadiatorControl</category>

    <!-- CHANNEL DEFINITIONS -->
    <channels>
      <channel id="basic_basic_number" typeId="basic_number">
        <label>Basic [Deprecated]</label>
        <properties>
          <property name="binding:*:DecimalType">COMMAND_CLASS_BASIC</property>

This file has been truncated. show original

But there the “re-initialisation” without a new install was ‘broken’.

But it is gone on M2 (and master):
https://github.com/openhab/org.openhab.binding.zwave/blob/3.0.0.M2/src/main/resources/OH-INF/thing/eurotronic/spirit_0_0.xml (404)

No idea why…

And on M2 I only tried to get it recognised on an existing network, so no hard reset and adding them again, which most probably also doesn’t work now.

Note: Its Aeotec branded twin is still in the database, but has a different manufacturer id.

chris · November 3, 2020, 10:04pm

Unfortunately there’s not a lot we can do to change that for M2. It should be added back again to the snapshots soon.

rogierhofboer · November 3, 2020, 10:06pm

No problem. Just reporting It is still a milestone build and openHAB is made with love in peoples spare time. I’ll never forget that. I’ll add it myself and see if the recognition works

chris · November 3, 2020, 10:14pm

There’s nothing to do - it will simply be in the next update. Someone edit the database so the device got dropped briefly. It was already approved so it will be in the next update.

rogierhofboer · November 3, 2020, 10:36pm

?
For testing, I uninstalled the zwave binding. Created a new zwave binding addon jar with the spirit xml file added again. I put it in the addons directory. Did a feature:install openhab-transport-serial on the CLI and restarted openHab (the last step might not even be needed).

And immediately two Eurotronic Spirits show up in my Inbox

frankie.delure · November 7, 2020, 2:48pm

I just installed OH3 over OH2.5 and so far a lot it is running. Nevertheless I have the same issues with the Eurotronic valves that were running as intended with the 2.5 installation. I reinstalled the binding, but unfortunately still unknown device. Anything I can do beside waiting?

BTW: Where can I find the log? openhab:9001 is not working anymore.

rogierhofboer · November 7, 2020, 4:09pm

Just waiting or moving to a snapshot version will not work.
The z-wave database needs be re-exported to github, because the spirit is still missing at the moment.

Until that time you can add the spirit manually by adding it to the z-wave binding and install this manually:

Download the 3.0.0-M2 addon kar file here: https://openhab.jfrog.io/openhab/libs-milestone-local/org/openhab/distro/openhab-addons/3.0.0.M2/openhab-addons-3.0.0.M2.kar
The .kar file is a .zip file in repository\org\openhab\addons\bundles\org.openhab.binding.zwave\3.0.0.M2
you can find org.openhab.binding.zwave-3.0.0.M2.jar
Extract this jar file from the .kar archive (tools like Total Commander (Windows) or Midnight Commander (sudo apt install mc, for Linux) really help out here.
Now download the the spirit_0_0.xml from the 3.0.0-M1 release: https://raw.githubusercontent.com/openhab/org.openhab.binding.zwave/3.0.0.M1/src/main/resources/OH-INF/thing/eurotronic/spirit_0_0.xml
Add the spirit_0_0.xml to the OH-INF/thing/eurotronic/ directory in the .jar archive.
In openHAB uninstall the z-wave add-on
Put the altered jar in the addons directory.
At the openHAB CLI run:
feature:install openhab-transport-serial
Restart openHAB

Logs are in : userdata/logs for a manual install and in /var/log/openhab for an install with a package manager.

Bruce_Osborne · November 7, 2020, 4:12pm

Chris just said he did that 4 days agp

Snapshots are built daily. since it has been 4 days it is assuredly in the latest snapshot.

rogierhofboer · November 7, 2020, 4:18pm

?

It is not there…

And also not in:

https://ci.openhab.org/job/openHAB3-Distribution/lastSuccessfulBuild/artifact/distributions/openhab-addons/target/openhab-addons-3.0.0-SNAPSHOT.kar

repository/org\openhab/addons/bundles/org.openhab.binding.zwave/3.0.0-SNAPSHOT/org.openhab.binding.zwave-3.0.0-SNAPSHOT.jar

OH-INF/thing/eurotronic

chris · November 7, 2020, 4:20pm

To be clear I didn’t say that it had been exported - I said that it was approved and would be in the next export.

So it will be in the binding for the next update. At the moment I’ve not done this for OH3 - only 2.5. I will do this later tonight.

Bruce_Osborne · November 7, 2020, 4:24pm

Sorry, I misinterpreted this.

to mean snapshot, not export.

chris · November 7, 2020, 4:35pm

I’ve just triggered the build so it should be done in the next 30 minutes or so…

rogierhofboer · November 7, 2020, 6:20pm

Unfortunately this is not the “correct” version for the Eurotronic Spirit.
It does not limit to options where it should do so.
It also does not include (all) the min and/or max values for parameters having options outside outside the min/max range.

As discussed here there was nothing wrong with the database definition, but it was a bug in the OH3 UI:

Because @ysc has made a quick fix for this, it works with the original database definition in OH3 M2. OH3 Number input configuration parameters do not allow option values outside of the defined range · Issue #459 · openhab/openhab-webui · GitHub

So all recent database changes should be reversed. The one in the spirit definition in the OH3 M1 build is the correct one:
https://raw.githubusercontent.com/openhab/org.openhab.binding.zwave/3.0.0.M1/src/main/resources/OH-INF/thing/eurotronic/spirit_0_0.xml

@chris Can you revert this easily or should I edit it back to the M1 version?

chris · November 7, 2020, 6:40pm

Please can you describe the problem? I thought that the limits in parameter 8 (if I remember correctly) were meant to be set to +/- 50?

rogierhofboer · November 7, 2020, 7:02pm

The strange thing is even the current database and the export on github are not in sync, but ok:

Parameter 1 should be limited to the options
Parameter 2 should be min 5 max 30 (no min/max on github, max 30 but no min in database)
Parameter 3 should be limited to the options
Parameter 4 should be limited to the options
Parameter 5 should be limited to the options, min should be 0 (not in the database, but on github, not sure 0 is the default if it is left blank in the database)
Parameter 6 should be limited to the options
Parameter 7 should be limited to the options
Parameter 8 should be min -50 max 50 (this is in the database, not on github)

with “should be limited to the options” I mean there is no:
<limitToOptions>false</limitToOptions>
in the definition of the parameter, assuming the default is true and if no options are present this parameter is ignored.

The above might be even more confusing. It just should be as defined here:
https://raw.githubusercontent.com/openhab/org.openhab.binding.zwave/3.0.0.M1/src/main/resources/OH-INF/thing/eurotronic/spirit_0_0.xml

If you want I can make the needed changes. Just let me know.

chris · November 7, 2020, 7:07pm

The database should be in sync with the binding - I just exported it an hour or two ago. If it’s incorrect, then there is likely an error in the exporter.

I really don’t have time to look at this now - sorry.