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

Add support for NCAR hobart. #153

Merged
merged 1 commit into from
Jun 27, 2018
Merged

Add support for NCAR hobart. #153

merged 1 commit into from
Jun 27, 2018

Conversation

dabail10
Copy link
Contributor

[Remove this and add a short summary line]:

  • Developer(s): D. Bailey

  • Please suggest code Pull Request reviewers in the column at right.

  • 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 at a later time? (Y/N)
Note: "Documentation" includes information on the wiki and .rst files in doc/source/,
which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-cice/.

N

  • Other Relevant Details:

This is simply just NCAR hobart support. I did have to change three calls to "flush" to "icepack_warnings_flush" as NAG was not happy with these. I also re-ordered the default namelist so that NAG was happy. I looked into rewind, but this was not working. I will work on the unused dummy variables request next.

Dave

@eclare108213
Copy link
Contributor

I don't understand the branch name, NCAR:master. Is NCAR the user? Why not dabail10? I don't think this matters, I'm just curious how you've got things set up. I'll merge it.

@eclare108213 eclare108213 merged commit 360c474 into CICE-Consortium:master Jun 27, 2018
@dabail10
Copy link
Contributor Author

Sorry about the confusion with the fork. I wanted to start a fresh fork and I mistakenly put it on the NCAR domain instead of on my domain. I'll fix that for the next one.

@eclare108213 eclare108213 mentioned this pull request Aug 8, 2018
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.

3 participants