8000 Added throttling reasons and cpu uptime by gerth2 · Pull Request #507 · PhotonVision/photonvision · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Added throttling reasons and cpu uptime #507

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Oct 29, 2022

Conversation

gerth2
Copy link
Contributor
@gerth2 gerth2 commented Oct 19, 2022

While doing some debugging last night, I had a number of issues with my power supply on the Pi. When I hooked up a keyboard and screen manually I quickly found root cause due to the messages in the console from the kernel, but I wasn't seeing these show up anywhere else in an obvious fashion.

The symptoms actually align some ancedotal reports from @dkt01 of settings not saving and recalling - if a reboot had happened on or during the settings write, it could cause intermittent issues.

I added Raspberry-Pi specific throttling reason and CPU uptime metrics to the Settings/General tab to help identify this as root cause.

image

@gerth2 gerth2 requested a review from a team as a code owner October 19, 2022 15:59
@gerth2 gerth2 force-pushed the additional_cpu_metrics branch from c8c5973 to dbb7a26 Compare October 22, 2022 01:31
Copy link
Contributor
@mcm001 mcm001 left a comment
< 8000 div data-view-component="true" class="js-comment-hide-on-error flash flash-warn flash-full">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait why is index.html no longer the default UI has not been copied text?

@mcm001
Copy link
Contributor
mcm001 commented Oct 29, 2022

I'm bad, it wasn't broken by this pr -- we should fix but I'll merge regardless

@mcm001 mcm001 merged commit c4500ce into PhotonVision:master Oct 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0