Add ISMIP7 postprocessing framework#733
Draft
hollyhan wants to merge 33 commits into
Draft
Conversation
This is an exact copy of ISMIP6 postprocessing as a starting point
This change follows updates to MALI in which flux outputs are averaged online over each output interval. Also update flux-variable metadata to match the ISMIP7 variable request (https://docs.google.com/spreadsheets/d/1yWDxk8pRwwp3gvRT6j0lsWA_5hAkAXqZdLGtgpnLSNE/edit?gid=2113789754#gid=2113789754) and stop calling the legacy flux cleaning/time-averaging functions from the main script: post_process_mali_to_ismip7.py
MALI now provides interval-averaged flux outputs directly, so these legacy ISMIP6 preprocessing steps are no longer needed.
Add the grounded basal mass balance scalar output required by ISMIP7, update the ice-front melting scalar to use face melting only, and remove the legacy total basal mass balance scalar output.
* eliminate mapping file name and mali grid name and replace with pre-generated map file name * add option to reuse existing map file
* Renames create_mapfile_mali_to_ismip7.py to grid_and_mapping.py * moves grid check stuff from main script to the new module
* if grid is invalid, err instead of generating grid * rename function to "check"
This eliminates need to cat files ahead of time.
* allow authors and group to be input * created single metadata dict for all stages to use
This makes it easy to set for DOE and Arete submissions
Move multifile validation to a helper module and call from both 1d and 2d state processing.
Fixes a number of small problems and general cleanup
I set up masks that could be used for flux vars, but then I disabled using them altogether. This is because the masks at the end of the year will not necessarily be consistent with where fluxes were applied during the year. It makes more sense to me to leave fluxes unmasked and just have 0 values for the flux at places where they did not occur.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Last updated: 2026/06/24
This PR updates the MALI postprocessing scripts for ISMIP7 based on the ISMIP6 postprocessing scripts.
Completed
tendlibmassbfgrtendlifmassbfto represent ice-front melting onlyRemaining work
litempavgin the ISMIP7 variable spreadsheet https://docs.google.com/spreadsheets/d/1yWDxk8pRwwp3gvRT6j0lsWA_5hAkAXqZdLGtgpnLSNE/edit?gid=2113789754#gid=2113789754/global/cfs/cdirs/fanssie/MALI_projects/ISMIP7/june30_runs/AISon Perlmutter.tendlifmassbf,tendligroundfin the ISMIP7 variable spreadsheet.