Skip to content

Add ISSM fields to HISTORY and initialization arguments#161

Draft
agstub wants to merge 3 commits intodevelopfrom
feature/agstub/issm-history
Draft

Add ISSM fields to HISTORY and initialization arguments#161
agstub wants to merge 3 commits intodevelopfrom
feature/agstub/issm-history

Conversation

@agstub
Copy link

@agstub agstub commented Feb 11, 2026

The purpose of this Draft PR is to add two new variables from ISSM (Ice-Sheet and Sea-Level System Model) to HISTORY. The variables are ICESMB (ice surface mass balance: exported from LandIce GridComp) and ICEEL (ice surface elevation: exported from ISSM GridComp).

  • Development by @weiyuan-jiang for outputting these variables with HISTORY on a tile space that corresponds to the (unstructured, triangular) elements of ISSM's mesh. Feature/jiang/issm gridcomp GEOSgcm_GridComp#1209
  • added new collection tavg24_1d_issm_Nt (Tile-space,Daily,Time-Averaged,Single-level,ISSM Diagnostics) for mesh tile space: 81842f0
  • The PR also adds configuration parameters to GEOSldas_LDAS.rc that are required by ISSM for initialization: ISSM_EXPDIR and ISSM_EXPNAME. ISSM_EXPDIR points to the directory containing the ISSM input file, which will be replaced by a restart directory in the future. ISSM_EXPNAME is the name of the binary input file (ISSM_EXPNAME.bin) and PETSc options file (ISSM_EXPNAME.toolkits).

Related issues and PRs

TO DO:

  • Add default ISSM outputs ICETHICK (ice thickness) and ICEVEL (ice-flow speed). This will require changes to ISSM branch, GEOSgcm_GridComp branch, module update (env), and history update in this branch.

@github-actions
Copy link

This PR is being prevented from merging because you have not added one of our required labels: 0 diff, 0 diff trivial, Non 0-diff, 0 diff structural, 0-diff trivial, Not 0-diff, 0-diff, automatic, 0-diff uncoupled, github_actions. Please add one so that the PR can be merged.

@weiyuan-jiang
Copy link
Contributor

The variables in the same collection should have the same tile or gridded space. We have discussion that ISSM should have output in mesh-like space. So ICEEL should not be in this collection

@agstub
Copy link
Author

agstub commented Feb 12, 2026

The variables in the same collection should have the same tile or gridded space. We have discussion that ISSM should have output in mesh-like space. So ICEEL should not be in this collection

That makes sense! Should I make new collections for ISSM outputs (mesh tile collection and gridded collection)? Will we still want to save a gridded version of ice elevation @lcandre2 ?

@lcandre2
Copy link
Contributor

The variables in the same collection should have the same tile or gridded space. We have discussion that ISSM should have output in mesh-like space. So ICEEL should not be in this collection

That makes sense! Should I make new collections for ISSM outputs (mesh tile collection and gridded collection)? Will we still want to save a gridded version of ice elevation @lcandre2 ?

I would say that for standard ISSM outputs (elevation, ice velocity, ice discharge, perhaps a few others) it would be good to have both a tile and gridded collection.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants