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

more doxygen for sorc/nst_ft_chg.fd #334

Closed

Conversation

edwardhartnett
Copy link
Collaborator

Part of #191

@GeorgeGayno-NOAA another PR with doxygen.

Copy link
Contributor

@yangfanglin yangfanglin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. These are changes made for documentation

Copy link
Collaborator

@GeorgeGayno-NOAA GeorgeGayno-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got two warnings when building. This code no longer works. Should we bother to update it or just remove it? See #90.

@yangfanglin
Copy link
Contributor

Ed, please add Xu.li@noaa.gov as a reviewer, and ask him if this program can be removed. I recall it was used to smooth NSST foundation temperature (tref) as an option to run CHGRES, and/or replace tref with an external dataset. George's new CHGRES_CUBE probably does not need or support this anymore. Not sure if any DA step is still using this program.

@edwardhartnett
Copy link
Collaborator Author

@XuLi-NOAA can you tell us if this program is still needed?

@XuLi-NOAA
Copy link
Contributor

@XuLi-NOAA can you tell us if this program is still needed?

This utility has been extended to do more updates of the tref in the sfc file, it is available for wcoss right now.
However, rarely, it will not be needed, so it can be removed, unless we really want to keep it for possible need in the future.

@edwardhartnett edwardhartnett marked this pull request as draft February 25, 2021 15:50
@edwardhartnett
Copy link
Collaborator Author

@XuLi-NOAA I'm not quite clear on whether we can remove this or not. ;-)

@yangfanglin should we remove this, moving forward? (Of course it is still in the public-v2 release and still in the repo, we're not really removing anything.)

We are doing a test + unit testing campaign on UFS_UTILS. So if we don't need it, that would be great. If we do need it, then we should document and test it.

@yangfanglin
Copy link
Contributor

yangfanglin commented Feb 25, 2021 via email

@GeorgeGayno-NOAA
Copy link
Collaborator

He opened this issue to update the program. #90 The current version does not work with the current GFS.

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

Successfully merging this pull request may close these issues.

4 participants