Improve CPU throttle lines in TinyPilot logs

tiny-pilot / tinypilot

Use your Raspberry Pi as a browser-based KVM.

https://tinypilotkvm.com

MIT License

3.05k stars 255 forks source link

Closed mtlynch closed 1 year ago

mtlynch commented 1 year ago

The CPU throttle line is currently inconsistent with most other lines in the TinyPilot logs.

We should clean it up to be more user-friendly and consistent with our other lines.

throttled=0x0

CPU throttled since boot: No

jdeanwallace commented 1 year ago

While reviewing https://github.com/tiny-pilot/tinypilot/pull/1543, I noticed that the current style of the logs is for the text after the colon to be lowercase. For example:

TinyPilot state
Read-only filesystem: off
SSH access: enabled
Mouse jiggler: disabled

So in this case, should we use a lowercase "yes"/"no"?

mtlynch commented 1 year ago

Yeah, that's true. We should stick to lowercase.