Rudd-O / homeassistant-meteoswiss

:sun_behind_rain_cloud: :switzerland: Meteo Swiss Integration for Home Assistant (forked from websylv)
74 stars 8 forks source link

hamsclientfork problem #8

Closed chruetli closed 1 year ago

chruetli commented 1 year ago

Hi

The latest homeassistant update produces the following error:

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [3 lines of output]
      error in hamsclientfork setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected end or semicolon (after version specifier)
          beautifulsoup4>=4.8.2geopy>=2.0.0
                        ~~~~~~~^
      [end of output]

The problem is a missing , in setup.py from hamsclientfork :

install_requires=[
        "requests>=2.22.0",
        "pandas>=0.25.3",
        "beautifulsoup4>=4.8.2" "geopy>=2.0.0",
    ],
Benson17 commented 1 year ago

I can confirm this problem - any quick fix for this? I don't find setup.py, seems it's loaded when setup is running...

dcear commented 1 year ago

Yes me too , i dont' find setup.py...., thank for your Help

chruetli commented 1 year ago

setup.py is found in the installation package of https://github.com/Rudd-O/hamsclientfork. There is probably no quick fix as an error in setup.py avoids running the installation of this package. I think @Rudd-O has to fix the package.

phil-schneider commented 1 year ago

I can confirm. This great fork add on is not working anymore since the HA update. How do I now can forecast the weather? 😉

Rudd-O commented 1 year ago

Oh wow. Let me fix that right away.

Rudd-O commented 1 year ago

A fix has been published. hamsclientfork 0.2.7 does not have this problem anymore. The latest version of this custom integration requires that version of hamsclientfork now.

Rudd-O commented 1 year ago

Thank you for the quick bug report!

chruetli commented 1 year ago

👍 thank you!!

Rudd-O commented 1 year ago

Please do confirm that the update works! I confirm it works here at home but I need independent confirmation.

phil-schneider commented 1 year ago

It's working for me, after reinstall and restart. Thanks a lot.

Benson17 commented 1 year ago

It's also working for me after re-download. Many thanks!

dcear commented 1 year ago

Thank you very much, it works fine

Rudd-O commented 1 year ago

Schönes Wochenende! :100:

razowski commented 1 year ago

Thanks for the fix 👍