Log calls should all be lazy-formatted

[Anthony Shaw]

I've noticed that the logger.debug() calls throughout the codebase use f-strings.

This isn't ideal because it eagerly formats the message even when logging isn't in debug mode.

It should use % style messages with the arguments as parameters. There is a pylint check for this