Unable to change temperature with ViCare TRVs

[enke5]

The problem

Hi,

I am trying to add my vicare components into home assistant to do some automation. I have expected that I will be able to set the target temperatures of my ViCare thermostats, as I have a developer subscription to the ViCare API.
Unfortunately, the thermostats show up as sensors only. Is this a bug, or is the feature missing?
Another observation: The ids of the thermostats do not show. There are also entries in the debug log, that device.serial datapoint was not offered.

Can you please have a look at this? Thanks in advance. I appreciate your efforts a lot.

Best wishes,
Andreas

What version of Home Assistant Core has the issue?

core-2024.12.4

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

home-assistant_vicare_2024-12-19T13-41-36.000Z.log

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

[home-assistant]

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]

Hey @enke5, you're right, this feature is missing but ready to be tested for quite a while. As I do not have a thermostat and could not find anyone to test this, it is not merged yet.

#127348

It would be great if you could try this. You can do so by running this in there terminal in HA:

curl -o- -L https://gist.githubusercontent.com/bdraco/43f8043cb04b9838383fd71353e99b18/raw/core_integration_pr | bash /dev/stdin -d vicare -p 127348

Regarding the device issue I will have a look, thanks for reporting.

[CFenner]

@enke5 please share also your diagnostics file.

[CFenner]

Serial issue will be fixed with #133619

[CFenner]

@home-assistant rename Unable to change temperature with ViCare TRVs

[enke5]

Hey,
thanks for your prompt reply. I gave it a try, assuming I had to issue your command in /homeassistant directory, but received the following output:
➜ /homeassistant curl -o- -L https://gist.githubusercontent.com/bdraco/43f8043cb04b9838383fd71353e99b18/raw/core_integration_pr | bash /dev/stdin -d domain -p 127348
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
100 2593 100 2593 0 0 7247 0 --:--:-- --:--:-- --:--:-- 7243
INFO: Current dir is /homeassistant
INFO: Cloning core to
Cloning into '/tmp/core_git'...
remote: Enumerating objects: 22022, done.
remote: Counting objects: 100% (22022/22022), done.
remote: Compressing objects: 100% (18405/18405), done.
remote: Total 22022 (delta 4356), reused 8848 (delta 2376), pack-reused 0 (from 0)
Receiving objects: 100% (22022/22022), 22.32 MiB | 9.16 MiB/s, done.
Resolving deltas: 100% (4356/4356), done.
Updating files: 100% (19189/19189), done.
INFO: Checking out PR#127348
remote: Enumerating objects: 553, done.
remote: Counting objects: 100% (553/553), done.
remote: Compressing objects: 100% (314/314), done.
remote: Total 330 (delta 224), reused 42 (delta 14), pack-reused 0 (from 0)
Receiving objects: 100% (330/330), 159.13 KiB | 5.68 MiB/s, done.
Resolving deltas: 100% (224/224), completed with 200 local objects.
From https://github.com/home-assistant/core

• branch refs/pull/127348/head -> FETCH_HEAD
  Note: switching to '1c11848e2de05e4361d00f6e48cbf52b74724450'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

git switch -c

Or undo this operation with:

git switch -

Turn off this advice by setting config variable advice.detachedHead to false

HEAD is now at 1c11848 Merge branch 'dev' into trv
ERROR: Integration with domain does not exist in PR#127348
INFO: Cleaning up

Did I do something wrong?

[CFenner]

@enke5 yes, I had the command wrong at first. It needs to be „-d vicare“ instead of „-d domain“, sorry.

[enke5]

Now it succeeded to switch to your branch, but...
home-assistant_vicare_2024-12-23T05-30-56.659Z.log

[enke5]

@enke5 yes, I had the command wrong at first. It needs to be „-d vicare“ instead of „-d domain“, sorry.

No need to apologize, I am overly grateful for you efforts.

[enke5]

With your latest commit, the devices show up, but setting the temperature does not succeed
home-assistant_vicare_2024-12-23T08-26-06.761Z.log

[CFenner]

Ok, there is an issue with the library I need to fix that first.