search

thevoltagesource / LennoxiComfort

Home Assistant custom component for controlling Lennox iComfort WiFi and AirEase Comfort Sync thermostats.
MIT License
33 stars 16 forks source link

Warning after upgrading to 2024.4 #48

Open mattgphoto opened 5 months ago

mattgphoto commented 5 months ago

The myicomfort custom integration is using deprecated climate auxiliary heater This stops working in version 2024.10.0. Please address before upgrading. The custom integration myicomfort implements the is_aux_heat property or uses the auxiliary heater methods in a subclass of ClimateEntity.

Please create a bug report at https://github.com/thevoltagesource/LennoxiComfort/issues.

Once an updated version of myicomfort is available, install it and restart Home Assistant to fix this issue.

Home Assistant About Page: Core - 2024.4.0 Supervisor - 2024.03.1 Operating System - 12.1 Frontend - 20240403.1

rberger commented 5 months ago

I'm seeing this as well and its not clear what action to take.

shaunvaidyan commented 5 months ago

I'm getting this issue as well

Sinfonia97 commented 5 months ago

I've been getting this for the last few releases myself.

ColoradoMark commented 5 months ago

Same here

SJLBoulder commented 5 months ago

Same here.

The myicomfort custom integration is using deprecated climate auxiliary heater This stops working in version 2024.10.0. Please address before upgrading. The custom integration myicomfort implements the is_aux_heat property or uses the auxiliary heater methods in a subclass of ClimateEntity.

Please create a bug report at https://github.com/thevoltagesource/LennoxiComfort/issues.

Once an updated version of myicomfort is available, install it and restart Home Assistant to fix this issue.

Warning - 4/11/2024

About: Home Assistant

thevoltagesource commented 5 months ago

Thanks for making me aware of this issue. I will dig in to this next week and see what needs to change.

Yomegaman commented 5 months ago

Thanks Jacob!

ColoradoMark commented 5 months ago

Thank you!

brianweb16 commented 2 months ago

Thanks for making me aware of this issue. I will dig in to this next week and see what needs to change.

Just checking in to see if this is still going to be resolved? Thanks!

Kev1000000 commented 1 month ago

Any updates on this? We're getting close to the removal in 2024.10 =(

ColoradoMark commented 1 month ago

Please..

stilldisturbing commented 1 month ago

