Skip to content

Conversation

@ekluzek
Copy link
Collaborator

@ekluzek ekluzek commented Feb 7, 2020

Description of changes

Start bringing in mizuRoute as a submodule and a compset that will exercise it. As well as a few tests for it.

Specific notes

Contributors other than yourself, if any: @serbinsh nmizukami

CTSM Issues Fixed (include github issue #):

Are answers expected to change (and if so in what way)? No (unless running with mizuRoute)

Any User Interface Changes (namelist or namelist defaults changes)? New compsets

Testing performed, if any:
Individual tests with mizuRoute at several different resolutions have been tested

@ekluzek ekluzek added enhancement new capability or improved behavior of existing capability PR status: work in progress labels Feb 7, 2020
@ekluzek ekluzek self-assigned this Feb 7, 2020
@ekluzek
Copy link
Collaborator Author

ekluzek commented Mar 21, 2020

There's at least one change needed in CMEPS. ESCOMP/CMEPS#57

@ekluzek
Copy link
Collaborator Author

ekluzek commented Mar 21, 2020

At this point the test case will build and link, and fails midway in initialization.

SMS_D_Vnuopc.f09_g17.I2000Clm50SpMizGs.cheyenne_intel.mizuroute-default

@ekluzek
Copy link
Collaborator Author

ekluzek commented Apr 18, 2020

We have more progress with running with a MOSART mesh with no ocean. It initializes and runs a single time step. There are changes in: cime, CMEPS, ctsm, and mizuRoute to make that happen.

@ekluzek ekluzek marked this pull request as draft June 4, 2020 22:42
 Conflicts:
	Externals.cfg
	Externals_cime.cfg
…e we are pointing to the cime branch needed to work with mizuRoute
Misc bfb enhancements and fixes

(1) If CISM is running over Antarctica, use virtual glacier columns over
    Antarctica

(2) Remove "mec" from some glacier/ice variable names (it is misleading
    to have "mec" in variable names when the ice landunit can actually
    have multiple columns *or* a single column) (ESCOMP#1294)

(3) Add history file metadata on each variable's l2g_scale_type (adds a
    landunit_mask attribute) (ESCOMP#1343)

(4) Use python3 in more shebang lines - needed to run python unit tests
    on cheyenne

(5) New compset naming for IG compsets (ESCOMP#1289)

(6) Remove calculation of fun_cost_fix that is overwritten
    (ESCOMP#1115)

(7) Bypass grid-level water mass check when fates hydro is active
    (ESCOMP#1334)

(8) Remove some dead code (ESCOMP#1333)

 Conflicts:
	Externals.cfg
	Externals_cime.cfg
Move the NEON_NIWO test to NEON/NIWO to match the names in usermods. Output particular fields for the NEON
cases, and use a reduced size dataset for lightning for NEON sites. Update NEON surface datasets with
new scripts included here. Fix stop option for NUOPC. NEON NIWO test in testlist explicity uses the user-mod
as well as the defaults testmod.

This script is for modifying the surface dataset at neon sites using data available from the neon server.

Also add the ability to output history fields for a particular time of day (in seconds) with the LXXXXX
history averaging option from Ronny Meier.

Externals are updated extensively to the version with cpl7 cime directories split out for CESM.

 Conflicts:
	Externals.cfg
Consolidate duplicated crop phenology code

 Conflicts:
	Externals.cfg
…or mizuRoute that run on more processors for mizuRoute
Implement dynamic (transient) urban capability

This implements the capability to run transient simulations with changes
in urban density types. However, see Caveats for users section below.
If transient urban is activated (do_transient_urban=.true.) and an
appropriate landuse dataset is provided, the model will read in
yearly PCT_URBAN for three density types from the landuse dataset and
adjust other landunits accordingly.
Water and energy are conserved for these transitions. Urban landunits
do not account for carbon and nitrogen, however, see the dynamic urban
design document (doc/design/dynamic_urban.rst) for a discussion of this.

 Conflicts:
	cime_config/config_pes.xml
Update Externals.cfg to work on Derecho

 Conflicts:
	Externals.cfg
…t fails on izumi, because of the updated ESMF version
Delete history and spurious _FillValue from parameter files

 Conflicts:
	Externals.cfg
	cime_config/config_pes.xml
@samsrabin samsrabin added science Enhancement to or bug impacting science and removed enh - new science labels Aug 8, 2024
Merge b4b-dev

Update .gitmodule to have branch for ccs_config and older
verison of mizuRoute that has baselines.

Update ccs_config for Derecho to point to esmf-8.6.1b04 so that
mizuRoute can run with the older files.

 Conflicts:
	Externals.cfg --> changed to .gitmodules
                 (also had a branch for cmeps that came to main)
	bld/namelist_files/namelist_defaults_ctsm.xml
	bld/unit_testers/build-namelist_test.pl
	cime_config/config_pes.xml
@ekluzek ekluzek added this to the CESM3 Answer changing freeze milestone Sep 27, 2024
@ekluzek ekluzek removed the size: large Large project that will take a few weeks or more label Dec 18, 2024
@ekluzek ekluzek moved this from In Progress to Todo in LMWG: Sprint Planning Board Mar 4, 2025
@ekluzek ekluzek moved this from Todo to In Progress in LMWG: Sprint Planning Board Mar 4, 2025
ekluzek added 3 commits March 14, 2025 13:23
Merge b4b-dev 20250313

PR ESCOMP#3013

 Conflicts:
	.gitmodules
	bld/unit_testers/build-namelist_test.pl
	ccs_config
	parse_cime.cs.status
<lname>2000_DATM%NLDAS2_CLM60%SP_SICE_SOCN_MIZUROUTE_SGLC_SWAV</lname>
</compset>

<compset>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'll remove this one. It can still be done with long compset names for a case.

Add a correction for oversaturated soil layers

Add a correction for oversaturated soil layers immediately after the soil water flux calculation in SoilWaterMovementMod.

 Conflicts:
	.gitmodules
	ccs_config
@wwieder
Copy link
Contributor

wwieder commented Jan 22, 2026

Let's get this done! Low lift for a supported feature in the release.

  • @ekluzek will bring in most of this PR into b4b dev (back out the CCS_config part of the PR).
  • @samsrabin requests this PR includes documentation in the users guide and tech note (link to existing resources is OK).
  • Create a remaining issue: get mizuRoute testlist working, which requires ccs_config updates
  • Make sure there's at least one CLM test for mizuRoute

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement new capability or improved behavior of existing capability science Enhancement to or bug impacting science

Projects

Status: Slow roast (incremental or external progress)
Status: Todo

Development

Successfully merging this pull request may close these issues.

4 participants