-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Maintenance/fix warnings #98
Conversation
FYI Python 3.8 is not supported by latest version of pandas https://pypi.org/project/pandas/#files |
@@ -345,10 +345,10 @@ def test_max_aggregation(self, res1d_river_network, column_mode): | |||
df_agg = df_agg.groupby("name").agg("max") # test for entire reach (not segments) | |||
|
|||
assert list(df_agg.columns.values) == [ | |||
"max_WaterLevel", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason to shuffle this around?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new stack algorithm used in future pandas has a different order than before unfortunately
Should we remove Python 3.8 support? It is reaching end-of-life in 2024 Q4, |
I think so, yes. |
I saw that the output of some notebooks is ditched. I personally like it to be there when statically viewing it online. I suggest to revert this change. |
Yes I agree with this. My thinking was mainly to avoid messy merge conflicts with upcoming branches |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
A bit of maintenance with new versions of GeoPandas and Pandas being released