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.

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

[FEATURE REQUEST] Customize update frequency per sensor #343

Open
fdw opened this issue Nov 24, 2024 · 2 comments
Open

[FEATURE REQUEST] Customize update frequency per sensor #343

fdw opened this issue Nov 24, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@fdw
Copy link

fdw commented Nov 24, 2024

Not sure how sensible this is, but I think I'd like to define for each sensor how often it should be updated. For example, it feels like I'd prefer more seldom updates to CPU frequency and temperature. But I'd need to try different values to see which period works best.

@fdw fdw added the enhancement New feature or request label Nov 24, 2024
@joshuar
Copy link
Owner

joshuar commented Nov 28, 2024

Hey @fdw this is a great idea. In 7f8450e I've added preferences for defining the polling interval for the cpu and hwmon sensors! That will be in the next release 🎉

Unfortunately, I can't at the moment add more granular preferences for, say, setting the polling interval of the cpufreq sensors to one value and the cpu usage sensors to another. All of these sensors are currently polled in a batch, so there is only one polling interval for all of them. I should be able to split this up in the future though.

At the very least, hopefully just exposing the ability to control the interval of all cpu/hwmon sensors, which are probably the two most "costly" polling sensor groups, will be useful.

@fdw
Copy link
Author

fdw commented Dec 1, 2024

Sounds good, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants