ukaea/PROCESS

Remove `test_solver`

Opened this issue · 0 comments

The routine test_solver in the integration tests takes a long time to run. Moreover, I don't know that it is very useful. Tim's comments in the code:
# TODO: I'm not sure how useful this test is anymore? TN
and at the end,
# TODO This might need to assert something, like ifail == 1?

Doc string:

Code to test PROCESS Solver by choosing different starting values
    for the iteration parameters around the initial values in the INPUT file
    Code modified by Sarah Medley in April 2015 to also calculate Q (fusion power/
    injected power) and output this to SolverTest.out

For the moment I will change the NITER, the number of times PROCESS is run, from 10 to 1 in integration/data/test_solver.conf.
Would anyone complain if I remove the test completely?
@timothy-nunn @jonmaddock @jmorris-uk