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

add compute_surface_fluxes function for component models; test land fluxes #1220

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

Conversation

juliasloan25
Copy link
Member

@juliasloan25 juliasloan25 commented Mar 11, 2025

Purpose

Closes #1242

Extract most of the code out of the partitioned_turbulent_fluxes! function and move it into a function that dispatches on the component model type. For atmosphere models, the function does nothing, and for surface models it performs the flux calculation that was previously in partitioned_turbulent_fluxes!.

This design allows us to dispatch on model type instead of checking the type with an if statement, and, more importantly, it allows us to extend compute_surface_fluxes! for surface models where we want to perform different calculations (e.g. the integrated land model).

Content

Dependency updates

This PR requires ClimaLand v0.15.13, so it updates the dependencies. The changes include:

    Updating `~/clima/ClimaCoupler.jl/experiments/ClimaEarth/Project.toml`
  [d414da3d] ↑ ClimaCore v0.14.28 ⇒ v0.14.29
  [08f4d4ce] ↑ ClimaLand v0.15.12 ⇒ v0.15.13
  [0bca4576] ↑ SciMLBase v2.79.0 ⇒ v2.81.0

@juliasloan25 juliasloan25 force-pushed the js/land_fluxes branch 4 times, most recently from aae3d3c to 592216e Compare March 28, 2025 21:36
@juliasloan25 juliasloan25 changed the title add compute_surface_fluxes function for component models add compute_surface_fluxes function for component models; test land fluxes Mar 31, 2025
@juliasloan25 juliasloan25 force-pushed the js/land_fluxes branch 2 times, most recently from eea1867 to 87414c1 Compare March 31, 2025 23:13
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.

Add model-specific flux calculations
1 participant