Fix bugs in REST mode

- If 'charge_limit' was commented out in REST mode there would be a crash - battery_scaling not working correctly in REST mode
springfall2008 · Jun 6, 2023 · b03feb3 · b03feb3
1 parent 3803e2f
commit b03feb3
Showing 1 changed file with 15 additions and 14 deletions.
diff --git a/apps/predbat/predbat.py b/apps/predbat/predbat.py
@@ -88,11 +88,12 @@ def update_status(self, minutes_now):
             self.soc_kw = self.base.sim_soc_kw
         else:
             if self.rest_data:
-                self.soc_kw = self.rest_data['Power']['Power']['SOC_kWh']
-                self.base.log("Inverter {} SOC_Kwh {}".format(self.id, self.soc_kw))
+                self.soc_kw = self.rest_data['Power']['Power']['SOC_kWh'] * self.base.battery_scaling
             else:
                 self.soc_kw = self.base.get_arg('soc_kw', default=0.0, index=self.id) * self.base.battery_scaling
 
+        self.base.log("Inverter {} SOC_Kwh {}".format(self.id, self.soc_kw))
+
         # If the battery is being charged then find the charge window
         if self.charge_enable_time:
             # Find current charge window
@@ -210,20 +211,20 @@ def adjust_battery_target(self, soc):
 
         if current_soc != soc:
             self.base.log("Inverter {} Current charge Limit is {} % and new target is {} %".format(self.id, current_soc, soc))
-            entity_soc = self.base.get_entity(self.base.get_arg('charge_limit', indirect=False, index=self.id))
-            if entity_soc:
-                if SIMULATE:
-                    self.base.sim_soc = soc
+            if SIMULATE:
+                self.base.sim_soc = soc
+            else:
+                if self.rest_api:
+                    self.rest_setChargeTarget(soc)
                 else:
-                    if self.rest_api:
-                        self.rest_setChargeTarget(soc)
-                    else:
+                    entity_soc = self.base.get_entity(self.base.get_arg('charge_limit', indirect=False, index=self.id))
+                    if entity_soc:
                         entity_soc.call_service("set_value", value=soc)
-                    if self.base.get_arg('set_soc_notify', False):
-                        self.base.call_notify('Predbat: Inverter {} Target SOC has been changed to {} % at {}'.format(self.id, soc, self.base.time_now_str()))
-                self.base.record_status("Inverter {} set soc to {} at {}".format(self.id, soc, self.base.time_now_str()))
-            else:
-                self.base.log("WARN: Inverter {} Unable to get entity to set SOC target".format(self.id))
+                    else:
+                        self.base.log("WARN: Inverter {} Unable to get entity to set SOC target".format(self.id))
+                if self.base.get_arg('set_soc_notify', False):
+                    self.base.call_notify('Predbat: Inverter {} Target SOC has been changed to {} % at {}'.format(self.id, soc, self.base.time_now_str()))
+            self.base.record_status("Inverter {} set soc to {} at {}".format(self.id, soc, self.base.time_now_str()))
         else:
             self.base.log("Inverter {} Current SOC is {} already at target".format(self.id, current_soc))