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

Mismatched spaces error in vertical_diffusion_boundary_layer_tendency! #2191

Closed
simonbyrne opened this issue Oct 4, 2023 · 2 comments · Fixed by #2190
Closed

Mismatched spaces error in vertical_diffusion_boundary_layer_tendency! #2191

simonbyrne opened this issue Oct 4, 2023 · 2 comments · Fixed by #2190

Comments

@simonbyrne
Copy link
Member

Current failure in the moist model
https://buildkite.com/clima/climaatmos-target-gpu-simulations/builds/88#018af76a-8b21-411b-a348-5aa181011b5f/138-220

which is here:

@. ᶠρK_E[colidx] =
ᶠinterp(Y.c.ρ[colidx]) * eddy_diffusivity_coefficient(
C_E,
norm(interior_uₕ[colidx]),
interior_coordinates.z[colidx] - z_surface,
ᶠp[colidx],
)

From what I can tell, the problem is that we have fields at two different levels (one at level 1, one at level half).

For now, I think the easiest option might be to simply get rid of the error. Hopefully we can address this properly as part of CliMA/ClimaCore.jl#1120

@simonbyrne
Copy link
Member Author

I tried removing the check here:
https://buildkite.com/clima/climaatmos-target-gpu-simulations/builds/89

@charleskawczynski
Copy link
Member

This will be fixed by #2190 (successfully passed this line in this build: https://buildkite.com/clima/climaatmos-ci/builds/13566#018afcbf-a78e-49da-b017-18d0c705c4fa).

@bors bors bot closed this as completed in 904aa59 Oct 5, 2023
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 a pull request may close this issue.

2 participants