@thevoltagesource I attempted a fix the other month and submitted a pull request (partial fix I would say as it doesn't add a new entity for the Aux Heat/Emergency Heat, however been running it for months and no issues for me).

ColoradoMark commented 1 month ago

Can we try it somehow? The integration is great and I would like to keep it.

thevoltagesource commented 1 month ago

Hi All. I apologize for the delay in getting this fixed, this summer did not go as planned. I did see the PR to strip the feature and resolve the warning, but I am saving that as the last ditch option. I had originally added the aux heat item to fix issues for some users that had aux heat as part of their system, so I'm hoping to not break their integration in the process.

My goal is to get this resolved by the end of the month.

ColoradoMark commented 1 month ago

Great, I really appreciate your work. If I can help test or anything let me know.

stilldisturbing commented 1 month ago

Hi All. I apologize for the delay in getting this fixed, this summer did not go as planned. I did see the PR to strip the feature and resolve the warning, but I am saving that as the last ditch option. I had originally added the aux heat item to fix issues for some users that had aux heat as part of their system, so I'm hoping to not break their integration in the process.

My goal is to get this resolved by the end of the month.

Figured that and understand. If you need any testers, happy to help out. Like others, really appreciate the work you put into this.

Talfurn commented 5 days ago

Hi All. I apologize for the delay in getting this fixed, this summer did not go as planned. I did see the PR to strip the feature and resolve the warning, but I am saving that as the last ditch option. I had originally added the aux heat item to fix issues for some users that had aux heat as part of their system, so I'm hoping to not break their integration in the process.

My goal is to get this resolved by the end of the month.

So, anybody have any alternatives they like?

ColoradoMark commented 5 days ago

I haven't found any alternative. I appreciate your work on this. I can't code, but if there is anything else I can do to help let me know.

Yomegaman commented 5 days ago

I'm not a coder either. I have two iComfort Thermostats in my home that I use in multiple automations. I'd hate to buy and replace my iComforts with a pair of Lennox S40 T-stats. S40s ARE supported by another developer.

To avoid that replacement and reprogramming hassle - I'd be happy to pledge $50 to "thevoltagesource" for both my iComforts ($100 in my case.) Maybe others in this thread would see the value in a similar contribution. That $100 is a small token of my appreciation for the years of flawless iComfort integration with Home Assistant.

Time's running short - if you're not going to update the integration, could you please let us all know? If you will be repairing the integration before the October HA release please let me know how I can make my promised contribution.

Respectfully;

"Yomegaman" in AZ

On Fri, Sep 6, 2024 at 11:17 AM ColoradoMark @.***> wrote:

I haven't found any alternative. I appreciate your work on this. I can't code, but if there is anything else I can do to help let me know.

— Reply to this email directly, view it on GitHub https://github.com/thevoltagesource/LennoxiComfort/issues/48#issuecomment-2334589259, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMPWJO74QBHCOPJGJJENGULZVHWSTAVCNFSM6AAAAABFWFCTNWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZUGU4DSMRVHE . You are receiving this because you are subscribed to this thread.Message ID: @.***>

thevoltagesource commented 5 days ago

No need for contributions, accepting money takes the sense of obligation up to a whole new level and I don't need any more pressure in my life. 😄 But I will try to keep this thing working for the community. I find it super cool and slightly terrifying that random people on the internet depend on my code.

I used the PR that @stilldisturbing submitted to pull the Aux Heat mode out of the integration. I realized the rest of the code handles the responses from the myicomfort API properly to not break the integration if the thermostat has the emergency heat mode turned on. You just can't control that option from the integration any longer. Let me know if there are any issues.

I am not a coder, but I understand code. This means that given enough time, determination, and examples, I can generate the lines I need to solve to solve a problem. So much has changed in the way HA integrations are written since I wrote this thing. The examples of the old ways are gone and it is overwhelming to see the new style/process/design. I do need a challenge to get me out of the funk I am in, so maybe I should work on learning the new way and rebuild this thing.

My biggest challenge is that my iComfort died 5 years ago. It was replaced under warranty, but with an S30. My iComfort account still works (or it did at the beginning of the year) so I can still connect, send commands, and read the last status my thermostat reported so long ago, I just can't verify that everything translates to the device. So, if I do start rebuilding, I can alpha test on the ghost of my thermostat, but I will need some beta testers.

ColoradoMark commented 4 days ago

I will be happy to help beta test.

Yomegaman commented 4 days ago

Jacob;

I'm the guy who offered you a $50 per T-stat token of appreciation. I completely understand your reason for saying "thanks, but no thanks" on that offer.

Now that I understand that you don't even have an iComfort to experiment with, I've come up with another idea that might really assist you.

I have two working iComforts in my home. I would happily share my iComfort account ID and password with you, so you can use one of my iComforts, remotely, for your testing. I'd be happy to work with you over-the-phone as you do your alpha testing.

Hey, just so you know a little about me - I invented and patented the "Yo-Yo with a Brain" when I was a college student. That yo-yo contains a centrifugal clutch. When you throw the yo-yo down, the clutch opens and spins freely at the bottom of the sting. When the yo-yo slows, the clutch closes and it returns to your hand automatically. You can still find these yo-yos on Amazon when you search "Yomega Brain Yo-Yo." (or you can research via US mechanical patent # 4,332,102)

Please let me know what you think of my proposal - I'm ready to assist ASAP.

Mike

On Sat, Sep 7, 2024 at 3:30 AM ColoradoMark @.***> wrote:

I will be happy to help beta test.

— Reply to this email directly, view it on GitHub https://github.com/thevoltagesource/LennoxiComfort/issues/48#issuecomment-2335145555, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMPWJO7DV5M7GQHIWIQHDO3ZVLIUNAVCNFSM6AAAAABFWFCTNWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZVGE2DKNJVGU . You are receiving this because you are subscribed to this thread.Message ID: @.***>