DDF for Eurotronic spirit radiator thermostat (SPZB0001) #8361

mattreim · 2025-10-17T19:10:47Z

This PR is a follow-up to device request #8333 (Eurotronic SPZB0001), and I hope we can finish it. If @SwoopX, @Smanar, @manup helps, it could happen quickly ;)

Spirit_ZigBee_BAL_web_DE_Okt.-2019.pdf

Smanar · 2025-10-18T07:59:03Z

The typo error is corrected ?
I don't see why the validor is saying there is an error.

mattreim · 2025-10-18T11:13:01Z

I don't understand the error either, but it's actually about the DDF.

Smanar · 2025-10-18T17:12:12Z

Perhaps it's because the PR is on Draft status ? So no new check.

mattreim · 2025-10-18T18:39:28Z

I've even deleted everything and the error still exists.

github-actions · 2025-10-18T20:12:33Z

Hey @mattreim, thanks for your pull request!

Tip

Modified bundles can be downloaded here.

DDB changes

Modified

eurotronic/eurotronic_spzb0001_trv.json : Spirit radiator thermostat ✔️

Validation

Tip

Everything is fine !

_{🦆 Updated for commit 4f47c45}

mattreim · 2025-10-18T20:16:24Z

There are no config/hostflags yet, which I added. The information is from the PDF, but I don't think it's correct yet.

Smanar · 2025-10-19T08:06:25Z

Ha yes, was the missing config_hostflags_item.json.

On the code this flag is hidden, but IDK if it's usefull.

sensor.addItem(DataTypeUInt32, RConfigHostFlags)->setIsPublic(false);

SwoopX

Thanks a lot for the PR. Way better to track changes :)

Please have a look at my comments. Apart from those, it seems that the config/offset item got lost from the issue discussion, so I'd suggest to add

        {
          "name": "config/offset",
          "refresh.interval": 3660,
          "write": {
            "at": "0x0010",
            "cl": "0x0201",
            "dt": "0x28",
            "ep": 1,
            "eval": "Item.val / 10;",
            "fn": "zcl:attr"
          },
          "parse": {
            "at": "0x0010",
            "cl": "0x0201",
            "ep": 1,
            "eval": "Item.val = Attr.val * 10;",
            "fn": "zcl:attr"
          },
          "default": 0
        },

SwoopX · 2025-10-22T19:58:09Z

devices/eurotronic/eurotronic_spzb0001_trv.json

+          "name": "state/on",
+          "refresh.interval": 660,
+          "read": {
+            "at": "0x0000",
+            "cl": "0x0006",
+            "ep": 1,
+            "fn": "zcl:attr"
+          },
+          "parse": {
+            "at": "0x0000",
+            "cl": "0x0006",
+            "ep": 1,
+            "eval": "Item.val = Attr.val",
+            "fn": "zcl:attr"
+          },
+          "default": false


Seems like the former content got lost. Here, it should be as follows (the attribute is read through state/valve):

"name": "state/on", "parse": { "at": "0x0008", "cl": "0x0201", "ep": 1, "eval": "Item.val = Attr.val > 3;", "fn": "zcl:attr" }, "read": { "fn": "none" }

SwoopX · 2025-10-22T19:59:48Z

devices/eurotronic/eurotronic_spzb0001_trv.json

+          "read": {
+            "fn": "none"
+          },


Is there any particular reason not to read the attribute? I would've left the default read function here.

SwoopX · 2025-10-22T20:08:02Z

devices/eurotronic/eurotronic_spzb0001_trv.json

+          "read": {
+            "at": "0x4008",
+            "cl": "0x0201",
+            "ep": 1,
+            "fn": "zcl:attr",
+            "mf": "0x1037"
+          },


I'd recommend to change the function here to

"read": { "fn": "none" },

and let config/hostflags do the read for all other items relying on it. That would save lots of unnecessary reads.

SwoopX · 2025-10-22T20:08:23Z

devices/eurotronic/eurotronic_spzb0001_trv.json

+          "read": {
+            "at": "0x4008",
+            "cl": "0x0201",
+            "ep": 1,
+            "fn": "zcl:attr",
+            "mf": "0x1037"
+          },


I'd recommend to change the function here to

"read": { "fn": "none" },

and let config/hostflags do the read for all other items relying on it. That would save lots of unnecessary reads.

SwoopX · 2025-10-22T20:08:57Z

devices/eurotronic/eurotronic_spzb0001_trv.json

+          "read": {
+            "at": "0x4008",
+            "cl": "0x0201",
+            "ep": 1,
+            "fn": "zcl:attr",
+            "mf": "0x1037"
+          },


I'd recommend to change the function here to

"read": { "fn": "none" },

and let config/hostflags do the read for all other items relying on it. That would save lots of unnecessary reads.

SwoopX · 2025-10-22T20:25:43Z

devices/eurotronic/eurotronic_spzb0001_trv.json

+        {
+          "name": "config/heatsetpoint",
+          "refresh.interval": 660,
+          "read": {
+            "at": "0x0012",
+            "cl": "0x0201",
+            "ep": 1,
+            "fn": "zcl:attr"
+          },
+          "write": {
+            "at": "0x0012",
+            "cl": "0x0201",
+            "dt": "0x29",
+            "ep": 1,
+            "eval": "Item.val",
+            "fn": "zcl:attr"
+          },
+          "parse": {
+            "at": "0x0012",
+            "cl": "0x0201",
+            "ep": 1,
+            "eval": "Item.val = Attr.val",
+            "fn": "zcl:attr"
+          },
+          "default": 0
+        },


This one here is very delicate, as the screwed firmware version does not allow for any differentiation which attribute must be written. For the Spirit, the legacy code uses mf attribute 0x4003, so I'd vouch for continuing using it in the read and parse functions.

As for the write function, I guess we can leave it as is, since it doesn't matter anyway. The legacy code has an ugly, but required workaround and sends a write request to any relevant attribute (so in total 2). As said, this must remain as you have no chance to know which attribute the hardware really accepts.

Please, this must be tested in strict mode!

SwoopX · 2025-10-22T20:30:52Z

devices/generic/items/config_hostflags_item.json

+  "id": "config/hostflags",
+  "datatype": "Int32",
+  "access": "RW",
+  "public": true,


I guess the item should not be public.

SwoopX · 2025-10-22T20:40:51Z

devices/eurotronic/eurotronic_spzb0001_trv.json

+          "dt": "0x29",
+          "mf": "0x1037",
+          "min": 1,
+          "max": 600


Legacy code has a reportable change of 50 (int value). Do we want to adapt that?

mattreim · 2025-10-22T21:28:20Z

@SwoopX Thank you for your help and I hope I understood the changes correctly but please take another look.

SwoopX

Seems as if the force push screwed the proper tracking of the changes...

From what I can see, quite some write functions have been deleted, presumably accidentally? Also, the question regarding the reportable change is still pending and if the DDF has been tested in strict mode.

mattreim · 2025-10-24T21:11:24Z

Now I'm completely confused, too. Probably something about ("read" => "fn": "none"), right? I haven't been able to test it yet because I'm using HA and am currently testing it on Windows. But I'd like to start with a somewhat finished DDF.

mattreim · 2025-10-25T14:17:18Z

@SwoopX I just tested everything in strict mode, and the device works exactly as expected. The valve shows values between 0 and 100, and the error code appears immediately, even if I remove the thermostat. The only thing I haven't been able to test is "windowopen", as it's apparently still too warm outside. So, for me, the DDF is now ready for use, but only in strict mode; otherwise, it doesn't work.

{
    "capabilities": {
        "sleeper": false
    },
    "config": {
        "battery": 60,
        "displayflipped": false,
        "heatsetpoint": 2100,
        "locked": false,
        "mode": "auto",
        "offset": 0,
        "on": true,
        "reachable": true
    },
    "ep": 1,
    "etag": "6088eb7dd4abf9d3f826e8ed75b7bdfe",
    "lastannounced": "2025-10-25T13:47:57Z",
    "lastseen": "2025-10-25T13:56Z",
    "manufacturername": "Eurotronic",
    "modelid": "SPZB0001",
    "name": "Thermostat - Diele",
    "nwkaddress": "0x9EB5",
    "state": {
        "errorcode": "none",
        "lastupdated": "2025-10-25T13:57:03.876",
        "on": false,
        "temperature": 2158,
        "valve": 0,
        "windowopen": "0"
    },
    "swversion": "18181120",
    "type": "ZHAThermostat",
    "uniqueid": "00:15:8d:00:07:74:34:93-01-0201"
}

Events:

    "16:15:42:46": {
        "e": "changed",
        "id": "36",
        "r": "sensors",
        "state": {
            "errorcode": "none",
            "lastupdated": "2025-10-25T14:15:42.045",
            "on": false,
            "temperature": 2087,
            "valve": 0,
            "windowopen": "0"
        },
        "t": "event",
        "uniqueid": "00:15:8d:00:07:74:34:93-01-0201"
    }
}

SwoopX · 2025-10-25T21:57:41Z

Awesome, I guess this one should be ready to go then. However, I'd need to prepare another PR to have the legacy code removed completely, otherwise the DDF is not used.

One last remark though: the legacy code sets the bits for the hostflags a little different. No clue if that has any (noticable) impact or not.

mattreim · 2025-10-25T22:11:39Z

Thanks for taking over the follow-up PR, and this old device is a bit slower than the new ones anyway, but that's not a big deal. So the legacy code is getting smaller and smaller. I'll now make it "Ready for review".

manup · 2025-10-29T11:24:04Z

Nice work guys, much appreciated when the legacy code can be removed too later on 👍

SwoopX · 2025-10-29T20:11:02Z

I fear it needs to go with the legacy code simultaneously. Will prepare the accompanying PR on the weekend at the latest.

DDF for Eurotronic spirit radiator thermostat (SPZB0001)

a96960e

mattreim marked this pull request as ready for review October 18, 2025 17:15

mattreim force-pushed the SPZB0001 branch from 8500290 to a96960e Compare October 18, 2025 18:19

Update eurotronic_spzb0001_trv.json

c2058dd

mattreim added 2 commits October 18, 2025 20:52

File update

7ce67f3

Add config_hostflags_item.json

6a100a0

mattreim marked this pull request as draft October 18, 2025 20:13

SwoopX requested changes Oct 22, 2025

View reviewed changes

SwoopX added the Device Request label Oct 22, 2025

Changes by @SwoopX

635a97b

mattreim force-pushed the SPZB0001 branch from 6545599 to 635a97b Compare October 23, 2025 20:58

Reduce some eval

1b89b6f

mattreim requested a review from SwoopX October 24, 2025 16:11

SwoopX requested changes Oct 24, 2025

View reviewed changes

Reverse write functions

4f47c45

mattreim requested a review from SwoopX October 24, 2025 21:25

SwoopX approved these changes Oct 25, 2025

View reviewed changes

mattreim marked this pull request as ready for review October 25, 2025 22:11

manup added this to the v2.32.1-beta milestone Oct 29, 2025

DDF for Eurotronic spirit radiator thermostat (SPZB0001) #8361

Are you sure you want to change the base?

DDF for Eurotronic spirit radiator thermostat (SPZB0001) #8361

Conversation

mattreim commented Oct 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Smanar commented Oct 18, 2025

Uh oh!

mattreim commented Oct 18, 2025

Uh oh!

Smanar commented Oct 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mattreim commented Oct 18, 2025

Uh oh!

github-actions bot commented Oct 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

DDB changes

Modified

Validation

Uh oh!

mattreim commented Oct 18, 2025

Uh oh!

Smanar commented Oct 19, 2025

Uh oh!

SwoopX left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mattreim commented Oct 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

SwoopX left a comment

Choose a reason for hiding this comment

Uh oh!

mattreim commented Oct 24, 2025

Uh oh!

mattreim commented Oct 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

SwoopX commented Oct 25, 2025

Uh oh!

mattreim commented Oct 25, 2025

Uh oh!

manup commented Oct 29, 2025

Uh oh!

SwoopX commented Oct 29, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

mattreim commented Oct 17, 2025 •

edited

Loading

Smanar commented Oct 18, 2025 •

edited

Loading

github-actions bot commented Oct 18, 2025 •

edited

Loading

mattreim commented Oct 22, 2025 •

edited

Loading

mattreim commented Oct 25, 2025 •

edited

Loading