I’d be inclined to say that would be a good idea, it seems many, if not all service status output’s for emonhub contain these lines.
There is little point in trying to restart a service before it’s ready. The default delay for restarts is just 100ms, extending that to 1s might result in some meaningful log messages when emonhub doesn’t start rather than just “too fast”. That will be why it help you find your issue and can also assist with other issues so I’ll flag this for @TrystanLea to consider.
Just by way of comment to streamline the settings code. Each interfacer has to have a unique name so the name = powerwall setting is a bit redundant as there is already the “PowerWall” name. Also readinterval = 10 is doubling up on the interval setting inherited by all interfacers from the parent interfacer code. Some time ago Trystan changed the http interfacer to use “sendinterval” rather than use the existing “interval” all interfacers have and it caused some confusion, it has since been changed back to “interval”.