-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat: add heatmap plot #245
Conversation
|
There is a _check_time_series function! |
|
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is good to go. Thanks!
Should we move the plt.show() inside the function? |
I think this is a larger conversation about what we want our plot functions to ultimately do: return a plot, or display a plot. I think right now we have a mixture, but it would be nice to standardize. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One more
28f702d
to
f15c6fc
Compare
Purpose
Add this plot, which has been used fairly extensively in the Macro Grid report, to the codebase.
What the code is doing
The code is mostly just passing appropriate parameters to other functions, and orchestrating the creation of a plot figure.
Usage Example/Visuals
Default behavior, using summed curtailment for an Ambitious scenario:
If we flip the colorbar colors:
..and then we want to plot in GW, rather than MW:
...and then we want to explicitly set the range to 250 GW, with updated labels to match:
...and then we want to translate from UTC to CST:
...and then we also want to add a contour for the saturated area:
etc.
Time estimate
15 minutes.