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

Added single cell ridging test case #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

akturner
Copy link
Collaborator

@akturner akturner commented Dec 1, 2022

Add a single cell test case that tests column ridging scheme. Idealized forcing input is applied to ridgeConvergence and ridgeShear. Only column ridging is active.

@akturner akturner added the enhancement New feature or request label Dec 1, 2022
@darincomeau
Copy link
Contributor

@akturner I've run the single_cell_ridging test case here using E3SM-Seaice-Discussion/E3SM#6. Basically I compiled the seaice_model executable with that E3SM branch, then made the ridging_forcing.nc and itd.nc forcing files with the scripts in testcases/single_cell_ridging/, and ran the case with the namelist and streams in that same directory. All ran successfully.

Let me know if there's anything else you'd like me to do for testing/review, but I think this looks good to me. Thanks for adding this capability!

Copy link
Contributor

@darincomeau darincomeau left a comment

Choose a reason for hiding this comment

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

Approved based on testing the new single_cell_ridging testcase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants