Skip to content

fix: synchronize investment period weightings via set_snapshots#1414

Merged
lkstrp merged 3 commits intoPyPSA:masterfrom
lkstrp:fix-1411
Oct 20, 2025
Merged

fix: synchronize investment period weightings via set_snapshots#1414
lkstrp merged 3 commits intoPyPSA:masterfrom
lkstrp:fix-1411

Conversation

@lkstrp
Copy link
Copy Markdown
Member

@lkstrp lkstrp commented Oct 20, 2025

Closes #1411

Changes proposed in this Pull Request

This was already introduced in v0.33.0 with #1113 and may be worthy of a patch.

Checklist

  • Code changes are sufficiently documented; i.e. new functions contain docstrings and further explanations may be given in docs.
  • Unit tests for new features were added (if applicable).
  • A note for the release notes docs/release-notes.md of the upcoming release is included.
  • I consent to the release of this PR's code under the MIT license.

@lkstrp lkstrp requested a review from FabianHofmann October 20, 2025 07:36
Copy link
Copy Markdown
Contributor

@FabianHofmann FabianHofmann left a comment

Choose a reason for hiding this comment

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

yes! thanks for fixing

@lkstrp lkstrp enabled auto-merge (squash) October 20, 2025 11:31
@lkstrp lkstrp disabled auto-merge October 20, 2025 12:58
@lkstrp lkstrp merged commit 5df400d into PyPSA:master Oct 20, 2025
21 of 23 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.

Multi-period network: export_to_netcdf() loses investment periods after extending snapshots

2 participants