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

set qt and h in extrap buoyancy to clear values #103

Merged
merged 1 commit into from
Aug 10, 2021

Conversation

yairchn
Copy link
Member

@yairchn yairchn commented Aug 10, 2021

First step to handle the definition of h and qt in extra buoyancy part in the updrafts

Copy link
Member

@charleskawczynski charleskawczynski left a comment

Choose a reason for hiding this comment

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

I agree that this is better than before, but is there not a local fix (at k rather than k-1) that can be applied here?

Also, let's delete the local qt and h initialized above

@charleskawczynski
Copy link
Member

bors r+

bors bot added a commit that referenced this pull request Aug 10, 2021
100: Move entr and detr to separate file  r=charleskawczynski a=yairchn

This PR moves the computation of entrainment and detrainment to a separate file  - while breaking of the non dimensional functions so that they could replaced by NN or any time of ML in the future.

Both this PR and the the recent changes to pressure should have all parameters read of CliMA parameters in the future 

103: set qt and h in extrap buoyancy to clear values r=charleskawczynski a=yairchn

First step to handle the definition of h and qt in extra buoyancy part in the updrafts 

Co-authored-by: yairchn <yairchn@caltech.edu>
@charleskawczynski
Copy link
Member

bors r-

@bors
Copy link
Contributor

bors bot commented Aug 10, 2021

Canceled.

@charleskawczynski charleskawczynski merged commit f810cfd into main Aug 10, 2021
@bors bors bot deleted the yc/fix_buoy_extrap branch August 10, 2021 21:46
yairchn pushed a commit that referenced this pull request Aug 10, 2021
set qt and h in extrap buoyancy to clear values

fix change in qt issue

use the correct qt and qv in extra bouy

update tables

fix tables
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