-
Notifications
You must be signed in to change notification settings - Fork 13
Improve plot output (hatching, unit formatting, mean lines, etc.) #171
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
Merged
andreww
merged 16 commits into
GreenScheduler:main
from
sadielbartholomew:plotting-improvements
Nov 28, 2025
Merged
Changes from all commits
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
e79e862
Plotting: add legened to clarify windows shown
sadielbartholomew 59674dd
Plotting: add hatching to shown windows to distinguish w/o colour
sadielbartholomew 9af5c2c
Plotting: remove gap below 0 CI value for y axis
sadielbartholomew a498d21
Plotting: add missing datetime element to axis label
sadielbartholomew 24100fc
Remove redundant call 'return None' in plotting.plotplan
sadielbartholomew e0dd94b
Plotting: improve plot style esp. colours for accessibility
sadielbartholomew 7b8445f
Remove redundant 'None' from 'return None' in plotting.plotplan
sadielbartholomew 8719718
Plotting: add subtle scatter markers to indicate forecast data points
sadielbartholomew 482b6af
Plotting: remove default linewidth specifications, ajdust transparency
sadielbartholomew 14c5028
Plotting: add overlap region to legend through dummy patch
sadielbartholomew 5965e6f
Improve title plus units formatting w/ Mathtext
sadielbartholomew 2adb694
Include horizontal lines at now & optimal mean values
sadielbartholomew 6163752
Only add overlap patch to legend if there is a calc'd overlap
sadielbartholomew cd094ec
Improve x-axis datetime parsability via tick formtting logic
sadielbartholomew ac71295
Plotting: prevent x-axis label cut off by adjusting figure size
sadielbartholomew 6d3d6bf
Plotting: fix lack of string-literal escaping & tidy plot code
sadielbartholomew File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Super minor comment but I would probably call this something like "readable_datetime_tick_formatter" (and then curse because I failed to find a reasonably short name)
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.
Good point, agreed that the title is too generic and doesn't explain it's purpose very well. If you can note that (or link to your comment here) on the new relevant issue, that would be great. Thanks for the prompt review :)