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

Graphs turn red although "hot" sensors with bad reasings have been removed from graphs and summaries #193

Open
KyodaiKen opened this issue Jun 9, 2022 · 1 comment

Comments

@KyodaiKen
Copy link

KyodaiKen commented Jun 9, 2022

Step 1: Describe your environment

  • System: AMD Ryzen 5600G on ASRock A520M-ITX/ac (WiFi module removed)
  • OS version: Ubuntu Server 22.04 minimum installation
  • s-tui version: 1.1.3
  • Installation method: pip via root

Step 2: Describe the problem:

Observed Results:

  • Temperature graphs turn red because some of the AUXTIN sensors report temperatures over 100 degrees.

Expected result:

  • Expectation would be that those sensors are not being included into the colorizing and scaling of the graphs since I have hidden the sensors from the graphs AND summaries. So the colors should be green.

Debug Results, output of s-tui -d created in a file _s-tui.log:

_s-tui.log

Step 3: Reproduce the problem:

Steps to reproduce:

  1. Open s-tui using sudo
  2. Observe the graphs being red
@amanusk
Copy link
Owner

amanusk commented Jun 22, 2022

Thank you for the issues and for the interest in the project.
Indeed this is the case right now.
The threshold for temperatures is set globally and not per-sensor.
The color change is triggered after the probing of all the sensors, and not based on the graphs displayed.

Let's leave this issue open to hopefully get to it one day. PRs are welcome of course :)

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

No branches or pull requests

2 participants