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

Correct sign error in y-component of momentum equation in doc #271

Merged
merged 1 commit into from
Jan 17, 2019

Conversation

phil-blain
Copy link
Member

The equation for the y-component of the momentum equation in the doc had a sign wrong in the ice-ocean stress.

  • Developer(s):Philippe Blain

  • Please suggest code Pull Request reviewers in the column at right. @eclare108213 , @duvivier

  • Are the code changes bit for bit, different at roundoff level, or more substantial? BFB

  • Please include the link to test results or paste the summary block from the bottom of the testing output below.
    No tests, only doc changes

  • Does this PR create or have dependencies on Icepack or any other models?
    No

  • Is the documentation being updated with this PR? (Y/N) Yes
    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/.

  • Other Relevant Details:
    It seems this error has been there for a very long time, a PDF that I had from a 2010 version of the doc had this error !

@duvivier duvivier self-requested a review January 17, 2019 16:55
@duvivier duvivier merged commit eef651f into CICE-Consortium:master Jan 17, 2019
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