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

Port changes to grid_1d package from Compass #147

Merged
merged 5 commits into from
Nov 2, 2023

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented Oct 31, 2023

This merge brings over clean-up and enhancements from MPAS-Dev/compass#581. This includes renaming functions specific to tanh layer thickness for clarity, adding support for the index_tanh_dz vertical grid, and adding missing json files that define E3SMv1 vertical coordinates.

Checklist

  • User's Guide has been updated
  • Documentation has been built locally and changes look as expected
  • Testing comment in the PR documents testing used to verify the changes

@xylar xylar added clean-up ocean Related to ocean tests or analysis labels Oct 31, 2023
@xylar xylar requested a review from cbegeman October 31, 2023 14:12
@xylar xylar self-assigned this Oct 31, 2023
@xylar xylar force-pushed the port-grid-1d-changes branch from 7b5a0aa to c78288d Compare November 1, 2023 09:49
Copy link
Collaborator

@cbegeman cbegeman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. I ran grid types uniform, tanh_dz and index_tanh_dz with single_column/cvmix on chrys with intel, openmpi. The results look reasonable.

@xylar
Copy link
Collaborator Author

xylar commented Nov 2, 2023

@cbegeman, thanks so much for testing! I hadn't had a chance yet.

@xylar xylar merged commit 06cbb87 into E3SM-Project:main Nov 2, 2023
4 checks passed
@xylar xylar deleted the port-grid-1d-changes branch November 2, 2023 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clean-up ocean Related to ocean tests or analysis
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants