Open Roguyt opened 6 months 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!
(message by CodeOwnersMention)
vicare documentation vicare source (message by IssueLinks)
Great that you're testing the dev version.
What do you expect to see, what sensors do your devices expose? Sharing the diagnostics would be helpful.
I think at least room sensors are not that difficult to add.
I added a PR to expose the room sensor, but I have no hardware to test this. If you would be so kind, we could make it to the march release with this. You can use this script in the terminal to checkout the PR: https://gist.github.com/bdraco/43f8043cb04b9838383fd71353e99b18
cd config
curl -o- -L https://gist.githubusercontent.com/bdraco/43f8043cb04b9838383fd71353e99b18/raw/core_integration_pr | bash /dev/stdin -d vicare -p 112027
Hi,
Thank you for the really fast answer :)
For the floor thermostat, it's a bit more complicated since I believe PyViCare doesn't support it yet, and I'm not really surprised given I'm one of the few, if not the first to use it in France... So I guess it's not widely sold in germany either.
I just tried the PR and I did get 2 new entities for one of the room sensor, but I suppose they should be added as separated device since they can be in different room. Also it only picked up one of my sensor while I have 2.
Please find attached the diagnostics and the logs.
config_entry-vicare-8059c83839f26cf125fdf59a4fc0b0f8.json home-assistant_vicare_2024-03-02T07-55-34.374Z.log
😭 due to the way entities get their id, the room sensors overwrite each other.
2024-03-02 08:55:04.040 ERROR (MainThread) [homeassistant.components.sensor] Platform vicare does not generate unique IDs. ID xxx227-room_temperature already exists - ignoring sensor.e3_roomsensor_room_temperature
2024-03-02 08:55:04.041 ERROR (MainThread) [homeassistant.components.sensor] Platform vicare does not generate unique IDs. ID xxx227-room_humidity already exists - ignoring sensor.e3_roomsensor_room_humidity
For historical reasons the gateway serial is used as identifier and not the device serial.
In PyViCare there is a radiator actuator, could that be a floor heating?
What you uploaded is just the log, the diagnostics file can be downloaded from a vicare device in HA when you click on the ...
.
It's two different product, but the end goal is pretty much to do the same when you have a floor heating/cooling. In theory couldn't we use the mac address of the sensor as "unique ID" ? They are just zigbee device so in theory they should have a unique mac address on the network to function properly ?
Isn't the json file I uploaded the diagnostics? Or it's something else I don't see on mobile?
@home-assistant rename Devices do not show up (room sensor, floor heating)
Hi, i am completely new to HA, but i know my way aroud in a terminal. I have a Vicare Setup with a 252a heatpump (with integrated low power Radio), 8 rooms and 12 radiator thermostats. If i can help with testing something let me know.
Device serial for room sensor is now exposed with https://github.com/somm15/PyViCare/pull/369 (release pending), floor heating is still pending https://github.com/somm15/PyViCare/pull/362.
There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.
This is not stale.
Unique id issue was solved (almost) in 2024.09
update. A new PR to add room sensors is open: https://github.com/home-assistant/core/issues/112009
Feel free to test @Roguyt @solbadguy2010
The problem
I've been toying with the dev version of HA to try to make my HomeAssistant show all of my ViCare sensors and connected stuff. I do see the device in the diagnostics as well as the debug log, but I can't get them to display in my HomeAssistant.
If it helps, I'm on the paid plan and my Viessmann installation is the following:
I do see them all on the debug, but not in the generated devices and entities :(
What version of Home Assistant Core has the issue?
core-2024.4.0.dev20240301
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
config_entry-vicare-1e69f700c2d474ca5937934ae2726753 (1).json
Example YAML snippet
No response
Anything in the logs that might be useful for us?
Additional information
No response