-
Notifications
You must be signed in to change notification settings - Fork 25
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 request: Mass-conserving vertical regridding function #242
Comments
Hi @pennelise and @hannahnesser. I think a vertical regridding capability would be a great addition to GCPy. I think it'd also be nice to have ability to apply the averaging kernels. I guess we have a couple of options here.
Either way is good, but I am probably leaning a bit towards the submodule approach...just for the fact that GCST's core competency is not really in averaging kernel manipulation. (Most of GCST's development in GCPy has been benchmarking related.) If people found bugs in your regridding/avg kernel code then it might be more appropriate for them to open an issue on your existing repository rather than on GCPy. I personally have never worked with averaging kernels so if I got an issue I would be at a loss how to fix it. @msulprizio @lizziel: Thoughts? |
For 2) we were also concerned about maintenance - Hannah has just graduated and I'm in my 5th year, so I am not sure if we can maintain it (@hannahnesser, any thoughts?). I think that the satellite operators will have more bugs than the vertical regridding (for example, when the products are updated). And as long as users can regrid, applying the AK is straightforward. Would it make sense to separate the two functions: a) add the vertical regridding to GCPy and b) keep the satellite operators in a separate repository/submodule? |
I took a quick look at the original repo. Because it's just a few files maybe it could be brought into GCPy directly in a subdirectory. |
@lizziel @sdeastham @pennelise @hannahnesser: Is this functionality now included in https://github.com/pennelise/GOOPy? If so, then perhaps we can close out this issue. |
Discussed in https://github.com/geoschem/gcpy/discussions/241
Originally posted by pennelise July 31, 2023
GCPy doesn't currently have a post-processing vertical regridding function. A mass conserving post-processing vertical regridding function for GEOS-Chem would be useful for:
Hannah Nesser & I have developed python & FORTRAN code for mass-conserving vertical regridding based on the methodology in Keppens et. al, 2019, using code from Hannah's TROPOMI operator and my GOSAT + AIRS operators. Currently we are refactoring it for use as a general satellite observation operator, but we believe this code (or at least the vertical regridding portion) may belong in GCPy. Here is a link to the draft repository.
Here is a basic outline of the functionality:
Our questions:
The text was updated successfully, but these errors were encountered: