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

Support depth in meters for mixed_layer_cape_cin #1718

Open
dopplershift opened this issue Feb 18, 2021 · 0 comments · May be fixed by #1912
Open

Support depth in meters for mixed_layer_cape_cin #1718

dopplershift opened this issue Feb 18, 2021 · 0 comments · May be fixed by #1912
Labels
Area: Calc Pertains to calculations Type: Bug Something is not working like it should

Comments

@dopplershift
Copy link
Member

Currently, depth is usable in mixed_layer_cape_cin only if given as pressure. If you try to pass e.g. 500 * units.m, you get a dimensionality error, because it uses that to figure out what levels to strip out of the sounding. Currently, though, the docs say that any argument for mixed_layer should work.

@dopplershift dopplershift added Type: Bug Something is not working like it should Area: Calc Pertains to calculations labels Feb 18, 2021
@dopplershift dopplershift added this to the 1.1.0 milestone Feb 18, 2021
@23ccozad 23ccozad linked a pull request Jun 15, 2021 that will close this issue
3 tasks
@dopplershift dopplershift modified the milestones: 1.1.0, 1.2.0 Aug 2, 2021
@dopplershift dopplershift modified the milestones: 1.2.0, 1.3.0 Oct 8, 2021
@dopplershift dopplershift modified the milestones: 1.3.0, May 2022 Mar 31, 2022
@dopplershift dopplershift modified the milestones: May 2022, July 2022 Jun 1, 2022
@dopplershift dopplershift removed this from the September 2022 milestone Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Calc Pertains to calculations Type: Bug Something is not working like it should
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant