Split time-stepper-history cleaning from update

[wthrowe]

Reduces the amount of data we need to store in checkpoints, mostly in LTS.

Proposed changes

Upgrade instructions

Evolution executables now need to include the Actions::CleanHistory action after each variable update. See the modifications to the executables in this PR for suggested exact changes.

Code review checklist

• [ ] The code is documented and the documentation renders correctly. Run make doc to generate the documentation locally into BUILD_DIR/docs/html. Then open index.html.
• [ ] The code follows the stylistic and code quality guidelines listed in the code review guide.
• [ ] The PR lists upgrade instructions and is labeled bugfix or new feature if appropriate.

Further comments

[nilsdeppe]

Hmm, do you understand the clang-tidy error? Tthis looks like it's telling us to change the STL?

[wthrowe]

I think it's just complaining about charm stuff, but the charm class happens to be stored in a std::optional so the traces go through there.

I haven't examined the charm code in detail, but it looks like it's doing reference counting, which can cause false positives for this sort of automated analysis, so it's probably fine.

[nilsdeppe]

Sounds good! Merging now :)