Peter Hill

@dschwoerer After discussing this with @bendudson, we're quite concerned about the potential overhead of the temporary field dumping mechanism, both in terms of memory and performance, for what seems like...

Ok, I think we understand this a bit better now. I'd still be much happier if this was a macro and only turned on for `CHECK > 0`. I'm quite...

This is a bit of a frustrating situation, I guess we didn't allow `zp(-1)` as it would involve extra branching to avoid negative numbers in the mod for periodicity. It...

This didn't run on the v5.1.1 release either, looks like it's expecting a secret that doesn't exist. We should switch it to use the trusted publisher workflow anyway

@dschwoerer This is on my to-do list to look at!

I think there's a bug in the current implementation in `master` and `next`, or at least behaviour that I find surprising. I've modified the `monitor` example to print the name...

I've got my head around the unit tests too, I'll push a fix for them shortly, along with some improved documentation about how the monitor timesteps work

I've fixed the unit tests by converting them to use a mock monitor to check exactly what's getting called and with what arguments, and writing the tests in terms of...

If you restart the simulation, does the IO time "reset"?

Ok, that's a little reassuring I guess. Next question is does this appear in non-hermes-3 models? I can probably check this with something like blob2d or conduction, as they're pretty...