Smarter Coffee / iKettle

nerdtech · May 5, 2024, 1:45pm

Just so i get it right? With component you mean your integration? Or a component in homeassistant?

And @kbirger also want to say big thank you for all your support!

kbirger · May 5, 2024, 1:48pm

You’re welcome.

I mean my integration. In home assistant, integrations are installed to a folder called custom_components

nerdtech · May 5, 2024, 1:53pm

In hacs it says i use version 042cf8f and according to github that is the latest commit made 2 weeks ago… I will try to delete it from hacs and reboot and then reinstall to see if that will help…

kbirger · May 5, 2024, 2:00pm

I didn’t have time to figure out how to properly publish to hacs. I was hurrying to get an MVP out to people. And then I got laid off. So now I’m job hunting instead

nerdtech · May 5, 2024, 2:08pm

Sorry to hear that you got laid off! But with your skills im sure that you will soon get a new one!

Is there a newer version than the one on github? Or should manual get it from github instead of using the link in hacs?

nerdtech · May 5, 2024, 2:11pm

The reinstall thru hacs didnt help same error in logs…

kbirger · May 5, 2024, 3:24pm

Thanks for the kind words. The GitHub version is the latest.

How much experience with Linux do you have? Can you shell into your install and look at the version inside the manifest.json of this component as well as the requirements.txt?

Also, I appreciate your patience and working with me here. If you have any interest in polishing this component with me, I’ll be happy for the help.

I can also provide some guidance if you don’t have much experience with coding or extensions, etc. I don’t know your background

nerdtech · May 8, 2024, 10:12am

Have been working in linux systems for over 10 years so thats no problem. But my coding skills is limted to easy cut and paste!
Manifest.json says version 0.1.14-dev of the client and under ssdp 0.1.2
And requirements.txt is missing i dont see any file with that name.

Btw. is toolbelt maybe used in the client? Could that be the problem?

kbirger · May 9, 2024, 5:28am

it looks like it’s fixed in the latest version of pyrebase4, or at least there are some related commits.

Not sure why it doesn’t happen for others.

If you Google that error, there’s a stack overflow page that recommends temporarily downgrading that package.

Otherwise, you’ll have to wait until I have time to push an update. Sorry about that

kbirger · May 9, 2024, 10:47am

New build up. I did it mostly from my phone so it’s not tested.

nerdtech · May 9, 2024, 10:02pm

Impressive that you did that from your phone! But no it gives the same error. Looks like it wont download the client either it goes to the error right away when trying to add the device…

kbirger · May 12, 2024, 12:25pm

I spun up a dev container with the latest home assistant. None of the core integrations are causing requests_toolbelt (any version) to be installed. So I’m wondering if you have another custom integration that is causing this package to be updated.

The error you are reporting is described here: python - Pyrebase4 error cannot import name 'gaecontrib' - Stack Overflow

You can try to use pipdeptree -p requests_toolbelt to understand what is requiring the problematic version to be installed on your system, or you can just downgrade it, since versions > 1.0 are explicitly unsupported by pyrebase4.

github.com

nhorvath/Pyrebase4/blob/b87a06f74c209635f7469ed18166e262d1bc434f/requirements.txt#L1


      
          requests-toolbelt>=0.7.1,<1.0
          requests>=2.31
          urllib3>=1.21.1,<2
          gcloud>=0.18.3
          oauth2client>=4.1.2
          python-jwt>=2.0.1
          pycryptodome>=3.6.4

I’m guessing it’s urllib3 version 2.0. You can check your version by doing pip show or using pipdeptree.
Context: Remove Google AppEngine support by pquentin · Pull Request #355 · requests/toolbelt · GitHub

If you definitely need this version at this time…
It’s possible that newer versions will come into support. Perhaps you can log an issue to that package’s github issue tracker. It seems like AppEngine support is being removed from urllib3, and probably likely reasonable to remove it from pyrebase.

I did mention that I had another package that works similarly to pyrebase4, but I have errors running that one. Comparing the usage stats for both, pyrebase4 seems to be much more broadly used, so I’m not sure it makes sense to switch, because it might result in an overall worse experience for all users (including you).

nerdtech · May 13, 2024, 1:27pm

This was what i got from pipdeptree… Seams that im using 0.10.1 version of request-toolbelt. So that good? Or isnt it right?

bild

kbirger · May 19, 2024, 7:13pm

Odd. It looks like those versions match the versions in the untouched home assistant core container, but the integration works. Your HA OS is containerized, right? Or are you running home assistant directly on the host OS?

nerdtech · May 20, 2024, 9:35am

Exaktly its conatinerized. So its always untouched besides what i put in config.

kbirger · May 20, 2024, 9:44am

What other custom integrations do you have installed, and what version of the container?

kbirger · June 22, 2024, 4:12am

I’ve created a separate thread for my component here: Custom Component: Smarter Kettle and Coffee