Skip to content

fix: multiindex future warning#501

Merged
coroa merged 2 commits intomasterfrom
fix/multiindex-future-warning
Apr 14, 2026
Merged

fix: multiindex future warning#501
coroa merged 2 commits intomasterfrom
fix/multiindex-future-warning

Conversation

@coroa
Copy link
Copy Markdown
Member

@coroa coroa commented Apr 14, 2026

Changes proposed in this Pull Request

Accounts for the API changes introduced by xarray v2023.08.0 (released August 20, 2023), via PR #7368 — "Expose Coordinates as part of Xarray's public API".

The index argument to convert_and_aggregate can now accept an xr.Coordinates object with a single dimension, and converts the previously used pd.Index or pd.MultiIndex objects correctly.

This contains changes from #500 , because i wanted to test it with PyPSA Eur and was unable to unless those were included. If we merge after it, that should be fine.

Checklist

  • Code changes are sufficiently documented; i.e. new functions contain docstrings and further explanations may be given in doc.
  • A note for the release notes doc/release_notes.rst of the upcoming release is included.
  • I consent to the release of this PR's code under the MIT license.

@coroa coroa force-pushed the fix/multiindex-future-warning branch from b8c474e to 9beb791 Compare April 14, 2026 09:47
@coroa
Copy link
Copy Markdown
Member Author

coroa commented Apr 14, 2026

Rebased ontop of the master branch after #500 was merged.

Copy link
Copy Markdown
Member

@lkstrp lkstrp left a comment

Choose a reason for hiding this comment

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

Assert and import sorting will be changed again by #492

But let's get this in as well and do a release

@coroa coroa merged commit d5582b4 into master Apr 14, 2026
18 of 19 checks passed
@coroa coroa deleted the fix/multiindex-future-warning branch April 14, 2026 19:55
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.

2 participants