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

Enable option to skip vertical interpolation in QGField if variables are already on uniform pseudoheight grid #115

Closed
Tracked by #116
csyhuang opened this issue Jun 15, 2024 · 1 comment

Comments

@csyhuang
Copy link
Owner

csyhuang commented Jun 15, 2024

def vertical_interpolation_to_pressure(ds, variable, save_topography=False, logging_object=None, \

Optimize the interpolation procedures and refactor it into utility module of falwa.

[Update] After some investigation, the time needed seems inevitable unfortunately. Perhaps we can reduce the cost a bit by allowing QGField to take in evenly-spaced pseudoheight grid without doing interpolation. But this would not save much time, since the time sink is mostly at the hybrid coordinate conversion.

TODO: Move the vertical interpolation from f2py to scipy. Let f2py code do only QGPV and static stability computation.

@csyhuang csyhuang changed the title Optimize hybrid to pressure coordinate interpolation Enable option to skip vertical interpolation in QGField if variables are already on uniform pseudoheight grid Jun 15, 2024
@csyhuang
Copy link
Owner Author

This is done with commit a0d9252 😄 I will close this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant