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

Regression - Overkiz Integration with Cozytouch hub - Only temperature and Energy comsuption supported , "heatingsystem" device not supported #80375

Closed
gerardsamara opened this issue Oct 15, 2022 · 17 comments · Fixed by #78659 or #78665

Comments

@gerardsamara
Copy link

The problem

After successful integration migration from Custom Component to Core Component , the water heater and heat devices are missing .
The sensors temperature and Energy consumptions are present (OK)

What version of Home Assistant Core has the issue?

2022.10.3 with core component

What was the last working version of Home Assistant Core?

2020.10.3 with custom component

What type of installation are you running?

Home Assistant Supervised

Integration causing the issue

OVERKIZ

Link to integration documentation on our website

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

Diagnostics information

overkiz-bd2ac765386d408e4ccb0ed075a2a77d-HP_Actuator-e6a2bcc841d44914f691d01922d0d7f9.json(1).txt

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No error in the log

Debug mode of Overkiz integration not activated . How to do it ?

Additional information

Debug mode of Overkiz integration not activated . How to do it ?

overkiz core component

@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)

@gerardsamara gerardsamara changed the title Overkiz Integration with Cozytouch hub - Onlry temperature and Energy consuption supported , Heat and water heater missing Overkiz Integration with Cozytouch hub - Only temperature and Energy consuption supported , Heat and water heater missing Oct 15, 2022
@iMicknl
Copy link
Contributor

iMicknl commented Oct 15, 2022

This is not a bug, but a feature request. One of your devices, the AtlanticPassAPCHeatPump / HeatingSystem (io:AtlanticPassAPCHeatPumpMainComponent) is not supported in our integration yet. More context can be found here: iMicknl/ha-tahoma#355.

Can you include the diagnostics file for both of your unsupported devices? Best would be to create another issue for the other device.

@gerardsamara
Copy link
Author

gerardsamara commented Oct 15, 2022

@iMicknl ,

Yes the heatingSytem device is already covered by #355 ha-tahoma as you indicated .
So I have updated the title accordingly .
Another issue will be opened for the other device not supported WaterHeatingsystem.

There are no log info for these devices not supported
There are no diagnostic file for these 2 devices not supported because there are no device created , other than the file already provided , with an extract below :

` "deviceURL": "io://--3482/1086078#1",
"controllableName": "io:AtlanticPassAPCHeatPumpMainComponent",
"widget": "AtlanticPassAPCHeatPump",
"uiClass": "HeatingSystem"

"deviceURL": "io://--3482/1086078#2",
"controllableName": "io:AtlanticPassAPCDHWComponent",
"widget": "AtlanticPassAPCDHW",
"uiClass": "WaterHeatingSystem"

"deviceURL": "io://--3482/1086078#8",
"controllableName": "io:AtlanticPassAPCHeatingAndCoolingZoneComponent",
"widget": "AtlanticPassAPCHeatingAndCoolingZone",
"uiClass": "HeatingSystem"`

Note : The option Cooling is not installed in the PAC

@gerardsamara gerardsamara changed the title Overkiz Integration with Cozytouch hub - Only temperature and Energy consuption supported , Heat and water heater missing Overkiz Integration with Cozytouch hub - Only temperature and Energy comsuption supported , "heatingsystem" device not supported Oct 15, 2022
@gerardsamara gerardsamara changed the title Overkiz Integration with Cozytouch hub - Only temperature and Energy comsuption supported , "heatingsystem" device not supported Regression - Overkiz Integration with Cozytouch hub - Only temperature and Energy comsuption supported , "heatingsystem" device not supported Oct 15, 2022
@nyroDev
Copy link
Contributor

nyroDev commented Oct 16, 2022

Hi @iMicknl and @gerardsamara

I think @gerardsamara has the exact same installation as me (I do have the same firmware and model reference on my setup)

That's why I created #78659 (AtlanticPassAPCHeatingAndCoolingZone) and PR #78665 (AtlanticPassAPCDHW)

AtlanticPassAPCHeatingAndCoolingZone was supported in ha-tahoma

AtlanticPassAPCDHW was supported in ha-tahoma, but as a climate entity.
There was an issue for migrating it from climate to water_heater.

As far as I can see and I checked on the different elements, There is no use for AtlanticPassAPCHeatPump.
Official Somfy UI don't show it (at least for me), and it was not supported in ha-tahoma.
The commands on it are not really standard in the HA world as far as I can tell.

So this issue will be fixed with the 2 PR I mentionned.

@iMicknl Would like me to update my PRs to indicate that ?

@r-jean-pierre
Copy link

I confirm also the "regression" on IPALA H BLC (O741008): there is no "controls" entity.
At least, the "ASAMA Connect" (D725008) it still there

2022-11-01 16_51_05-Settings – Home Assistant

2022-11-01 16_56_22-Settings – Home Assistant

The two different models were okay with the" Overkiz (by Somfy) - Custom component":

2022-11-01 16_54_24-Settings – Home Assistant

@jncanches
Copy link

Same here for the water heater AtlanticDomesticHotWaterProductionV2_CETHI_V4_IOComponent from a cozy touch.
Before HA 2022.11, I was using the custom_component which has the water_heater discovered successfully but now the custom_component cannot start because of a dependency conflict :(
Don't know why water_heater are not created using official integration as they are correctly discovered with custom_component...

@iMicknl
Copy link
Contributor

iMicknl commented Nov 3, 2022

Don't know why water_heater are not created using official integration as they are correctly discovered with custom_component...

This is due to time constraints. The first water heater implementation is currently in review, you can check the open PR's on core yourself. Moving components to core just takes time, not only on the side of the development, but also on the review side. It took us more than a year just to get the basic functionality from custom component to core..

You can keep using the custom component for now, if it is broken after 2022.11, you can create an issue on the custom component repo.

@antibill51
Copy link

Hi.
Maybe that can help you?

@jncanches
Copy link

Don't know why water_heater are not created using official integration as they are correctly discovered with custom_component...

This is due to time constraints. The first water heater implementation is currently in review, you can check the open PR's on core yourself. Moving components to core just takes time, not only on the side of the development, but also on the review side. It took us more than a year just to get the basic functionality from custom component to core..

You can keep using the custom component for now, if it is broken after 2022.11, you can create an issue on the custom component repo.

I totally understand that! My apologies if my words has suggested a reproach. That was not my purpose.

Your work is highly appreciated!

@iMicknl
Copy link
Contributor

iMicknl commented Nov 3, 2022

No worries. I have created a test version of the custom component that should fix the issues. Hope to merge and release it soon.

Regarding the other issues, we have some more contributors now, thus we should be able to merge some in soon. I am still quite swamped (and focusing on the local integration), but I will see if we can merge in some.

@antibill51 would you perhaps like to help in the development? You can just create a PR for the core integration, but best is to briefly check on our Discord if others are not working on the same device :).

@nyroDev
Copy link
Contributor

nyroDev commented Nov 4, 2022

This issue was closed because the initial issue was the missing of AtlanticPassAPCHeatingAndCoolingZone and AtlanticPassAPCDHW.
The relative PR have been merged, and closed this issue autmatically.

@r-jean-pierre You're issue seems to be the missing of AtlanticElectricalHeaterWithAdjustableTemperatureSetpoint which sould be ported in HA-Core.
I'll take a look in a week or so to do that.

@jncanches I couldn't find any place in Ha-Tahoma for AtlanticDomesticHotWaterProductionV2_CETHI_V4_IOComponent
There is AtlanticDomesticHotWaterProductionV2_CV4E_IOComponent
Is it the same and should it works the same?
For this last one, I just created a new PR to port DomesticHotWaterProduction

@jncanches
Copy link

jncanches commented Nov 4, 2022

@nyroDev I don't know if it will work the same. I don't have elements to know that.
AtlanticDomesticHotWaterProductionV2_CETHI_V4_IOComponent is the device model shown by the cozy-touch device.

If AtlanticDomesticHotWaterProductionV2_CV4E_IOComponent is the same as the model used in the custom component, it should work :-)

@nyroDev
Copy link
Contributor

nyroDev commented Nov 4, 2022

@r-jean-pierre it looks like there is already a PR ready for integration made by @tetienne

@tetienne
Copy link
Contributor

tetienne commented Nov 4, 2022

@r-jean-pierre it looks like there is already a PR ready for integration made by @tetienne

Indeed I will try to finish it next week as there was several request for it.

@r-jean-pierre
Copy link

@r-jean-pierre it looks like there is already a PR ready for integration made by @tetienne

Yep it seems, I deleted the HA core integration and switched back to the Overkiz Custom component which with the new update is running well

@r-jean-pierre
Copy link

Just to be sure to understand: soon or later, all the custom components will be part of one unique official HA-core integration?

@nyroDev
Copy link
Contributor

nyroDev commented Nov 4, 2022

Yes, and we're getting close to it: iMicknl/ha-tahoma#697

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