Dump snapshots if the next time is strickly greater than the dump time.

Fixes #447 (closed).

The problem was arising when we were asking for a dump (of any kind) at exactly a time where we would have had a snapshot. That's rare hence went unnoticed. It was triggering a spurious drift_all.

Could you check whether we now pass all your tests successfully?

mentioned in issue #447 (closed)

Can confirm this fixes the issue.

Plot is broken (as always with the automated suite in 1D, still don't know why) but there are no wiggles.

Is this the usual thing?

Yeah, this looks exactly correct.

Comparison with the unfixed version:

So what's wrong with the test suite?

mentioned in commit 15939e28

merged