amanusk / s-tui

Terminal-based CPU stress and monitoring utility
https://amanusk.github.io/s-tui/
GNU General Public License v2.0
4.06k stars 140 forks source link

Catch missing threshold on temperature sensors #88

Closed mkesper closed 5 years ago

mkesper commented 5 years ago

It's strictly undefined what should be returned if threshold of a sensor is not set. I defined both get_edge_triggered and get_max_triggered as True.

mkesper commented 5 years ago

An alternative would be to preset the threshold to zero instead of None.

mkesper commented 5 years ago

Alternative is in PR #89

mkesper commented 5 years ago

89 is the better approach, so closing this one.