Skip to content

Conversation

@szy21
Copy link
Member

@szy21 szy21 commented Dec 4, 2025

Purpose

Including du3_dq and drhoe_tot_dq

To-do

Content


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

@szy21 szy21 force-pushed the zs/1m_jacobian branch 13 times, most recently from a6e3d6a to 3de24cb Compare December 9, 2025 00:18
@szy21 szy21 changed the title add du3_dq jacobian for microphysics tracers add jacobian terms related to dp_dq for microphysics tracers Dec 9, 2025
@szy21 szy21 marked this pull request as ready for review December 9, 2025 00:49
Copy link
Member

@sajjadazimi sajjadazimi left a comment

Choose a reason for hiding this comment

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

LGTM! thank you!

@szy21
Copy link
Member Author

szy21 commented Dec 9, 2025

Ah, we can't solve the matrix if we add the ∂ρe_tot_err_∂ρq terms: https://buildkite.com/clima/climaatmos-ci/builds/27130#019b0133-c255-4264-923b-e8297ec3cfa8/178-245

@szy21 szy21 force-pushed the zs/1m_jacobian branch 4 times, most recently from 1af55e8 to 5e2a84e Compare December 9, 2025 23:54
@szy21 szy21 enabled auto-merge December 9, 2025 23:56
@szy21
Copy link
Member Author

szy21 commented Dec 10, 2025

hmm I don't know why it breaks MSE now. It didn't break it in this build: https://buildkite.com/clima/climaatmos-ci/builds/27171

@szy21
Copy link
Member Author

szy21 commented Dec 10, 2025

Ah, it seems MSE is broken on the main branch too

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.

3 participants