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
71.15k stars 29.83k forks source link

Add smart grid mode sensor to ViCare integration #122751

Open x4N70pHyLL opened 1 month ago

x4N70pHyLL commented 1 month ago

The problem

It seems the heat pumps are also reporting which SG-mode they are in. This is quite helpful if you are either having a load-depending stop signal from your electrical power provider or solar electric energy generation (PV). The value is set via electrical contact on 2 connectors. With variation of these, there are following modes available: Mode 1: power provider lock Mode 2: default, no action Mode 3: use generated power (heat/cool everything up to max values "if it makes sense") Mode 4: use power (heat/cool everything up to max values "if it is possible") - used when there is to much electricity in the grid

As Viessman is displaying them in the ViCare App I would assume they are in the (free) API as well. With a quick check from the diagnostic data I did not find something promising yet, maybe we need to look a little better - possibly the reason is that I don't have activated that functionality yet (PV about to be installed) Screenshot_20230507_191642_Gallery

What version of Home Assistant Core has the issue?

core-2024.7.3

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

ViCare

Link to integration documentation on our website

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

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 month ago

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

Code owner commands Code owners of `vicare` 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 vicare` 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)


vicare documentation vicare source (message by IssueLinks)

CFenner commented 1 month ago

You can also have a look at the data point docs. I already had a short glimpse. Probably you need the advanced API for this (the app does not care). If there is a new api data point, we would need to extend the library first, but actually there is no traffic on the repository the last 8 months. I'm trying to establish a new fork were we can continue the improvement.

x4N70pHyLL commented 1 month ago

You can also have a look at the data point docs. I already had a short glimpse.

I did not find something that looks promising. You did?

If there is a new api data point, we would need to extend the library first, but actually there is no traffic on the repository the last 8 months. I'm trying to establish a new fork were we can continue the improvement.

I would appreciate that, we have the same issue with the cooling consumption I guess. Plus, there are several datas points labeled with "basic" that should be included / look promising for integration, such as viewing and setting schedule for hot water and heating, top/bottom temperatures of hot water and heating buffer, schedule and trigger for hygienic mode against legionella, status of heating boiler (today we have only temperature and compressor phase, but compressor running could mean hot water, heating or cooling) etc.

If I can help let me know. Maybe at least with identifying and explicitly naming the data points that could be included - once you know how to fork or include the library

CFenner commented 1 month ago

@home-assistant rename Add smart grid mode sensor to ViCare integration

CFenner commented 5 days ago

@x4N70pHyLL Could you add a diagnostic file? Do you use the basic API?

The Heizwasserdurchlauferhitzer. would also be something that we could add if there are data points.

x4N70pHyLL commented 5 days ago

config_entry-vicare-362ff0329410f90a890487635b6f5bba.json Sure, please find attached. However, a quick search did not reveal the name of the attribute... I suppose as it is not set up yet in my installation, the data point is not shown? I will get my PV setup hopefully next months and with that also the SG ready set up for my heat pump.

And yes, I am using the basic API