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

Remove utils/interpolate_data #307

Open
jimmielin opened this issue Oct 2, 2024 · 0 comments
Open

Remove utils/interpolate_data #307

jimmielin opened this issue Oct 2, 2024 · 0 comments
Labels
code clean-up Made code simpler, better, and/or easier to read.

Comments

@jimmielin
Copy link
Member

Issue Type

Code Clean-up

Issue Description

src/utils/interpolate_data.F90 was brought in entirely from CAM to support reading the tropopause climatology file.

During the review of tropopause_find CCPP-ization, @nusbaume wrote:

It's unclear to me what the long-term state of things like lat/lon interpolation will be in CAM-SIMA (e.g. we might end up using external libraries like ESMF to do all the work for us). Thus I decided to skip reviewing this particular Fortran module.
That being said, I don't want to us to completely forget about it either (it could certainly use some modifications if we do decide to stick with it long-term). So when you get the chance could you create an issue in CAM-SIMA reminding us to either clean-it-up or replace it at some point? Thanks!

This issue reminds us of the eventual clean-up needed for this module or replacement with a ESMF approach.

Will this change answers?

No

Will you be implementing this yourself?

Any Software Engineer can do this

@jimmielin jimmielin added the code clean-up Made code simpler, better, and/or easier to read. label Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code clean-up Made code simpler, better, and/or easier to read.
Projects
Status: To Do
Development

No branches or pull requests

1 participant