Closed milesgranger closed 3 years ago
Thanks for the feedback @milesgranger!
I agree and I've already written some test code for the connector, just need to implement it.
@milesgranger I've managed to get GitHub's actions working to run some tests when a push or pull request is made on the dev or main branches.
I'll start adding more tests in the test
module to ensure that the results are verifiable against the test data (i.e. the ISCWSA clearance and error model test data).
Added test for the iscwsa_mwd
module, comparing the model results against the controlled test data.
Merged into the dev
branch for now.
Added test for the clearance
module and all the tests have now been pushed to the latest release.
Time to close this one I think (although I'll continue to add tests).
Looks like a good bit of software, handling some complex computation. :clap: It may be worthwhile to add some CI/CD with good unit test coverage to help encourage stability and outside contributions. :+1: