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

Sync develop into main #588

Merged
merged 52 commits into from
Mar 1, 2024
Merged

Sync develop into main #588

merged 52 commits into from
Mar 1, 2024

Conversation

sdrabenh
Copy link
Collaborator

@sdrabenh sdrabenh commented Mar 1, 2024

No description provided.

dr0cloud and others added 30 commits September 19, 2023 14:14
This commit introduces a logical variable 'wavewatch'. Said flag
controls the handling of WAVEWATCH namelists and restarts in the
gcm_run.j script.
Get rid of unnecessary linking of files.
sdrabenh and others added 22 commits February 21, 2024 09:20
…ress

Use nccmp, add readers/writers test to regress
Disable Catchment-CN4.5 land model option (LSM_CHOICE=3) for GCM simulations
…adas

add surface forcing  lfo_Nx+-   to HISTORY.rc.tmpl
Testing by @sanAkel and @rtodling found that `IS_FCST` was a resource parameter in both the AGCM Grid Comp and in the DataSea and DataIce Grid Comps that was doing different things. In the DataSea and DataIce Grid Comps, it determined persistence of SST/ICE via `MAPL_ReadForcing`. But in the AGCM, it altered IAU handling (I think?).

Either way, there was no good way to set one without setting the other. So, for separability, we rename all the oceanic `IS_FCST` to be `OGCM_IS_FCST`. We keep the AGCM `IS_FCST` as it currently is named unless there is a desire to refactor it from @sdrabenh or others.
Refactor DataSea/DataIce IS_FCST to OGCM_IS_FCST
@sdrabenh sdrabenh added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Mar 1, 2024
@sdrabenh sdrabenh requested a review from mathomp4 March 1, 2024 19:11
@sdrabenh sdrabenh requested a review from a team as a code owner March 1, 2024 19:11
@sdrabenh sdrabenh merged commit 0772723 into main Mar 1, 2024
10 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 diff The changes in this pull request have verified to be zero-diff with the target branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants