Skip to content

Conversation

@szy21
Copy link
Member

@szy21 szy21 commented Nov 26, 2025

Purpose

Clean up buoyancy gradient calculation and use the actual cloud fraction.

To-do

Content


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

@szy21 szy21 force-pushed the zs/buoyancy_gradient2 branch 6 times, most recently from a832aea to d69f216 Compare December 3, 2025 00:10
@szy21 szy21 force-pushed the zs/buoyancy_gradient2 branch 3 times, most recently from 3b43c2e to eb02af6 Compare December 4, 2025 22:02
@szy21 szy21 marked this pull request as ready for review December 4, 2025 22:04
@szy21 szy21 force-pushed the zs/buoyancy_gradient2 branch 2 times, most recently from f9ce5b3 to 36c9bfe Compare December 4, 2025 23:14
@szy21 szy21 requested review from costachris and tapios December 4, 2025 23:14
@szy21 szy21 force-pushed the zs/buoyancy_gradient2 branch from 36c9bfe to 0423e17 Compare December 4, 2025 23:17
@szy21 szy21 force-pushed the zs/buoyancy_gradient2 branch 2 times, most recently from 94514cf to 5323e2a Compare December 5, 2025 05:22
Copy link
Member

@tapios tapios left a comment

Choose a reason for hiding this comment

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

Looks a lot better than before. Thank you!

@szy21
Copy link
Member Author

szy21 commented Dec 5, 2025

working amip here: https://buildkite.com/clima/climacoupler-coarse-nightly-amip/builds/655 It actually changes the result a bit

@szy21 szy21 force-pushed the zs/buoyancy_gradient2 branch from 5323e2a to d5d62ee Compare December 5, 2025 17:46
@szy21 szy21 force-pushed the zs/buoyancy_gradient2 branch from d5d62ee to beafd42 Compare December 5, 2025 19:20
@szy21 szy21 enabled auto-merge December 5, 2025 19:21
@szy21 szy21 added this pull request to the merge queue Dec 5, 2025
Merged via the queue into main with commit b3696f7 Dec 5, 2025
16 of 17 checks passed
@szy21 szy21 deleted the zs/buoyancy_gradient2 branch December 5, 2025 21:46
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.

4 participants