-
Notifications
You must be signed in to change notification settings - Fork 1
Web monitoring figures for GSI brightness temperature errors and replay file counts and increments #7
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
base: develop
Are you sure you want to change the base?
Web monitoring figures for GSI brightness temperature errors and replay file counts and increments #7
Conversation
…(mean, rms) errors, organized by sensor and platform for reanalysis experiment inter-comparisons
…explicit mpi4py send/recv (instead of bcast) calls to distribute filtered data frames to child processes
…to obs batch plotting
…t size to 12 and time axis label adjustments
…e subset of sensors for more frequent (i.e., daily) updates
…ay observer diagnostic overlapping time periods
…ay observer diagnostic overlapping time periods
… sbatch scripts out of the python package and into a new top level directory named batch_scripts
…ard list of experiments for radiance monitoring webpage
|
|
||
| from score_db import score_db_base | ||
|
|
||
| def run(make_plot=False, make_line_plot=True, select_array_metric_types=True, |
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.
Can you please clean up some of the comments in this function and run_line_plot? That will clear up how these functions work a bit.
|
|
||
|
|
||
| def main(): | ||
| #run() |
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.
Does the run() function get called at all? If not, that can be removed.
|
|
||
| self.timestamp_dict = dict() | ||
| self.timelabel_dict = dict() | ||
| #self.timelabel_dict = dict() |
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.
A few comments in this function need to be cleaned up.
| def main(): | ||
| """ | ||
| """ | ||
| #run_avhrr() |
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.
Are these comments used to run different plot types? If so, can you please document that at the top of main, or make it a CLI argument to swap between plot types?
| else: | ||
| title_str1 = "]" | ||
|
|
||
| #fig.suptitle(f"{title_str0}{title_str1}") |
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.
There are a few comments in this function that should be cleaned up.
KevinCounts
left a comment
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.
I've added comments to a few spots where comments should be cleaned up and main functions that need some documentation. I know one of those scripts is legacy. Other than those, I think this is looking great.
Support for GSI stats radiance monitoring figures and minor updates to replay file count and increment plots.
GSIstats figures:
Replay figure updates:
Also includes new matplotlib style sheets for better web appearance
Example figures available at https://psl.noaa.gov/UFS/