home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.89k stars 30.96k forks source link

Palazzeti ambient temperature #130276

Open gregory-03 opened 1 week ago

gregory-03 commented 1 week ago

The problem

Hello, I have just integrated the Palazzeti component, and I noticed that the temperature returned by the thermostat is not the right one. It displays 125° instead of displaying 21.5°. I use an "Emily 9 us" stove

image

What version of Home Assistant Core has the issue?

core-2024.11.1

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Supervised

Integration causing the issue

Palazzeti

Link to integration documentation on our website

https://www.home-assistant.io/integrations/palazzetti

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

home-assistant[bot] commented 1 week ago

Hey there @dotvav, mind taking a look at this issue as it has been labeled with an integration (palazzetti) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `palazzetti` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign palazzetti` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


palazzetti documentation palazzetti source (message by IssueLinks)

dotvav commented 1 week ago

@gregory-03 thanks for sharing. If 125°C is not the ambient temperature (duh), do you know if it could be the temperature of another of the internal probes?

I haven't implemented the diagnostics entity yet. It should be helpful if you could share the API response from your unit. Visit the following addresses on your browser: http://ip-address-of-connbox/cgi-bin/sendmsg.lua?cmd=GET+STDT http://ip-address-of-connbox/cgi-bin/sendmsg.lua?cmd=GET+ALLS

Feel free to anonymize the content (IP and Mac address, SN...) and join as a file here. Or send to me on Discord: https://discord.gg/YSKvDNK2

The Palazzetti API is not very straightforward about which temperature probe (T1, T2, T3, T4, T5) is for what. And I feel like they may differ per model.

gregory-03 commented 1 week ago
{
    "INFO": {
        "RSP": "OK",
        "CMD": "GET ALLS",
        "TS": 1731245177
    },
    "SUCCESS": true,
    "DATA": {
        "T2": 22.5,
        "F2LF": 2,
        "PQT": 53,
        "PWR": 2,
        "CHRSTATUS": 0,
        "FANLMINMAX": [
            0,
            5,
            0,
            1,
            0,
            1
        ],
        "FDR": 0,
        "F2V": 0,
        "MOD": 1295,
        "DPT": 0,
        "APLWDAY": 1,
        "MAC": "AA:AA:AA:71:F2:72",
        "SETP": 20,
        "FSTATUS": 0,
        "APLTS": "2024-11-10 14:26:30",
        "BECO": 1,
        "STATUS": 9,
        "T3": 26,
        "T1": 125,
        "PUMP": 0,
        "T5": 21.8,
        "F1RPM": 0,
        "OUT": 0,
        "F1V": 0,
        "SN": "XXXXXXXXXXXXXXXXXX",
        "EFLAGS": 1,
        "T4": 0,
        "LSTATUS": 51,
        "F2L": 7,
        "CORE": 44,
        "DP": 0,
        "VER": 3,
        "IN": 3,
        "SECO": 1.2,
        "MBTYPE": 0,
        "FWDATE": "2024-07-17"
    }
}

I think the ambiant temperature is T5 PQT is number of pellets consumed in kg

{
    "INFO": {
        "RSP": "OK",
        "CMD": "GET STDT",
        "TS": 1731245113
    },
    "SUCCESS": true,
    "DATA": {
        "GWDEVICE": "wlan0",
        "GATEWAY": "XXX.XXX.XXX.XXX",
        "NOMINALPWR": 9,
        "PSENSLTSH": 10,
        "WMAC": "AA:AA:AA:71:F2:72",
        "plzbridge": "2.2.1 2022-10-24 11:13:21",
        "HWTYPE": 7,
        "AUTONOMYTYPE": 2,
        "CHRONOTYPE": 5,
        "WMODE": "sta",
        "STOVETYPE": 1,
        "SPLMIN": 6,
        "DSPTYPE": 3,
        "FAN2MODE": 3,
        "MOD": 1295,
        "FAN2TYPE": 2,
        "EGW": "192.168.0.116",
        "MAC": "AA:AA:AA:71:F2:72",
        "FLUID": 0,
        "SPLMAX": 51,
        "ECBL": "down",
        "PSENSLMIN": 240,
        "WADR": "XXX.XXX.XXX.XXX",
        "CLOUD_ENABLED": true,
        "EPR": "static",
        "PELLETTYPE": 3,
        "PSENSLMAX": 12,
        "SNCHK": 1,
        "PSENSTYPE": 0,
        "CONFIG": 1,
        "DSPFWVER": 37,
        "LABEL": "Emily",
        "EMSK": "255.255.255.0",
        "BLEMBMODE": 17,
        "WGW": "XXX.XXX.XXX.XXX",
        "UICONFIG": 1,
        "BLEDSPMODE": 1,
        "MAINTPROBE": 4,
        "ICONN": 1,
        "EBCST": "",
        "FWDATE": "2024-07-17",
        "WSSID": "XXX.XXX.XXX.XXX",
        "WPR": "dhcp",
        "VER": 3,
        "WPWR": "-72 dBm",
        "MBTYPE": 0,
        "WENC": "psk2",
        "CBTYPE": "miniembplug",
        "EADR": "192.168.0.177",
        "sendmsg": "2.2.1 2021-10-12 12:22:53",
        "WMSK": "255.255.255.0",
        "EMAC": "AA:AA:AA:71:F2:72",
        "CORE": 44,
        "SN": "XXXXXXXXXXXXXX",
        "APLCONN": 1,
        "DNS": [
            "192.168.1.254",
            "192.168.1.254",
            "127.0.0.1"
        ],
        "SYSTEM": "2.5.3 2022-11-08 15:52:19 (9e8e347)",
        "WBCST": "XXX.XXX.XXX.XXX",
        "WCH": "6"
    }
}

I use previously custom marcopal74/home-assistant-palazzetti and it works properly except for its obsolete

dotvav commented 1 week ago

@gregory-03 would you be able to test the PR #130293 ? It works locally with my setup, and I think it should work with yours too based on the diagnostic data that you shared above, and on my code review of marcopal74/home-assistant-palazzetti and palazzetti/palazzetti-sdk-asset-parser-python

gregory-03 commented 2 days ago

Sorry for time response, but i don't have any skills for test the PR

dotvav commented 2 days ago

No worries. The bug should be fixed in the next release.