-
Notifications
You must be signed in to change notification settings - Fork 13
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
[Feature]: Add vertical regridding #45
Comments
I came across this script that we may be able to refer to.. |
@tomvothecoder @chengzhuzhang the xgcm code base (also xarray) could be a useful reference, see https://xgcm.readthedocs.io/en/latest/transform.html |
@durack1 Thank you for the reference. |
Meeting Notes (7/20): Alternative solutions:
Action items:
|
@tomvothecoder Thank you for summarizing and updating this issue. I added CDAT's vertical regridding code as a reference. Just a note here that the GeoCAT solution actually calls the interpolate from metpy package to actually do the vertical remapping. By a brief examination, I think both solutions should meet our requirements in terms of functionality. And perhaps performance and use APIs will be two aspects we should focus on. |
@jasonb5 Hey, Jason. I spent some time to experiment with the existing vertical regridding methods. Please see the jupyter notebook here: https://github.com/xCDAT/xcdat_test/blob/main/validation/v1.0.0/vertical_interp/Vertical_interp.ipynb Notice that only xgcm method (the third one in the notebook) is working. And I have hard time to trouble shooting the first two methods. When you have time, would you please also take a look for me. Let me know if you'd like any clarification on this. Thank you! |
@chengzhuzhang Thanks, I'll give it a look over. |
Hey @jasonb5 , as discussed I added a 4th method, which is to use the cdms2/cdutil approach. The note book now is updated. https://github.com/xCDAT/xcdat_test/blob/main/validation/v1.0.0/vertical_interp/Vertical_interp.ipynb Thank you for looking over! |
Blocked by #71
Describe the solution you'd like
Vertical regridder that can remap hybrid vertical coordinates to standard pressure levels
Acceptance criteria:
Additional context
cdms has reconstructPressureFromHybrid, logLinearInterpolation, and linearInterpolation
The text was updated successfully, but these errors were encountered: