From 2c4b42850eab58ac45a7d15331c577b480874a99 Mon Sep 17 00:00:00 2001
From: Wolfgang Hottgenroth <wolfgang.hottgenroth@icloud.com>
Date: Thu, 11 Jul 2019 12:06:05 +0100
Subject: [PATCH] introduce inter comm delay, solves most error situations

---
 src/CmdServer.py              |  12 +++++++++++-
 src/CommunicationProcessor.py |   4 ++++
 src/Config.py                 |   2 +-
 src/registers.pkl             | Bin 1322 -> 1384 bytes
 4 files changed, 16 insertions(+), 2 deletions(-)

diff --git a/src/CmdServer.py b/src/CmdServer.py
index 89c49b2..34d3ce2 100644
--- a/src/CmdServer.py
+++ b/src/CmdServer.py
@@ -210,10 +210,20 @@ class CmdInterpreter(cmd.Cmd):
         self.__println("-----------")
         self.__println("List the configured datapoints")
 
+    def do_reset(self, arg):
+        for r in self.registers:
+            r.errorCount = 0
+            r.processCount = 0
+    
+    def help_reset(self):
+        self.__println("Usage: reset")
+        self.__println("-----------")
+        self.__println("Resets the statistics of configured datapoints")
+
     def do_stats(self, arg):
         for i, r in enumerate(self.registers):
             if r.processCount == 0:
-                ratio = None
+                ratio = -1
             else:
                 ratio = float(r.errorCount) / float(r.processCount)
             self.__println("#{0:2d}: {1:15s} ({2:2d}, {3:5d}), pc: {4:7d}, ec: {5:7d}, q: {6:1.4f}"
diff --git a/src/CommunicationProcessor.py b/src/CommunicationProcessor.py
index ba70c2f..f2a03dc 100644
--- a/src/CommunicationProcessor.py
+++ b/src/CommunicationProcessor.py
@@ -5,6 +5,7 @@ import RegisterDatapoint
 from pymodbus.client.sync import ModbusSerialClient
 import wiringpi
 import MyRS485
+import time
 
 
 ERROR_PIN = 29
@@ -44,6 +45,9 @@ class CommunicationProcessor(threading.Thread):
                 if client.socket is None:
                     print("renew socket")
                     client.socket = self.__getSerial()
+            finally:
+                time.sleep(self.config.interCommDelay)
+
 
 
 
diff --git a/src/Config.py b/src/Config.py
index 89cd369..2a9abf6 100644
--- a/src/Config.py
+++ b/src/Config.py
@@ -10,4 +10,4 @@ class Config(object):
         self.registerFile = 'registers.pkl'
         self.serialPort = '/dev/ttyAMA0'
         self.serialBaudRate = 9600
-
+        self.interCommDelay = 0.025
diff --git a/src/registers.pkl b/src/registers.pkl
index 5fb85c42f7ab85136e32a9d2a1277ec91892a433..ff7ea93077aad709327224db400ac0f5bd0830f6 100644
GIT binary patch
literal 1384
zcmaiz+fvg|6o%8%Hl^h#;1LxMXi)??C<va53IqxU#N2R^!<HQlN&aMKkc-afj2)-%
zz+-sh8~74l`37F-tvl&~(sssPbkcwCto8kC{r%Lj3n8b=WY6>&@s?Dkx{hhH%!=&>
ztW`#A>@(s;bmWwzAPAPK5eo?=%b&N&zDqon1s*|C$#5lb4V94%+d4$5tfaX@Yy>1A
z2E@v5Usr9J-$7aruQGF=WWq0lSWL~JQxc?ylJFH78Cj)^7jOg3GJU%3xTcOSrI$;a
zfi~ILyE~$>RG;M?o2fcO&vEg37xd;c;x93qof}UYKhRe~zl>8#f)`|mF2R7(!ynCX
z3L{>1Iv7+sxfqt-QLTVrXcwoYziMR7^-RYz*&)tIu?ni^=*0K)PGB>fm4s*{#Pb}l
zBAw$>*e<Y5hVxP~8mJ!KzQ9X}{75wnkC(u(6bm#Oj{ag1mvS@_bzBbiO(X1hnWoBA
zEYoy>c15kjk<?E|Xb-KMjb(p-#FY}R%D6@c=pf}5uCM=$sgz?=PDwcy<usHt&|xVa
zXl=r=Wcs>C7>Po1oEeOA!W%{0tPyS<m`ta{N3I=V{dBC6a8x{6#H|u;%NVDXEg02y
zBhfl=Cxo8Jm8&1pgr2CP3vJPt9m_E7y*A07<nVWkn5w}?TO!^Rrul56H28tWeVZDo
z#BZiR(=?1|ER5)^jC&2^ufKi2!Q8x<{4QXwgn1bcxVb*QV_E?^4%vdz&E<E57@Deo
zsI*!vDE(YK-~l5t;s2pp0v{Ht?&H<Ph3?~YczyMVCnmJ?sEEf+mZtF}jLS*>r!t<k
z@=xP=BY!IS?LG1(EXjDm<Fh~)3$%_9YeHD7A-pW&RTIJtmRk{4WURI#%s^^H5K~`<
zuvWslj6w@S^Arj-gpDFLn-FI4x)ni|vDJz&i|rE#V&M<ol&~Y??SBzAIRcsZ2O-v$
A3;+NC

literal 1322
zcmZ{j%Tv=(6vkWVn<5}0;0t_V@c{z$jnASy0tGLKals;wD>q|E^CdUKu;`4A)6s5S
zyY%mI<A0z#w=Udzlh%?#$m-_gp7Z^_b57oOZXs6C2;DYZPMtZ0D|W@Ocy7_MJ$^Wh
z`1l8ORMA<G6R}u~@7WXy`ItBwe!{5JEQF+-5kYFfqaM|fst513V^kc2?;$P4<)pY^
zDk?RRk-NlsgYMYWQMl(&WP^yQxV%)cxT107G+p788#^@DI-skROaDq|^ND=d#}VXf
z=q5N8D7l(q5s`qN{X9QerzTf&=#^s9tQ7l%K7!+NHXw2Nreb;&{aZL8{ZZtch<dg!
zwrZJs7^nxtTE(+CP740K_-=cuX}D~oVjCI;<sK1Q^VG?(l_5DTqKd9N)O9f|68+A3
zmcemKN_r~uTY9>RGX<7lNtUWJ(HxcuTr7AyhVJ+4EX%Qw+5W#cSHlRwDC=P@FXss^
zFhPHD<x5;)19c`S*<irwL$!)sV#87g3&bNPKWDg_LpcpLV1FTSSzwM;aYbMT1J|4P
zMuW+t&`!R-YjBNG7vo{_q3rxJuGVmk;Ch(+2El|+K3S+YA9|R-Rr03bj|u*Sr$+EE
zR7~Blwj*|TN;sIVV#asSD`vnNH}{ve7#MRsRdm2mX=AFl${`2Ij?|CexLv~?g1haG
zOa5X|XA1Y^Zc+R#rMjwU?;1t!mz#Fu{?a`lm=(k6ko&~NUGYOuLdB2re*qtsLsM&&
zeZ+=WmcBM@GvCVNDxUZ|Q)KOksjT-CPiuHaP>MvH#vE%0dQR}d2Vzmu)0h|1%VFt-
zDi))plbP>HEY%<pEVI<-&?Yx+%wXk!beUlFNa-1@3F&fJ`ehX)N;;7l`-b%zHV9rt
YN*A$tK>9U-e57;{Z-n$(Q<_fv1BO?HG5`Po