-
Notifications
You must be signed in to change notification settings - Fork 37
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
external check of parameter table model and obs data #320
Comments
which means no computations (just read in data) fix #320
@doutriaux1 @durack1 Last week I was really impressed at how well they were trapping user errors in the ILAMB package. This can make all the difference to users. We should revisit this for our parameter files, having a dry run test case accomplished via a dedicated script that can be continuously modified with new exceptions as we bang on it. So, I propose again this be done in a separate file, not from within the pcmdi_metrics_driver.py... lets chat about this... |
@gleckler1 it would seem that there are many things to chat about - wanna schedule a sit down so we can go over the more comprehensive and targeted issue list and bang things into line regarding the milestones and then also set dates? |
@doutriaux1 @durack1 With a switch available on the parameter table it would be great to be able to do a dry run only on trying to read all model (including land sea masks) and obs data and to report the results in a file that can be browsed. If there is a single script for doing all of these checks CD, PG and PD via iteration should be able to provide some comprehensive error trapping.
The text was updated successfully, but these errors were encountered: