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

Update "What's New in CTSM 5.3" #2789

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

Conversation

samsrabin
Copy link
Collaborator

@samsrabin samsrabin commented Sep 25, 2024

Description of changes

  • Rearranging a bit to highlight things that users probably care about more than software engineers.
  • Removing line breaks, as this harms searchability.
  • Using hash marks as Markdown headers instead of title underlines, as these are easier to understand.

It might be easier for further discussion to take place in a Google Doc for now... I've made one here.

Specific notes

Contributors other than yourself, if any: @ekluzek

CTSM Issues Fixed: None

Are answers expected to change (and if so in what way)? No

Any User Interface Changes (namelist or namelist defaults changes)? No

Does this create a need to change or add documentation? Did you do so? No

Testing performed, if any: None needed.

Remaining questions/tasks

  • Add links to PRs/issue throughout, so that users can get more information.
  • Move to docs directory?

@samsrabin samsrabin added documentation additions or edits to user-facing documentation bfb bit-for-bit labels Sep 25, 2024
- Snow thermal conductivity is now Sturm_1997
- New IC file for f09 1850
- Snow thermal conductivity method is now `Sturm_1997`
- New initial conditions file for f09 ("1-degree") 1850, with more in progress
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

“with more in progress”—yes?

- Surface datasets now provided for the `ne0np4.POLARCAP.ne30x4` grid.
- Surface datasets now have their version number embedded to prevent mismatch of surface dataset and CTSM version.
- Remove the `--hires_pft` option from `mksurfdata_esmf` as we don't have the datasets for it.
- Remove `VIC` fields from surface datasets.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

What are (were) VIC fields?

- Updates to BGC fire method

Changes for all physics versions:

- Parameter files updated
- FATES parameter file updated
- Glacier region 1 is now undefined
Copy link
Collaborator Author

@samsrabin samsrabin Sep 25, 2024

Choose a reason for hiding this comment

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

"Glacier region 1 is now undefined": What does this mean in terms of answer changes? Why should users care?


- With new surface datasets the following GLC fields have region "1" set to UNSET:
glacier_region_behavior, glacier_region_melt_behavior, glacier_region_ice_runoff_behavior
- With new surface datasets the following GLC fields have region "1" set to UNSET: glacier_region_behavior, glacier_region_melt_behavior, glacier_region_ice_runoff_behavior
Copy link
Collaborator Author

@samsrabin samsrabin Sep 25, 2024

Choose a reason for hiding this comment

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

"With new surface datasets the following GLC fields...": What does this mean in terms of answer changes? Why should users care?

- Fix an important bug on soil fields that was there since `ctsm5.2.0`. This has the side effect of `mksurfdata_esmf` now giving identical answers with a change in number of processors, as it should.
- Surface datasets now provided for the `ne0np4.POLARCAP.ne30x4` grid.
- Surface datasets now have their version number embedded to prevent mismatch of surface dataset and CTSM version.
- Remove the `--hires_pft` option from `mksurfdata_esmf` as we don't have the datasets for it.
Copy link
Collaborator Author

@samsrabin samsrabin Sep 25, 2024

Choose a reason for hiding this comment

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

Is --hires_pft removal maybe not worth mentioning? If the datasets aren't available, was anyone using it? Moreover, this tool change doesn’t fit with the substantive changes to surface datasets described in the rest of the section.

- Pass active glacier (CISM) runoff directly to river model (MOSART)
- Add the option for using matrix for Carbon/Nitrogen BGC spinup
- Add the option for using Matrix CN method for Carbon/Nitrogen BGC spinup
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Since CN Matrix is an option, does it really fit in the “answer changes” section?


- With new surface datasets the following GLC fields have region "1" set to UNSET:
glacier_region_behavior, glacier_region_melt_behavior, glacier_region_ice_runoff_behavior
- With new surface datasets the following GLC fields have region "1" set to UNSET: glacier_region_behavior, glacier_region_melt_behavior, glacier_region_ice_runoff_behavior
- Updates to allow creating transient landuse timeseries files going back to 1700.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

So mksurfdata_esmf can now make land-use timeseries files back to 1700. Have any of those actually been made?


New surface datasets:
=====================
## New surface datasets
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Should this section title also mention land-use timeseries files?


Changes to CTSM Answers:
========================
## Changes to CTSM answers
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This section should mention new surface datasets and land-use timeseries files. Are those only used in clm6_0 physics or all physics versions?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bfb bit-for-bit documentation additions or edits to user-facing documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant