Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AtlanticElectricalHeaterWithAdjustableTemperatureSetpoint not supported in Overkiz integration #65427

Closed
thundergreen opened this issue Feb 2, 2022 · 47 comments · Fixed by #72790

Comments

@thundergreen
Copy link

thundergreen commented Feb 2, 2022

The problem

I used to use the custom integration of https://github.com/iMicknl/ha-tahoma for my Atlantic Cozytouch Hub to control my heaters. Aber now this custom integration has found its way into core, i miss the climate entities for my Cozytouch Heaters (Malao). With the "old" Tahoma custom integration i get all entities created:

image

But with the new Overkiz integration those climate entities are completely missing:

image

What version of Home Assistant Core has the issue?

2022.2.0b6

What was the last working version of Home Assistant Core?

never

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Home Assistant OS

Link to integration documentation on our website

https://rc.home-assistant.io/integrations/overkiz

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Additional information

2022-02-02 13:53:16 INFO (MainThread) [homeassistant.setup] Setting up overkiz
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.setup] Setup of domain overkiz took 0.0 seconds
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.components.binary_sensor] Setting up binary_sensor.overkiz
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.components.button] Setting up button.overkiz
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.components.cover] Setting up cover.overkiz
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.components.light] Setting up light.overkiz
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.components.lock] Setting up lock.overkiz
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.components.number] Setting up number.overkiz
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.components.scene] Setting up scene.overkiz
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.components.sensor] Setting up sensor.overkiz
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.components.switch] Setting up switch.overkiz
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new binary_sensor.overkiz entity: binary_sensor.io_13570863_3_contact_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new binary_sensor.overkiz entity: binary_sensor.io_13570863_4_occupancy_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new binary_sensor.overkiz entity: binary_sensor.io_4057317_3_contact_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new binary_sensor.overkiz entity: binary_sensor.io_4057317_4_occupancy_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new button.overkiz entity: button.radiateur_start_identify
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new button.overkiz entity: button.radiateur_identify
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new button.overkiz entity: button.radiateur_stop_identify
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new button.overkiz entity: button.radiateur_start_identify_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new button.overkiz entity: button.radiateur_identify_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new button.overkiz entity: button.radiateur_stop_identify_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new button.overkiz entity: button.pass_actuator_start_identify
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new button.overkiz entity: button.pass_actuator_identify
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new button.overkiz entity: button.pass_actuator_stop_identify
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new scene.overkiz entity: scene.passage_en_absence_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new scene.overkiz entity: scene.mode_confort_bathroom_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new scene.overkiz entity: scene.mode_eco_bathroom_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.radiateur_discrete_rssi_level
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.radiateur_priority_lock_timer
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.radiateur_rssi_level
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.radiateur_priority_lock_originator
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_13570863_2_temperature_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_13570863_5_consumption_tariff_1_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_13570863_5_consumption_tariff_2_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_13570863_5_consumption_tariff_3_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_13570863_5_consumption_tariff_4_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_13570863_5_consumption_tariff_5_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_13570863_5_consumption_tariff_6_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_13570863_5_consumption_tariff_7_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_13570863_5_consumption_tariff_8_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_13570863_5_consumption_tariff_9_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_13570863_5_electric_energy_consumption_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.radiateur_discrete_rssi_level_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.radiateur_priority_lock_timer_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.radiateur_rssi_level_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.radiateur_priority_lock_originator_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_4057317_2_temperature_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_4057317_5_consumption_tariff_1_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_4057317_5_consumption_tariff_2_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_4057317_5_consumption_tariff_3_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_4057317_5_consumption_tariff_4_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_4057317_5_consumption_tariff_5_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_4057317_5_consumption_tariff_6_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_4057317_5_consumption_tariff_7_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_4057317_5_consumption_tariff_8_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_4057317_5_consumption_tariff_9_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.io_4057317_5_electric_energy_consumption_2
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.pass_actuator_discrete_rssi_level
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.pass_actuator_priority_lock_timer
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.pass_actuator_rssi_level
2022-02-02 13:53:16 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.overkiz entity: sensor.pass_actuator_priority_lock_originator
2022-02-02 13:59:04 INFO (MainThread) [homeassistant.components.binary_sensor] Setting up binary_sensor.overkiz
2022-02-02 13:59:04 INFO (MainThread) [homeassistant.components.button] Setting up button.overkiz
2022-02-02 13:59:04 INFO (MainThread) [homeassistant.components.cover] Setting up cover.overkiz
2022-02-02 13:59:04 INFO (MainThread) [homeassistant.components.light] Setting up light.overkiz
2022-02-02 13:59:04 INFO (MainThread) [homeassistant.components.lock] Setting up lock.overkiz
2022-02-02 13:59:04 INFO (MainThread) [homeassistant.components.number] Setting up number.overkiz
2022-02-02 13:59:04 INFO (MainThread) [homeassistant.components.scene] Setting up scene.overkiz
2022-02-02 13:59:04 INFO (MainThread) [homeassistant.components.sensor] Setting up sensor.overkiz
2022-02-02 13:59:04 INFO (MainThread) [homeassistant.components.switch] Setting up switch.overkiz
2022-02-02 13:58:46 INFO (MainThread) [homeassistant.components.binary_sensor] Setting up binary_sensor.overkiz
2022-02-02 13:58:46 INFO (MainThread) [homeassistant.components.button] Setting up button.overkiz
2022-02-02 13:58:46 INFO (MainThread) [homeassistant.components.cover] Setting up cover.overkiz
2022-02-02 13:58:46 INFO (MainThread) [homeassistant.components.light] Setting up light.overkiz
2022-02-02 13:58:46 INFO (MainThread) [homeassistant.components.lock] Setting up lock.overkiz
2022-02-02 13:58:46 INFO (MainThread) [homeassistant.components.number] Setting up number.overkiz
2022-02-02 13:58:46 INFO (MainThread) [homeassistant.components.scene] Setting up scene.overkiz
2022-02-02 13:58:46 INFO (MainThread) [homeassistant.components.sensor] Setting up sensor.overkiz
2022-02-02 13:58:46 INFO (MainThread) [homeassistant.components.switch] Setting up switch.overkiz

