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

Snow surface temperature parameterization #878

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kmdeck
Copy link
Member

@kmdeck kmdeck commented Oct 22, 2024

Purpose

Adds the parameterization that @sarahhzhangg developed for snow surface temperature

To-do

This currently gives some poor results - is this due to snow depth being incorrect?
Make this an optional parameterization using Julia structs
Rebase after Andy's snow depth model is merged into main

Address other questions:
Using snow_thermal_conductivity instead of kappa_ice seems to overall worsen the model performance (based on RMSE, although it improved cdp temperature RMSE a bit)
Might be better to use kappa_ice in the rebased version?
The stability issue seems to be gone (runs for snb)
It seems like your rebased version inputs FT(0) instead of h_sfc (line 331)
Does it matter what is used as the initial guess (Tair vs Tbulk) in the root solver?
Is the removal of runoff energy from the fluxes (Fh) intentional?

Content


  • I have read and checked the items on the review checklist.

@kmdeck kmdeck changed the title Temp branch Snow surface temperature parameterization Nov 12, 2024
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.

2 participants