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

change displayed items in plot "Total power" #192

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

stefansc1
Copy link
Collaborator

  • no more individual loads like batteries
  • sum of all battery powers (if any batteries are present)
  • one line for every GC (if more than one GC)

- no more individual loads like batteries
- sum of all battery powers (if any batteries are present)
- one line for every GC (if more than one GC)
stefan.schirmeister added 2 commits October 24, 2023 13:50
- add option save_plots, which takes a string (path). May contain wildcard "%gc" which gets replaced by grid connector ID
- save_plots creates one plot per GC
- default plotting (visual) aggregates values in Total power subplot
@jakobgemassmer
Copy link
Collaborator

Proposal: We enter the totals of all loads in the plot under "Total Power". In other words, the sum of all CS (already the case), all stationary batteries, all "Local Generation" and all fixed loads. And also "Total" (which is already there).
This would not really change anything for individual SpiceEV simulations (unless you are simulating with several fixed loads or similar). It would even be "nicer" because, for example, the name of the CSV of a "PV time series" would no longer appear in the legend, but only "Local Generation".

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