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

external check of parameter table model and obs data #320

Closed
gleckler1 opened this issue Apr 6, 2016 · 3 comments · Fixed by #322
Closed

external check of parameter table model and obs data #320

gleckler1 opened this issue Apr 6, 2016 · 3 comments · Fixed by #322

Comments

@gleckler1
Copy link
Contributor

@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.

@gleckler1
Copy link
Contributor Author

@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...

@durack1
Copy link
Collaborator

durack1 commented May 25, 2016

@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?

@gleckler1
Copy link
Contributor Author

@durack1 @durack1 noon lunch in courtyard?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants