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] An new widget showing CPU temperature #452

Closed
henkiewie opened this issue Jan 28, 2022 · 5 comments
Closed

[FEATURE_REQUEST] An new widget showing CPU temperature #452

henkiewie opened this issue Jan 28, 2022 · 5 comments
Labels
🦄 Feature Request [ISSUE] Suggestion for new feature, update or change

Comments

@henkiewie
Copy link

Is your feature request related to a problem? If so, please describe.

No response

Describe the solution you'd like

Hi Lissy,
I'm interested in a widget that shows me the current temperature of the CPU of my Raspberry PI 4.
My Glances docker container shows it under sensors as cpu_temp1. It can have the same look as gl-current-cpu and gl-current-mem.
I'm very happy with the dashboard. Got netdata up and running also. That dives very deep into all possible parameters you can think of. I wish I had that tool available in my working days as datacenter system engineer 😃
regards
Henk

Priority

Low (Nice-to-have)

Is this something you would be keen to implement

Yes!

@henkiewie henkiewie added the 🦄 Feature Request [ISSUE] Suggestion for new feature, update or change label Jan 28, 2022
@liss-bot

This comment was marked as outdated.

@Lissy93
Copy link
Owner

Lissy93 commented Feb 14, 2022

Hi @henkiewie , I've implemented this in #488 / released in V 2.0.3.
See docs for usage. Let me know how you get on with it :)

If you're enjoying Dashy, please consider Starring the repo, as it helps new users find it :)

screenshot

If there's another widget idea you'd like, then you can use the code from this widget as a template.

@henkiewie
Copy link
Author

The new widget is working (ofcourse 😄 ), but it doesn't seem to accept the interval parameter? If I look at the (F12) console api/3/mem and cpu are constantly updating. but api/3/sensors only if you refresh by hand. Maybe a crazy idea. But wouldn't it be nicer (I mean less resource intensive) if you could make these kind of requests (xhr?) depended on the parameter if the widget is collapsed or not? My poor PI 4 is having a hard time 😢 .
I am already an advocate of dashy on discord and reddit whenever I can 😃 .
Using this as a template is still a bridge too far for me 😄 .

@Lissy93
Copy link
Owner

Lissy93 commented Feb 15, 2022

Glad it's working :) And thank you for sharing Dashy, that means a lot :D
That's a good idea, I can make requests be paused when the widget is collapsed.

The continuous updates seems to be working. If you open up the browser tools, under the networking tab, select XHR and you should see the updates coming through. It might be that the actual Glances sensors plugin isn't showing changes as regularly as your updating.

  - type: gl-cpu-temp
+   updateInterval: 20
    options:
      hostname: http://dns-device.local:61208

Also in terms of performance, it seems that enabling Glance's sensors plugin is known to cause a hit in performance on some systems. Increasing the time between updates (higher update interval) should help with this.

Hope that helps :)

@henkiewie
Copy link
Author

Oh I'm sorry, you are right. I put the updateInterval in the wrong widget. That's the dashboard I wanted.
Screenshot 2022-02-15 at 15 46 38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🦄 Feature Request [ISSUE] Suggestion for new feature, update or change
Projects
None yet
Development

No branches or pull requests

3 participants