Skip to content

First try Plotting library#1118

Closed
FabianHofmann wants to merge 69 commits intomasterfrom
plotting-library
Closed

First try Plotting library#1118
FabianHofmann wants to merge 69 commits intomasterfrom
plotting-library

Conversation

@FabianHofmann
Copy link
Copy Markdown
Contributor

@FabianHofmann FabianHofmann commented Jan 13, 2025

Closes #912 #1102
addresses #1106

Changes proposed in this Pull Request

Introduce a new plotting accessor scheme, to quickly generate plots based on the statistics module (as described in #1106). The implementation now only works for static plots. Run it with n.plot.map.energy_balance or n.plot.bar.optimal_capacity etc.

Future work

  • think about moving code to components class where possible to avoid redundancy
  • extend for multiinvestment networks (have stochastic networks in mind)
  • extend for interactive plots, via iplot. we can recycle a lot of code here
  • extend for plotting multiple networks at once

Checklist

  • make the old namespace pypsa.plot deprecated in favour of pypsa.plot.maps
  • autogenerate_coordinates was renamed to apply_layouter (keep backwards compat)
  • @lkstrp I kept the namespace order now this way, as it helped me structuring things, I think we can still change if to n.plot.energy_balance.map, once we agreed on the fundamentals.
  • Code changes are sufficiently documented; i.e. new functions contain docstrings and further explanations may be given in doc.
  • Unit tests for new features were added (if applicable).
  • 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.

@FabianHofmann FabianHofmann requested a review from lkstrp January 13, 2025 12:39
@lkstrp lkstrp added this to the v0.34 milestone Jan 15, 2025
@lkstrp lkstrp mentioned this pull request Feb 26, 2025
21 tasks
@lkstrp
Copy link
Copy Markdown
Member

lkstrp commented Mar 6, 2025

Closed via #1156

@lkstrp lkstrp closed this Mar 6, 2025
@FabianHofmann
Copy link
Copy Markdown
Contributor Author

@lkstrp

I have applied the modularization here. This is not a push to merge this instead of #1156 but this should ease the comparison of default conventions

@FabianHofmann FabianHofmann reopened this Mar 10, 2025
@FabianHofmann
Copy link
Copy Markdown
Contributor Author

FabianHofmann commented Mar 23, 2025

ignore

1 similar comment
@FabianHofmann
Copy link
Copy Markdown
Contributor Author

FabianHofmann commented Mar 23, 2025

ignore

@FabianHofmann FabianHofmann reopened this Mar 23, 2025
@FabianHofmann FabianHofmann changed the title Plotting library First try Plotting library Mar 23, 2025
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