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

QC Script Debug #108

Merged
merged 2 commits into from
Mar 28, 2018
Merged

Conversation

mattdturner
Copy link
Contributor

Update the QC script to properly handle the case where the effective sample size is greater than 30.

Developer(s): Matt Turner
Are the code changes bit for bit, different at roundoff level, or more substantial? bfb
Is the documentation being updated with this PR? (Y/N) N
If not, does the documentation need to be updated separately? (Y/N) N
"Documentation" includes information on the wiki and .rst files in doc/source/,
which are used to create the online technical docs at https://cice-consortium.github.io/CICE/
Please suggest code reviewers in the column at right.
Other Relevant Details:

  • I also commented out the call to the post_to_cdash function since we are no longer using cdash.

@eclare108213
Copy link
Contributor

Thanks, Matt!

@eclare108213 eclare108213 merged commit 89aebde into CICE-Consortium:master Mar 28, 2018
@mattdturner mattdturner deleted the qc_script_debug branch May 29, 2018 17:26
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.

2 participants