-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add hide option to sensors plugin #1596
Conversation
@@ -127,14 +127,20 @@ def update(self): | |||
|
|||
pass | |||
|
|||
# Set the alias for each stat | |||
# Update the stats | |||
self.stats = [] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had no idea what would be the best, and good, way to implement the hide option. I have supposed there is no sync issue doing this way, using no temporary variable.
Done in version 3.1.4. Thanks for the PR @cdeletre |
@nicolargo is it possible to reapply this pull? It looks like somehow it didn't make it into either version 3.1.4 or the develop branch. |
Does this mean i can set the alias=hide and it will hide the senor? Assuming its put back in 3.1.5... |
Not exactly, but similar. It follows the other plugin style of:
|
oh, thanks for the clarification! That makes perfect sense, i hadnt used the hide function yet and when i searched the docs for hide sensor it wasnt shown as available(so i didnt try it). Edit: not working 3.1.5, could of sworn i saw it working.... |
Description
This pull request proposes to add the support of hide option to the sensors plugin. It may be useful when you want to hide core temperatures and only show main CPU temp of 10+ core CPUs.
Resume