As you can see NO climate entity is being setup.

@probot-home-assistant
Copy link

Hey there @iMicknl, @vlebourl, @tetienne, mind taking a look at this issue as it has been labeled with an integration (overkiz) you are listed as a code owner for? Thanks!
(message by CodeOwnersMention)


overkiz documentation
overkiz source
(message by IssueLinks)

@tetienne
Copy link
Contributor

tetienne commented Feb 2, 2022

Hi, the Core integration is still a work in progress. While we haven’t yet fully backport the whole custom component, you can still use it.

@thundergreen
Copy link
Author

I am running both now because on custom component I got keyerror: username. Once added the overkiz integration my custom component started working again.

All right. then i will wait the official release to get climate entities?

@tetienne
Copy link
Contributor

tetienne commented Feb 2, 2022

If you got an error with the custom component, you can submit us an issue: https://github.com/iMicknl/ha-tahoma

@vlebourl
Copy link
Contributor

vlebourl commented Feb 2, 2022

All right. then i will wait the official release to get climate entities?

climate and water_heater will be addressed soon. For now they are not yet integrated into core...

@NicolasBoulanger
Copy link

All right. then i will wait the official release to get climate entities?

climate and water_heater will be addressed soon. For now they are not yet integrated into core...

Hello, same plan for home alarm ?thanks

@iMicknl
Copy link
Contributor

iMicknl commented Feb 4, 2022

Please have a look at iMicknl/ha-tahoma#697 for the roadmap. (subject to change, order could change).

@thundergreen
Copy link
Author

Stupid question wh it has been merged without being finished ?

@iMicknl
Copy link
Contributor

iMicknl commented Feb 4, 2022

Stupid question wh it has been merged without being finished ?

Not having support for all available devices does not equal 'not finished'. If you see the roadmap, you see the huge amount of pull requests that are required to get even basic functionality in. We are dependent on code reviews and the release cycle of core.

In the past we have tried to bring our huge custom integration all at once, however this is just not doable. The review times will be too long and in the mean-time the project has already had so many changes, that you need to refactor your code again. The velocity of core is very fast and thus the best strategy is to bring integrations piece by piece.

And to be honest, I don't even see why this would be an issue for end-users. The supported platforms are documented and in the mean-time you can still use the custom integration.

@tetienne
Copy link
Contributor

tetienne commented Feb 4, 2022

@thundergreen
Copy link
Author

Sorry all I don't want to offend at all sorry sorry sorry I really appreciate the hard work and awesome efforts in all your work . I should sometimes maybe just shit the fuck up and be patient

@arandax
Copy link

arandax commented Feb 5, 2022

I can't use the custom HACS tahoma because it says "You cannot use Overkiz from core and custom component at the same time" so now I'm without climate entities

@iMicknl
Copy link
Contributor

iMicknl commented Feb 5, 2022

I can't use the custom HACS tahoma because it says "You cannot use Overkiz from core and custom component at the same time" so now I'm without climate entities

Please report this issue at ha-tahoma. Make sure you update to the last version of the custom component first. We added a check to make sure that users don't configure both integrations at the same time.

@arandax
Copy link

arandax commented Feb 5, 2022

Thank you, just updated though

@iMicknl
Copy link
Contributor

iMicknl commented Feb 24, 2022

@thundergreen could you include your diagnostics file here? Go to your device page and click on "Download Diagnostics", now you can just drag and drop the text file here in a comment. (or your first post).

@thundergreen
Copy link
Author

Hello. Last version of overkiz gave me even less:

image

no climate entities at all . Also No sensors. Here attached my diagnosis:
config_entry-overkiz-7d0e35eff8cdf6c74febdf492d7fb8aa.json.txt

hope it helps.

I have 2 Sauter Malao Heateers and 1x Interface IO for the Bathroom heater. If you need further info, please let me know.
http://casammedia.s3.eu-west-3.amazonaws.com/uploads/Cat%C3%A9gories/Accessoires%20Chauffage%20Electrique/Pass%20Cozytouch/Interface%20Cozytouch%20Pass%20IO%20Fil%20Pilote.jpg

@iMicknl
Copy link
Contributor

iMicknl commented Feb 24, 2022

I don't understand what you mean with the last version of Overkiz. If you talk about core, nothing should have changed. If you talk about the custom component, the core repository is not the place to report issues.

However, I will notify you when we implemented your climate devices:

{
	"widgetName": "AtlanticElectricalHeaterWithAdjustableTemperatureSetpoint",
	"uiClass": "HeatingSystem",
	"qualifiedName": "io:AtlanticElectricalHeaterWithAdjustableTemperatureSetpointIOComponent"
}

This one is already implemented in 2022.3.b0, and will thus be in the 2022.3 release.

{
	"widgetName": "AtlanticElectricalHeater",
	"uiClass": "HeatingSystem",
	"qualifiedName": "io:AtlanticElectricalHeaterIOComponent"
}

@thundergreen
Copy link
Author

thundergreen commented Feb 24, 2022

I use the custom component (Tahoma) as well as the core integration (Overkiz)

image

As you can see that core integration just has 3 entities where tahoma custom component has 57. I am bloody beta tester and will report once I got some more info. Thanks a lot.

EDIT: I just saw i am already using core-2022.3.0b0

image

@bran2000
Copy link

Hello, it seems i met some problem since yesterday with the overkiz integration, i tried to uninstall the custom component and let the new one overkiz (Hassio found automatically) and i tried to keep both but i have still an error when i check my configuration and i can't restart HA with this error, see my logs :

Error occurred loading configuration flow for integration tahoma: Exception importing custom_components.tahoma.config_flow
8:31:10 AM – (ERROR) config_entries.py - message first occurred at February 27, 2022, 9:44:28 PM and shows up 252 times
Unexpected exception importing component custom_components.tahoma
8:31:10 AM – (ERROR) Overkiz (by Somfy) - Custom component (custom integration) - message first occurred at February 27, 2022, 9:44:16 PM and shows up 253 times

Any idea ?Thanks for your help, for information it was perfectly working before, i haven't done any HA upgrade, my version is Home Assistant 2022.2.9

@thundergreen
Copy link
Author

@bran2000 you cannot load both components at the same time. Either Tahoma or core . But normally you should also see thos message on the integration widget itself.

@iMicknl
Copy link
Contributor

iMicknl commented Feb 28, 2022

@bran2000 first of all, please create a new issue since this is not related. If you have an issue with the Overkiz integration, create it here on core, otherwise create it on ha-tahoma repository.

As @thundergreen mentioned, you should not use both integration at the same time, however normally you should have a nice error message. Please update to the latest ha-tahoma integration and if it still has an error, turn on debug logging and create an issue on ha-tahoma repository.

For now, I would advise to remove your current config entry and just set it up via Overkiz from core.

@bran2000
Copy link

bran2000 commented Mar 1, 2022

THANKS for your replys i only leave the core version for the moment no error configuration, i'll restart my HA tomorrow.
Thanks again

@brun0-t
Copy link

brun0-t commented Apr 22, 2022

Hi,

Like @thundergreen since the big update of Overkiz (by Somfy) i miss the climate entities for my Cozytouch Heaters

With the last version of Overkiz (by Somfy) and the core_2021..., i can create for my Gyali radiator a Climate card.

Do you have a solution or can you tell me if the climate card will be back or totally abandoned ?

Thank you in advance.

@iMicknl
Copy link
Contributor

iMicknl commented Apr 22, 2022

@brun0-t please create a new issue with your diagnostic information attached.

However, on a quick glance it seems you use your temperature sensor and not your climate entity...

@brun0-t
Copy link

brun0-t commented Apr 22, 2022

@iMicknl Ok.

#70455

@arandax
Copy link

arandax commented May 29, 2022

Will they be implemented in future updates?

@frenck frenck modified the milestone: 2022.6.0 May 30, 2022
@Loustik
Copy link

Loustik commented May 30, 2022

Will they be implemented in future updates?

I don't know actually... It's been I while since I'm following this issue : iMicknl/ha-tahoma#697. The tick box related to "Climate platform" remains unchecked, and moreover it shifts from month to month... Initially scheduled in February, then March then April... Finally "May+++" when climate platforms will be less useful than in winter. I don't know if there is any activity, I've seen 2 times the automatic robot setting the "no-issue-activity" label, and the admin systematically removes it... I really don't know.

@iMicknl
Copy link
Contributor

iMicknl commented May 30, 2022

@arandax @Loustik implementation of climate devices is not a quick and easy task. We rewrite all old implementations to match the core codestyle, and without owning a device, this is hard sometimes.

See https://github.com/home-assistant/core/pulls?q=is%3Apr+is%3Aopen+label%3A%22integration%3A+overkiz%22+ for the open pull-requests at core. Since the load on core is high (370 PR's open, 1.3K issues) the review time can be quite long. This means that we cannot just move the climate implementation as fast as with the custom component. Currently there are 2 new climate implementations waiting for review.

What helps to see the demand, is to at least understand which climate devices we should prioritize. There are a lot of them, and some are just used by a single user.

@Loustik
Copy link

Loustik commented May 30, 2022

@iMicknl we know that, for sure... We are supporting you . Moreover, rewriting all legacy code to reduce technical debt & adapt to the new APIs is great, so I've forgotten to tell you, but at first thank you to do so...

We, as end users, have some devices, and could test some features for you :)

@iMicknl
Copy link
Contributor

iMicknl commented May 30, 2022

@Loustik what is your climate device? (just post the model + hardware here). And if possible, your diagnostics.

@arandax
Copy link

arandax commented May 30, 2022

I have 2 hitachi air-to-air heatpump / air conditioners with HLRR (SPX-TAG01) interface on both.
Here is my diagnostic (hope is the correct one that you need.)
config_entry-overkiz-c41eeefc5fcd466ddaf8a5686c8baaca.json.txt

@iMicknl
Copy link
Contributor

iMicknl commented Jun 26, 2022

@arandax you have the HitachiAirToAirHeatPump (HitachiHeatingSystem) - ovp:HLinkMainController. I will see if I can make a PR for that one as well.

Does this current version in the custom component work 100%?

@raman325 raman325 changed the title Overkiz Integraton not creating Climate entities AtlanticElectricalHeaterWithAdjustableTemperatureSetpoint not supported in Overkiz integration Jun 27, 2022
@arandax
Copy link

arandax commented Jun 27, 2022

@arandax you have the HitachiAirToAirHeatPump (HitachiHeatingSystem) - ovp:HLinkMainController. I will see if I can make a PR for that one as well.

Does this current version in the custom component work 100%?

Thank you for your reply!
Was working when I was using it, now I've switched to Overkiz integration quite some months ago

@Loustik
Copy link

Loustik commented Jun 27, 2022

@Loustik what is your climate device? (just post the model + hardware here). And if possible, your diagnostics.

@iMicknl, I have many Atlantic Heaters from the ONIRIS product range and one Atlantic ElectricalDryer from the SERENIS product range.

@browetd
Copy link

browetd commented Aug 9, 2022

My three Thermor radiators (Equateur 3) were working fine with the custom component but are not recognized (no climate entities created but a bunch of sensors, binary_sensors, etc...) I will continue to use the custom component...

@iMicknl
Copy link
Contributor

iMicknl commented Aug 9, 2022

