Skip to content
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

Merged
merged 7 commits into from
Sep 5, 2024
Merged

Maintenance/fix warnings #98

merged 7 commits into from
Sep 5, 2024

Conversation

ryan-kipawa
Copy link
Collaborator

A bit of maintenance with new versions of GeoPandas and Pandas being released

@ecomodeller
Copy link
Member

ecomodeller commented Sep 4, 2024

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",
Copy link
Collaborator

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?

Copy link
Collaborator Author

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

@gedaskir
Copy link
Collaborator

gedaskir commented Sep 4, 2024

FYI Python 3.8 is not supported by latest version of pandas https://pypi.org/project/pandas/#files

Should we remove Python 3.8 support? It is reaching end-of-life in 2024 Q4,

@ecomodeller
Copy link
Member

FYI Python 3.8 is not supported by latest version of pandas https://pypi.org/project/pandas/#files

Should we remove Python 3.8 support? It is reaching end-of-life in 2024 Q4,

I think so, yes.

@gedaskir
Copy link
Collaborator

gedaskir commented Sep 4, 2024

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.

@ryan-kipawa
Copy link
Collaborator Author

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

Copy link
Collaborator

@gedaskir gedaskir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@ryan-kipawa ryan-kipawa merged commit 3800041 into main Sep 5, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants