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

Sigma pr317 #317

Merged
merged 12 commits into from
Jun 12, 2021
Merged

Sigma pr317 #317

merged 12 commits into from
Jun 12, 2021

Conversation

JonathanGregory
Copy link
Contributor

@JonathanGregory JonathanGregory commented Mar 25, 2021

Fixes: #314

See issue #314 for discussion of these changes.

Release checklist

  • [NA] Authors updated in cf-conventions.adoc?
  • [NA] Next version in cf-conventions.adoc up to date? Versioning inspired by SemVer.
  • [Y] history.adoc up to date?
  • [NA] Conformance document up-to-date?

For maintainers

After the merge remember to delete the source branch.
Tags are set at the conclusion of the annual meeting; until then master always is a draft for the next version.

@johnwilkin
Copy link

I think this will work OK. Regardless of how the modeler orders the k index it is a universal convention that sigma, or s, increases upward from -1 at the bottom (or of c_depth) to 0 at the sea surface.

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.

Correction to the definition of "ocean sigma over z coordinate" in Appendix D
3 participants