search

earthobservations / wetterdienst

Open weather data for humans.
https://wetterdienst.readthedocs.io/
MIT License
342 stars 52 forks source link

README fixes #1275

Closed 1Maxnet1 closed 2 months ago

1Maxnet1 commented 3 months ago

Remove mention of http extra from README: When going through the docs, I stumbled upon a little issue, that the http extra is not available anymore. So I think it makes sense to remove it from the README.rst, so others do not stumble upon the same issue again.

Fix code example: The given example does give a list of strings as parameters, although the code expects a string.

gutzbenj commented 2 months ago

Daer @1Maxnet1 ,

thanks for looking at those details! Did we miss listing other extras in the README.rst?

1Maxnet1 commented 2 months ago

Daer @1Maxnet1 ,

thanks for looking at those details! Did we miss listing other extras in the README.rst?

Yes. As far as I can tell bufr, eccodes, excel, explorer, import, mpl, radar, radarplus and restapi are not described in the README.rst

gutzbenj commented 2 months ago

Would you be so kind and add them to the README.rst? Maybe also putting a second extra in the pip install wetterdienst[restapi,sql].