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

Fortran Tests #48

Merged
merged 11 commits into from
Feb 1, 2024
Merged

Fortran Tests #48

merged 11 commits into from
Feb 1, 2024

Conversation

jatkinson1000
Copy link
Member

@jatkinson1000 jatkinson1000 commented Jan 25, 2024

As part of the coupling process and developing the interface it is useful to have a testing framework.

This ensures code continues to behave as expected during refactoring, and also helps check the validity of new code being written.

This is discussed in #45 but will cover more than just regridding.

Tests:

  • relu function
  • Reading in NN parameters from netCDF
  • Running the net with ones compared to Pytorch
  • Running parameterisation with ones compared to previous results.

@jatkinson1000
Copy link
Member Author

I think this is now ready to be merged back into the main repo, performing tests on the original extracted parameterisation.

My plan is to then merge that into #44 and add additional tests for the developments that have occurred there, as well as checking that the code continues to function as expected.

Copy link
Collaborator

@dorchard dorchard left a comment

Choose a reason for hiding this comment

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

Could we add a brief line in the README that says how to run the tests?
Otherwise looks good (nice test_utils...!)

NN_module/precision.f90 Outdated Show resolved Hide resolved
NN_module/Makefile Show resolved Hide resolved
NN_module/test.f90 Outdated Show resolved Hide resolved
@jatkinson1000 jatkinson1000 merged commit ccf28a4 into main Feb 1, 2024
1 check passed
@jatkinson1000 jatkinson1000 deleted the tests branch February 1, 2024 11:40
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.

2 participants