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.

Sign up for GitHub

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

Jump to bottom

Add unit testing to ocean_merge program #997

Draft
wants to merge 20 commits into
base: develop
Choose a base branch
from
Draft

Add unit testing to ocean_merge program #997

wants to merge 20 commits into from

Conversation

GeorgeGayno-NOAA
Copy link
Collaborator

DESCRIPTION OF CHANGES:

Refactor the ocean_merge code into subroutines. Add unit testing.

TESTS CONDUCTED:

If there are changes to the build or source code, the tests below must be conducted. Contact a repository manager if you need assistance.

  • Compile branch on all Tier 1 machines using Intel (Orion, Jet, Hera, Hercules and WCOSS2).
  • Compile branch on Hera using GNU.
  • Compile branch in 'Debug' mode on WCOSS2.
  • Run unit tests locally on any Tier 1 machine.
  • Run relevant consistency tests locally on all Tier 1 machines.

DEPENDENCIES:

None.

DOCUMENTATION:

N/A

ISSUE:

Fixes #944.

George Gayno and others added 20 commits September 26, 2024 12:41
Separate the namelist read routine and netcdf error
e17b8d8 
routines into their own library.

Write a unit test for the namelist read routine.

Fixes ufs-community#944.
Remove some commented out code.
b204a05 
Fixes ufs-community#944.
Move the merging algorithm to its own routne.
d89bb26 
Fixes ufs-community#944.
Create an initial unit test for the 'merge' routine.
70c1233 
Fixes ufs-community#944.
Add doxygen to the 'merge' routine.
610c760 
Fixes ufs-community#944.
Add more test points to the 'merge' unit test.
e8e8343 
Fixes ufs-community#944.
Finish unit test for 'merge' routine.
c9d9a96 
Fixes ufs-community#944.
Merge branch 'develop' into ocnmrg_tests
da2fd09 
Fixes ufs-community#944.
Move write of output file to its own routine.
f013a59 
Fixes ufs-community#944.
Move read of lake mask file to its own routine.
c64161b 
Fixes ufs-community#944.
All unit test for routine read_lake_mask.
b4188b9 
Fixes ufs-community#944.
Merge branch 'develop' into ocnmrg_tests
e1c0e34 
Fixes ufs-community#944.
Place the read of the model grid dimensions in its own
b4d6651 
routine.

Fixes ufs-community#944.
Merge branch 'develop' into ocnmrg_tests
95a548d 
Fixes ufs-community#944.
Place read of ocean fraction file in its own routine.
7d4f128 
Fixes ufs-community#944.
Complete doxygen in read_write.F90. Add some diagnostic
f96db7b 
print.

Fixes ufs-community#944.
Add unit test for 'read_ocean_frac'.
e4a2f33 
Fixes ufs-community#944.
Minor updates.
8c79cbe 
Fixes ufs-community#944.
Minor updates to program comments and print statements.
4327472 
Fixes ufs-community#944.
Minor updates to unit tests.
bda77cd 
Fixes ufs-community#944.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create unit tests for ocean_merge code
1 participant
@GeorgeGayno-NOAA