Closed TomWagg closed 10 months ago
Attention: 306 lines
in your changes are missing coverage. Please review.
Comparison is base (
8772c07
) 86.91% compared to head (a2f556c
) 87.74%. Report is 22 commits behind head on develop.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Two main sets of changes here:
checkstate.py
now checks whether a column is actually present in the CHECKSTATE_COLUMNS and throws an error if it isn't rather than fails silently. I also added a check that there aren't more than 15 columns if some insane users wants to condition on 16 variables because that will break it hahateff_1
andteff_2
can now be used in timestep_conditions just like the other variables 😄I also added some instructions to
checkstate.f
in case anyone else needs to add more condition variables one day.p.s. This is my first attempt at coding in fortran so please do check I didn't break something (it doesn't seem like I have but just in case).
This will fix #609