-
Notifications
You must be signed in to change notification settings - Fork 4
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
add ctest and nemsio utilities. #11
add ctest and nemsio utilities. #11
Conversation
Change-Id: Ibb9ae9d92106bd9389d4fe2a61c3e5cde3f30805
Change-Id: I76efdda7a3e38775dbd05229008219850c54e928
Change-Id: I3e99e64a7ad09359722edef6d42ab564a65b4828
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I ran mkgfsnemsioctl
, nemsio_chgdate
, nemsio_get
and nemsio_read
using a canned dataset. I then reran using the UFS_UTILS versions of these utilities. The output matched. So, these utilities were correctly incorporated into the nemsio library. I can't speak to the other changes made under this pull request.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
Thanks, All,
It is very good.
Hang
…On Mon, May 18, 2020 at 11:53 AM Kyle Gerheiser ***@***.***> wrote:
***@***.**** approved this pull request.
Looks good
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#11 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKWSMFG5K6APWIBX5BJYR63RSFKVXANCNFSM4NCP6GPA>
.
|
Please let me know when this gets installed on WCOSS and our RDHPCS machines. Then, I will remove them from UFS_UTILS. |
This PR:
nemsio_get
,nemsio_read
,nemsio_chgdate
,mkgfsnemsioctl
from UFS_UTILS here. These are the current versions fromfeature/cmake
branch from the fork of @GeorgeGayno-NOAA