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

Developed Peters et al. 2022 lifter #17

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

Conversation

a-urq
Copy link

@a-urq a-urq commented Jun 25, 2024

The Peters et al. lifter has been tested with all four types of moist ascent using a modified version of my sounding viewer as a Skew-T plotter, which has itself been tested against the ECAPE_FUNCTIONS python script written by John Peters. The SHARPlib-computed parcel is represented by a translucent green line.

I also found a few issues related to ECAPE computation in thermo.cpp's sharp::entrainment_cape and fixed them, most of which were related to differences between the preprint and the final published paper.

Pseudoadiabatic non-entraining:
pseudoadiabatic nonentraining

Pseudoadiabatic entraining:
pseudoadiabatic entraining

Irreversible adiabatic non-entraining:
irrev-adiabatic nonentraining

Irreversible adiabatic entraining:
irrev-adiabatic entraining

a-urq added 23 commits April 23, 2024 17:51
…ngements to parcel.h that Kelton may or may not be okay with.
… include issues in a much more elegant way than reshuffling <parcel.h> the way I did earlier
…o develop-peters-lifter

I forgot that I deleted .vscode from the remote directory lol
…o develop-peters-lifter

Remote repository had extraneous files deleted while local repository did not.
@keltonhalbert
Copy link
Owner

keltonhalbert commented Jul 4, 2024

Thanks for making this PR @a-urq ! Sorry I didn't see this or get to it sooner. I'm on operational training shifts this summer, but I'll start taking a look at this on my non-ops shifts and start reviewing/testing the code and let you know if I have any further questions. I'm definitely interested in getting this merged into the main branch, as having the entraining parcel traces would be very useful and is something I haven't had the time to work on myself!

@keltonhalbert keltonhalbert self-assigned this Jul 4, 2024
@keltonhalbert keltonhalbert added the enhancement New feature or request label Jul 4, 2024
@keltonhalbert keltonhalbert added this to the Conda Forge Release milestone Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants