Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[BFW-6579] [ENHANCEMENT] Add Uptime to Stats #4392

Open
magistersmax opened this issue Jan 6, 2025 · 1 comment · May be fixed by #4397
Open

[BFW-6579] [ENHANCEMENT] Add Uptime to Stats #4392

magistersmax opened this issue Jan 6, 2025 · 1 comment · May be fixed by #4397
Assignees
@CZDanol
Labels
enhancement Improvement proposal based on existing features. good first issue Good first issue for new developers.

Comments

@magistersmax
Copy link

Printer model

All Models

Describe the enhancement

Is it possible to add a line for the printer’s uptime in the Stats page?

I’m troubleshooting my printer randomly stopping prints with no errors or notifications and it would be handy to know if the printer has rebooted recently.

Expected functionality

A line in the statistics menu that lists the time since boot up.
@magistersmax magistersmax added the enhancement Improvement proposal based on existing features. label Jan 6, 2025
@CZDanol CZDanol self-assigned this Jan 8, 2025
@CZDanol CZDanol changed the title [ENHANCEMENT] Add Uptime to Stats [BFW-6579] [ENHANCEMENT] Add Uptime to Stats Jan 8, 2025
@CZDanol CZDanol added the good first issue Good first issue for new developers. label Jan 8, 2025
@CZDanol
Copy link
Contributor

CZDanol commented Jan 8, 2025 •
edited
Loading

Hello, good idea :) Created the BFW-6579 internal ticket.

geogod42 added a commit to geogod42/Prusa-Firmware-Buddy that referenced this issue Jan 10, 2025
@geogod42
Add uptime to statistics menu
7f288c0 
Fixes prusa3d#4392

Add uptime to the statistics menu.

* Add a new function `RecordUptime` in `src/common/app_metrics.cpp` to record the uptime in hours.
* Modify `RecordRuntimeStats` in `src/common/app_metrics.cpp` to call `RecordUptime`.
* Update `src/common/appmain.cpp` to call `RecordUptime` in the `app_run` function.
* Add a new line to display the uptime in hours in the statistics menu in `src/gui/screen_menu_statistics.cpp`.
* Add a new member variable to store the uptime and a new method to update the uptime in `src/gui/screen_menu_statistics.hpp`.
@geogod42 geogod42 linked a pull request Jan 10, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CZDanol CZDanol

Labels
enhancement Improvement proposal based on existing features. good first issue Good first issue for new developers.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add uptime to statistics menu geogod42/Prusa-Firmware-Buddy
2 participants
@CZDanol @magistersmax