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 plot_mass_balance.py to use new land ice mass budget components #464

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

trhille
Copy link
Collaborator

@trhille trhille commented May 19, 2022

Update plot_mass_balance.py to use new mass budget components introduced in MALI-Dev PR 38. It now uses globalStats.nc to plot global, grounded, and floating time-step-wise mass budgets, cumulative mass budgets, and both relative and absolute error to ensure that budgets close.

@trhille trhille marked this pull request as draft May 19, 2022 20:34
@trhille
Copy link
Collaborator Author

trhille commented May 19, 2022

As of commit 3c4ba20b1f8c3474ef01411a0a59876bc4dcd107, grounded and floating budgets don't yet fully close, but this is an illustration of the output of plot_mass_balance.py:
image

@trhille trhille changed the title Update plot_mass_balance.py to use new mass budget components Update plot_mass_balance.py to use new land ice mass budget components May 19, 2022
Update plot_mass_balance.py to use new globalStats budgets and plot
total, grounded, and floating components, as well as fractional
error between total volume change and sum of budget components.
Previous commit used total calving and facemelting on plot instead of
just grounded components.
Plot absolute errors on another vertical axis on same panel as
fractional errors.
@xylar
Copy link
Collaborator

xylar commented Feb 15, 2024

@trhille, this is pretty old. Is it time to resurrect it or perhaps time to let it go?

@trhille
Copy link
Collaborator Author

trhille commented Feb 15, 2024

@trhille, this is pretty old. Is it time to resurrect it or perhaps time to let it go?

@xylar, this is required for some MALI updates that I'm still planning to complete, so it will move ahead relatively soon.

@xylar
Copy link
Collaborator

xylar commented Feb 16, 2024

Okay, sounds good. Just wanted to make sure this wasn't abandoned work.

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

Successfully merging this pull request may close these issues.

2 participants