pail23 / stiebel_eltron_isg_component

Stiebel Eltron ISG integration for Home Assistant
MIT License
45 stars 19 forks source link

[FEATURE] Add some more Sensors for LWZ #51

Closed northalpha closed 1 year ago

northalpha commented 1 year ago

I really like you integration and it might be replacing my clumpsy modbus config ;)

May i ask you if you may add following sensor to the integration?

      - name: "LWZ VORLAUFTEMP"
        slave: 1
        address: 11
        input_type: input
        unit_of_measurement: °C
        device_class: temperature
        scale: 0.1
        precision: 1

      - name: "LWZ RUECKLAUFTEMP"
        slave: 1
        address: 12
        input_type: input
        unit_of_measurement: °C
        device_class: temperature
        scale: 0.1
        precision: 1

      # Power consumption
      - name: "LWZ P-HEIZUNG-TAG"
        slave: 1
        address: 3021
        input_type: input
        unit_of_measurement: kWh
        device_class: energy
      - name: "LWZ P-HEIZUNG-SUMME MWh"
        slave: 1
        address: 3023
        input_type: input
        unit_of_measurement: MWh
        device_class: energy
      - name: "LWZ P-HEIZUNG-SUMME kWh"
        slave: 1
        address: 3022
        input_type: input
        unit_of_measurement: kWh
        device_class: energy
      - name: "LWZ P-WW-TAG"
        slave: 1
        address: 3024
        input_type: input
        unit_of_measurement: kWh
        device_class: energy
      - name: "LWZ P-WW-SUMME MWh"
        slave: 1
        address: 3026
        input_type: input
        unit_of_measurement: MWh
        device_class: energy
      - name: "LWZ P-WW-SUMME kWh"
        slave: 1
        address: 3025
        input_type: input
        unit_of_measurement: kWh
        device_class: energy

      # Running times
      - name: "LWZ VERDICHTER-HEIZEN"
        slave: 1
        address: 3027
        input_type: input
        unit_of_measurement: h
      - name: "LWZ VERDICHTER-WW"
        slave: 1
        address: 3029
        input_type: input
        unit_of_measurement: h
      - name: "LWZ ELEKTR-NE-HEIZEN"
        slave: 1
        address: 3030
        input_type: input
        unit_of_measurement: h
      - name: "LWZ ELEKTR-NE-WW"
        slave: 1
        address: 3031
        input_type: input
        unit_of_measurement: h

Also i am trying to get the total of VERDICHTERSTARTS from modbus via

      - name: "LWZ VERDICHTERSTARTS"
        slave: 1
        address: 31
        input_type: input

but i do not see these are realistic values:

Bildschirmfoto 2023-05-05 um 14 24 46

vs.

Bildschirmfoto 2023-05-05 um 14 23 38

maybe you have more insights? I am using an LWZ 5S plus here.

pail23 commented 1 year ago

The missing entities should be implemented in 2023.05. I appreciate testing since I don't have a LWZ/LWA model.

For the Verdichterstarts: I think it's register 30 (not 31) according the documentation. Please check if my implementation works.

northalpha commented 1 year ago

Yes the new sensor are there, thanks, i might would have choosen another name, but thats okay for me :)

Unfortunately the VERDICHTERSTARTS seems to give us a valid, non changing "number" but this is not correct either:

ISG: Bildschirmfoto 2023-05-09 um 13 11 22

HA: Bildschirmfoto 2023-05-09 um 13 12 09

pail23 commented 1 year ago

I guess I have found a issue: The register for the compressor start was wrong. Could you please test again with version 2023.5.1? In case it is not working: You can now download diagnostics information an attach them to this issue.

northalpha commented 1 year ago

Integration reads: 6 ISG shows: 6676

Maybe this is something strange with lower / higher bit/int in the registers? Or maybe my ISG Web Version 12.1.2 having some issues. i have contacted Stiebel Support and asked them if there is any hint or bugs known for this.

thanks for your support and patience.

here are the Diag Data

pail23 commented 1 year ago

Currently I use the modbus register 31 for the compressor starts sensor. Looking at your data I have the following hypothesis:

Compressor starts = register_31 *1000 + register 34 (This would be 6677 in the attached Diag Data). Maybe you could verify this by another Diag Data Dump.

northalpha commented 1 year ago

Yes! Epic discovery!

ISG Reads: 6695

Dump:

        "compressor_starts": 6,
        "system_values": [
          250,
          250,
          0,
          250,
          250,
          0,
          112,
          265,
          295,
          265,
          325,
          252,
          265,
          15,
          0,
          536,
          460,
          29,
          115,
          25,
          115,
          25,
          253,
          36,
          32768,
          32768,
          32768,
          313,
          1622,
          1184,
          6,
          0,
          0,
          695, <--- here it is ... :)
          32768,
          32768,
          32768,
          32768,
          32768,
          32768
        ],

so the formula you suggest adds up! 🎉

thorsten commented 1 year ago

Some ventilation values are missing as well:

  - name: "LWZ ZULUFT-IST-LUEFTERDREHZAHL"
    slave: 1
    address: 17
    input_type: input
    unit_of_measurement: Hz
  - name: "LWZ ZULUFT-SOLL-VOLUMENSTROM"
    slave: 1
    address: 18
    input_type: input
    unit_of_measurement: m³/h
  - name: "LWZ ABLUFT-IST-LUEFTERDREHZAHL"
    slave: 1
    address: 19
    input_type: input
    unit_of_measurement: Hz
  - name: "LWZ ABLUFT-SOLL-VOLUMENSTROM"
    slave: 1
    address: 20
    input_type: input
    unit_of_measurement: m³/h
  - name: "LWZ STUFE-TAG"
    slave: 1
    address: 1017
    input_type: holding
  - name: "LWZ STUFE-NACHT"
    slave: 1
    address: 1018
    input_type: holding
thorsten commented 1 year ago

@pail23 I have a Stiebel Eltron LWZ 304 Trend and I can do some testing.

northalpha commented 1 year ago

LWZ 5s plus here for testing ready :)

pail23 commented 1 year ago

@thorsten I have implemented the registers in 2023.5.4. 1017 was already there. It's now called fan level day

northalpha commented 1 year ago

Currently I use the modbus register 31 for the compressor starts sensor. Looking at your data I have the following hypothesis:

Compressor starts = register_31 *1000 + register 34 (This would be 6677 in the attached Diag Data). Maybe you could verify this by another Diag Data Dump.

could it be that this was not released in 2023.5.4?

also i got a new information bit from Stiebel Support:

Hierzu muss man aber beachten, das bei jeder Abtauung der Verdichter abgeschaltet wird, um die Abtauung einzuleiten. Dies wird auch als Verdichterstart berechnet. Bei einer Wärmepumpe oder Lüftungsintegralgerät sind 6-9 Verdichterstarts am Tag völlig normal und in Ordnung.

So deicing (do not know the correct term for this) from the compressor also counts as 1 start.

pail23 commented 1 year ago

Yes, this was not yet fixed in the last release. It's fixed in the main branch an will be released with the next version.

pail23 commented 1 year ago

@northalpha This should now be implemented in 2023.6