From 1857fd168eb29ab7cbcf22cd6a39122066e2915e Mon Sep 17 00:00:00 2001
From: Domen Tabernik <domen.tabernik@fri.uni-lj.si>
Date: Tue, 10 Dec 2019 17:07:38 +0100
Subject: [PATCH] Fixed issue when faulty FAN returns invalid values and
 clipped RPM to percent converter to zero for small values.

---
 superfans.py | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/superfans.py b/superfans.py
index a7b99a3..4b98175 100644
--- a/superfans.py
+++ b/superfans.py
@@ -61,7 +61,7 @@ FAN_ZONES_MEMBERS= {
 # SYS1 and SYS2 fans use the following linear equations to
 # convert from RPM to % value
 def SUPERMICRO_4029GP_TRT2_RPM_to_percent(rpm):
-  return rpm * 0.0098 - 11.5479
+  return max(rpm * 0.0098 - 11.5479,0)
 
 def set_fan_with_full_preset(config, speed, zone):
   """
@@ -219,8 +219,11 @@ def ipmi_fan_status(hostname = 'localhost', username=None, password=None, use_en
     if len(fan_str.strip()) > 0:
       fan_stat = fan_str.split("|")
       fan_name = fan_stat[0].strip()
-      fan_rpm = float(fan_stat[1].strip())
-      fan_status_return[fan_name] = fan_rpm
+      try:
+        fan_rpm = float(fan_stat[1].strip())
+        fan_status_return[fan_name] = fan_rpm
+      except ValueError:
+        pass
   return fan_status_return
 
 def get_preset(config):