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

added option for dry run #322

Merged
merged 6 commits into from
Jul 20, 2016
Merged

added option for dry run #322

merged 6 commits into from
Jul 20, 2016

Conversation

doutriaux1
Copy link
Contributor

which means no computations (just read in data)
fix #320

which means no computations (just read in data)
fix #320
@doutriaux1
Copy link
Contributor Author

@gleckler1 should be what you want.

@doutriaux1
Copy link
Contributor Author

@gleckler1 is that what you wanted?

@gleckler1
Copy link
Contributor

@doutriaux1 @durack1 its ok for now, so I'm ok to add it to master. Later we'll want to improve on it as it prints out stuff that does not relate to errors (e.g., "obs shape is") and at some point Paul can make it more of a log file.

@durack1
Copy link
Collaborator

durack1 commented Jun 1, 2016

@doutriaux1 is this good to go - tests are failing, does this also need master merged in?

@doutriaux1
Copy link
Contributor Author

yp. I'm on it

@durack1
Copy link
Collaborator

durack1 commented Jul 19, 2016

@doutriaux1 is this dead - or does it need to be rebased?

@gleckler1
Copy link
Contributor

@doutriaux1 maybe it has been replaced by the recently added '-t' option ?

@doutriaux1
Copy link
Contributor Author

@durack1 let me update the branch.

@doutriaux1
Copy link
Contributor Author

@durack1 at your convenience

@durack1
Copy link
Collaborator

durack1 commented Jul 19, 2016

Geez @doutriaux1 even pep8'd.. We might have to create that Robust label after all.. I'll wait for a green travis before merging, all looks good to me

@durack1 durack1 merged commit c7ae783 into master Jul 20, 2016
@durack1 durack1 deleted the issue_320_dryrun branch July 20, 2016 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

external check of parameter table model and obs data
3 participants