earthobservations / wetterdienst

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

Question about removing upper dependency bounds #1346

Open amotl opened 2 months ago

amotl commented 2 months ago

Observation

That other commit 57d7904e48 removed the upper bounds of all dependency packages.

Thoughts

This may cause troubles because we can't control too much which software versions users are using. Was that intended?

gutzbenj commented 2 months ago

Dear @amotl ,

with the commit I tried to get rid of #1344 and #1331 . During the bump of numpy and pandas I ran into the issue https://github.com/pandas-dev/pandas/issues/55519 but could only somehow resolve it with the removal of upper bounds. Could you figure out the issue there and try to define sensible upper bounds again?

amotl commented 2 months ago

Ah, thanks. Sorry, we are currently on vacation, not too much able to dive into dependency resolution debugging actions. If that solves a problem for you, good luck. I may be able to look into it when we are back, when applicable.