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.

Sign up for GitHub

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

Jump to bottom

Option to return full (cumulative) profile from thickness_hydrostatic functions #1312

Open
andreas-h opened this issue Jan 29, 2020 · 3 comments · May be fixed by #3148
Open

Option to return full (cumulative) profile from thickness_hydrostatic functions #1312

andreas-h opened this issue Jan 29, 2020 · 3 comments · May be fixed by #3148
Labels
Area: Calc Pertains to calculations Subarea: Thermo Pertains to thermodynamic calculations and their physical correctness Type: Feature New functionality
Milestone
1.7.0

Comments

@andreas-h
Copy link

It would be useful if one could calculate the thickness of layers of the full atmosphere in one go, i.e., if thickness_hydrostatic_from_relative_humidity() and thickness_hydrostatic() would (optionally) not only return the full thickness (i.e., between bottom and top) but instead all the layer thicknesses individually.

To explain more technically: The current return value (the full thickness) would then be the top layer of the cumsum of the individual thicknesses.
@andreas-h andreas-h added the Type: Feature New functionality label Jan 29, 2020
@dopplershift dopplershift added the Area: Calc Pertains to calculations label Feb 14, 2020
@deeplycloudy deeplycloudy added the Subarea: Thermo Pertains to thermodynamic calculations and their physical correctness label Aug 21, 2020
@jthielen jthielen changed the title Return full profile from thickness_hydrostatic_from_relative_humidity Option to return full (cumulative) profile from thickness_hydrostatic functions Aug 30, 2021
@jthielen
Copy link
Collaborator

This would help clean up some of #2064.

@jthielen jthielen added this to the 1.2.0 milestone Aug 30, 2021
@jthielen jthielen mentioned this issue Aug 30, 2021
Open
9 tasks
@jthielen jthielen modified the milestones: 1.2.0, 1.3.0 Oct 8, 2021
@brianmapes
Copy link
Contributor

+1 in 2022

@dopplershift dopplershift modified the milestones: 1.3.0, May 2022 Mar 31, 2022
@dopplershift dopplershift modified the milestones: May 2022, July 2022 May 16, 2022
@Z-Richard
Copy link
Contributor

I want to give this implementation a shot but I am a bit confused about what the interface should be. IMO, the interface should be a combination of several types of usages of the function:

  • The user can specify a bottom, and the depth defaults to the full profile;
  • The user can specify both a bottom and a depth (1000 - 500 hPa thickness)
  • The user can specify a depth, and bottom defaults to the bottom of the layer
  • The user can specify that they want the thickness of each individual layer. In this case, the bottom and depth are invalid parameters.

It can be quite difficult to specify these three parameters in a single function, i.e., thickness_hydrostatic, as the rich behavior provided by a combination of these parameters can confuse the user. I think it would be better to implement a new function, e.g., thickness_hydrostatic_each_layer, to return the thickness of individual layers, but would also like to know what others think the correct approach should be.

@dopplershift dopplershift modified the milestones: 1.4.1, April 2023 Mar 16, 2023
@Z-Richard Z-Richard linked a pull request Aug 11, 2023 that will close this issue
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area: Calc Pertains to calculations Subarea: Thermo Pertains to thermodynamic calculations and their physical correctness Type: Feature New functionality
Projects
Python
Status: No status
Milestone
1.7.0
Development

Successfully merging a pull request may close this issue.

Update thickness_hydrostatic to have an option to return full profile Z-Richard/MetPy
6 participants
@dopplershift @andreas-h @deeplycloudy @brianmapes @jthielen @Z-Richard