Skip to content

add perturbed RH run #1314

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

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

add perturbed RH run #1314

wants to merge 4 commits into from

Conversation

kmdeck
Copy link
Member

@kmdeck kmdeck commented Aug 12, 2025

Purpose

Adds perturbed RH forcing

Moves the perturbed run to a separate buildkite run
https://buildkite.com/clima/climaland-long-runs/builds/4137

To-do

Content


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

@kmdeck kmdeck requested a review from juliasloan25 August 12, 2025 23:31
Copy link
Member

@juliasloan25 juliasloan25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

A Simple Conversion and Applications".
Bulletin of the American Meteorological Society. 86 (2): 225–234.
"""
function perturbed_specific_humidity_from_dewpoint_and_rh(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
function perturbed_specific_humidity_from_dewpoint_and_rh(
function perturbed_specific_humidity(

I think this is more concise and still readable, but I'm ok either way!

@@ -1082,6 +1083,46 @@ function perturbed_specific_humidity_from_dewpoint(
return q
end

"""
perturbed_specific_humidity_from_dewpoint_and_rh(dewpoint_temperature, temperature, air_pressure, earth_param_set, Δrh)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should move the perturbed forcing functions into a different file, e.g. src/shared_utilities/drivers_perturbed.jl to avoid crowding this file. Since the perturbed ones are only used for experiments

@@ -95,6 +83,36 @@ steps:
env:
CLIMACOMMS_DEVICE: "CUDA"

- group: "Perturbed Low-res Global Land Models"
if: build.env("PERTURBED_RUN") != null
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how does this var get set?

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