@browetd when this issue is closed (and #72790 is merged), it will be added to Home Assistant core. In the mean-time you would need to use the custom component.

@Voileux78
Copy link

Hi,
I fully support you in doing such a great job, and I understand the difficulties you have to determine the priorities…
The same for me as for Browetd, my 12 Thermor radiations (Équateur 3) were being controlled fine by the custom component, I had a lot of automations working to control each radiator depending on several criteria, all was working fine.
After the migration in V 2022.2, I adapted my automations and I succeeded in making them work, by using the value setting on the ‘number.xxxx_eco_room_temperature’ entity for each radiator. It was working fine
Unfortunately, it doesn’t work anymore.
I guess the only thing which seems to be missing in my case is that the value setting service cannot be applied to the ‘number.xxxx_eco_room_temperature’ entity (the entity is not recognized by the service number_set_value).
Any suggestion to solve this issue ?
Many thanks in advance

@tetienne
Copy link
Contributor

tetienne commented Nov 5, 2022

Hi 👋
Sorry for this long delay, I was quite busy since few month, and still now.
I’ve applied latest modification to #72790
If some of you want to test this PR, feed back are welcome.

@Voileux78
Copy link

Voileux78 commented Nov 5, 2022 via email

@tetienne
Copy link
Contributor

tetienne commented Nov 7, 2022

@thundergreen @Voileux78 @browetd @Loustik If any of you can’t test the associated PR, will you accept to share with me on Discord your credentials (that will have change temporary of course), so I can confirm the behavior of my implementation?

@thundergreen
Copy link
Author

thundergreen commented Nov 7, 2022 via email

@browetd
Copy link

browetd commented Nov 7, 2022

I installed the latest version of the custom components (V 2.14.0) and it is not working for me. The warning messages disappeared but now, I have the following error messages (I have three Equateur 3 radiators from Thermor):

2022-11-07 12:35:50.061 ERROR (MainThread) [custom_components.tahoma] Error fetching device events data: Too many requests, try again later.
2022-11-07 12:35:50.104 ERROR (MainThread) [custom_components.tahoma.climate_devices.atlantic_electrical_heater_with_adjustable_temperature_setpoint] Unable to update from sensor: could not convert string to float: 'unavailable'
2022-11-07 12:35:50.105 ERROR (MainThread) [custom_components.tahoma.climate_devices.atlantic_electrical_heater_with_adjustable_temperature_setpoint] Unable to update from sensor: could not convert string to float: 'unavailable'
2022-11-07 12:35:50.105 ERROR (MainThread) [custom_components.tahoma.climate_devices.atlantic_electrical_heater_with_adjustable_temperature_setpoint] Unable to update from sensor: could not convert string to float: 'unavailable'
2022-11-07 12:35:53.213 ERROR (MainThread) [custom_components.tahoma] Error fetching device events data: Too many requests, try again later.
2022-11-07 12:35:53.255 ERROR (MainThread) [custom_components.tahoma.climate_devices.atlantic_electrical_heater_with_adjustable_temperature_setpoint] Unable to update from sensor: could not convert string to float: 'unavailable'
2022-11-07 12:35:53.256 ERROR (MainThread) [custom_components.tahoma.climate_devices.atlantic_electrical_heater_with_adjustable_temperature_setpoint] Unable to update from sensor: could not convert string to float: 'unavailable'
2022-11-07 12:35:53.257 ERROR (MainThread) [custom_components.tahoma.climate_devices.atlantic_electrical_heater_with_adjustable_temperature_setpoint] Unable to update from sensor: could not convert string to float: 'unavailable'

Not sure if the version of the custom component (V 2.14.0) is including this PR... Let me know if you need more details...

@iMicknl
Copy link
Contributor

iMicknl commented Nov 7, 2022

@browetd the custom component is using totally different code and is quite outdated. This issue (+ PR) is for the core component and will hopefully fix all your issues.

@browetd
Copy link

browetd commented Nov 7, 2022

OK I will reinstall the core component but how can I apply this specific PR ?

Update: I finally found a way to copy the code to a custom_components to test the PR... but unfortunately it is not working (custom_components not starting due to unknown modules)

@tetienne
Copy link
Contributor

tetienne commented Nov 7, 2022

@browetd Yes indeed, it’s no so easy to test a PR from the Core repository :(

Can you tell me if your device support any of these modes?

{
        "type": "DiscreteState",
        "values": [
          "antifreeze",
          "auto",
          "away",
          "eco",
          "frostprotection",
          "manual",
          "max",
          "normal",
          "off",
          "on",
          "prog",
          "program",
          "boost"
        ],
        "qualifiedName": "core:OperatingModeState"
      }

@iMicknl
Copy link
Contributor

iMicknl commented Nov 7, 2022

@browetd you can always come to our Discord for support! https://discord.gg/3Ee6s5Ar.
You should be able to copy the components/overkiz to custom_components and add a version key to the manifest.json.

@browetd
Copy link

browetd commented Nov 8, 2022

@tetienne The modes/presets_mode I am using are: "auto", "off", "prog", "frostprotection" and "comfort"...

@github-actions github-actions bot locked and limited conversation to collaborators Dec 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.