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

mushy cice physics #402

Closed
rgrumbine opened this issue Jan 30, 2021 · 4 comments
Closed

mushy cice physics #402

rgrumbine opened this issue Jan 30, 2021 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@rgrumbine
Copy link

From Avichal email to Bob:

Other than numerical stability, how does this [mushy physics] impact our results in the Polar regions? What influence does it have on Sea Ice-Atmosphere fluxes, Sea Ice-Ocean fluxes? We are far away from any code freeze, so all these questions have merit for P6 and subsequent prototypes. Are there relevant pubs which can provide some clues? How should we approach a decision on its inclusion?

Description

Provide a clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Solution

Add a clear and concise description of what you want to happen.

Alternatives

If applicable, add a description of any alternative solutions or features you've considered.

Related to

Directly reference any issues or PRs in this or other repositories that this is related to, and describe how they are related. Example:

  • required to support noaa-emc/nems/issues/<issue_number>
  • needed by noaa-emc/fv3atm/pull/<pr_number>
@rgrumbine rgrumbine added the enhancement New feature or request label Jan 30, 2021
@JoshuaFu-NOAA
Copy link

hi, Bob: Just to let you know that I run into the well-known ice model 'thermo iteration error' in P5 with initial condition on 20140101. This error exists even reducing the time step from 450s to 300s and increasing iteration step from 100 to 200 as suggested by some one from ice model group at NCAR. However, the 'thermo iteration error' was resolved by changing the 'ktherm' from 1 (no mushy physics) to 2 (with mushy physics). It is still a mystery why the 'mushy physics' helps resolve the 'iteration error' issue.

@rgrumbine
Copy link
Author

Thanks for the word Joshua. That sounds consistent with the DA group's experience. Good to have an independent confirmation.

@junwang-noaa
Copy link
Collaborator

@rgrumbine @DeniseWorthen is this issue fixed in PR#610?

@DeniseWorthen
Copy link
Collaborator

The script changes required to set mushy optionally were included in PR #623

The original issue created apparently was for an evaluation of the impact of using mushy, so I think that will only be resolved after the P7 is run.

epic-cicd-jenkins pushed a commit that referenced this issue Apr 17, 2023
## DESCRIPTION OF CHANGES: 
This change will add the capability to run regional_workflow (as part of the SRW app) on MacOS and generic LINUX platforms. Most of these changes are identical to those in #402 (hash e6b8c74) but some additional modifications needed to be made due to intervening changes in the develop branch.

## TESTS CONDUCTED: 
Ran Graduate Student Test on new platforms:
 - my personal Mac machine (MacOS Catalina 10.15.7) MacOS with gnu 9.4.0 compilers. 
 - Cheyenne compute node as a faux "stand-alone" machine, intel 19.1.1 compilers

Ran suite of end-to-end tests on Cheyenne (intel/19.1.1) and Hera (intel/18.0.5.274). All passed as expected.

Tests also passed on WCOSS, MacOS Mojave, RedHat Linux.

## ISSUE: 
Will resolve #369
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

No branches or pull requests

4 participants