-
Notifications
You must be signed in to change notification settings - Fork 406
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
height_to_pressure_std has limited validity #1890
Labels
Type: Feature
New functionality
Comments
I'd be happy to see those functions improved. I should note that our functions aren't trying to fit the ICAO standard atmosphere, but instead the 1976 U.S. Standard Atmosphere. I'm not sure how they differ. |
They should be the same, but I'll double check. We come (partially) from aviation so have a different first stop :-) |
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The height_to_pressure_std and pressure_std_to_height routines are valid only up to 11km. Above 11km, according to the ICAO standard atmosphere, the temperature structure changes and thus pressure calculation is increasingly off.
The routine should take the full ISA atmosphere into account (or at least document the valid region).
We have a rather complicated solution in MSS
https://github.com/Open-MSS/MSS/blob/5c47796f98fedbab4d76d8ec283e5432b7574d23/mslib/thermolib.py#L923
in the flightlevel2pressure_a routine (which I'd actually like to replace with something from metpy...).
Due to our gravity wave research, we are interested in the altitude range up to 60km. Beyond it gets even trickier.
I could prepare a merge request based on our code, but I am not really happy with how the conversion is solved for arrays there either...
The text was updated successfully, but these errors were encountered: