Mmodarre / pyfujitsu_for_homeassistant

MIT License
31 stars 33 forks source link

Addon no longer works with homeassistant 0.96 #11

Closed chiefcomm closed 4 years ago

chiefcomm commented 5 years ago

As per title the add on no longer works in hassio 0.96.x, it seems the changes to Climate in 0.96 may have broken the integration for pyfujitsu as there are quite a few climate related issues in 0.96 :-(

The error I'm seeing in logs is "Integration fujitsu_general_heatpump not found when trying to verify its climate platform"

Hope there are some suggestions out there, in the mean time I'm rolling back to 0.95.4 which was last know stable for pyfujitsu ....

https://developers.home-assistant.io/blog/2019/07/03/climate-cleanup.html

leonlmf commented 5 years ago

Very annoying with this 0.96 both of my climate Fujitsu and tuya both got problems

lyapich commented 5 years ago

ХА изменили компонент Climate кардинально, в нем проблема. Вот, что в описании:

Климат
chiefcomm commented 5 years ago

@Mmodarre & @brent0np any chance you gents can help? I've looked at the climate code changes between 0.95.4 & 0.96.x and frankly my skills with python are completely insufficient to tackle this. Any assistance you could offer here would be greatly appreciated by those of us using Pyfujitsu.

Mmodarre commented 5 years ago

Hi All,

Very glad that the component has helped so many people.

I am aware of breaking changes and for that reason refrained from updating my HA.

Currently travelling for work and will try to retrofit the code ro new climate platform once I got the chance. If anyone elese also can help out it would be much appreciate. More than happy to review the code.

On Sun, 21 Jul. 2019, 20:10 chiefcomm, notifications@github.com wrote:

@Mmodarre https://github.com/Mmodarre & @brent0np https://github.com/brent0np any chance you gents can help? I've looked at the climate code changes between 0.95.4 & 0.96.x and frankly my skills with python are completely insufficient to tackle this. Any assistance you could offer here would be greatly appreciated by those of us using Pyfujitsu.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Mmodarre/pyfujitsu_for_homeassistant/issues/11?email_source=notifications&email_token=AFSPWGAQHZ5TGMRXSEBY4Q3QAUQMFA5CNFSM4IFBM4AKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2OUOUQ#issuecomment-513623890, or mute the thread https://github.com/notifications/unsubscribe-auth/AFSPWGAIZEP6UXTEYAWLBQLQAUQMFANCNFSM4IFBM4AA .

graperkins commented 4 years ago

Recently upgraded and having the same problem

"Entity not available: climate.fujitsu_heatpump"

Doesn't matter much during the summer months, but in the winter I rely on Home Assistant Automations to lower the heating temperate when nobody is home and save $$$ on heating cost in Canada.

meichthys commented 4 years ago

@Mmodarre Has there been any update on this? I'm still seeing the same error wen trying to validate my configuration:

image

There appears to be a pull request (#13) that would fix this although i haven't been able to verify if the pull request fixes the issue.

meichthys commented 4 years ago

Some changes need to be made, but it will work using the changes that are discussed in #14

meichthys commented 4 years ago

@chiefcomm Can we close this now? The more recent issue for this is #16

chiefcomm commented 4 years ago

@chiefcomm Can we close this now? The more recent issue for this is #16

@meichthys sure go ahead and close it - not relevant post the changes made to support later versions.