Two questions:
- what exactly do you mean by: "it may need extending for more insidious failure modes"
- is the new "periodic_adaptivity_at_first_timestep" test indeed simply a copy of the existing 'periodic_adaptivity' test with 'adapt_at_first_timestep' switched on? In this case, I'd much prefer to only have a single test (i.e. only switch on adapt_at_first_timestep in the exisitng 'periodic_adaptivity' test), as 'periodic_adaptivity_at_first_timestep' would duplicate the test coverage of 'periodic_adaptivity'.
Hey James, thanks for committing this.
Two questions: adaptivity_ at_first_ timestep" test indeed simply a copy of the existing 'periodic_ adaptivity' test with 'adapt_ at_first_ timestep' switched on? In this case, I'd much prefer to only have a single test (i.e. only switch on adapt_at_ first_timestep in the exisitng 'periodic_ adaptivity' test), as 'periodic_ adaptivity_ at_first_ timestep' would duplicate the test coverage of 'periodic_ adaptivity' .
- what exactly do you mean by: "it may need extending for more insidious failure modes"
- is the new "periodic_
The changes in the code look correct to me.