Pass models.Period around by pointer

[amdw]

This is quite a big struct now, and it's getting bigger over time, so passing it around by value may not be super-efficient. It's probably best to pass it by pointer instead. We do this in some places, but not consistently.