Ankündigung

**henfri** · 04.10.2018, 12:25

Moin,

ich weiß, nicht jeder hat onewire.
Aber vielleicht kann mir jemand bzgl. des Codes weiterhelfen?

Gruß,
Hendrik

**bmx** · 04.10.2018, 12:38

Es gibt die lib/network.py, damit könnte man das onewire umstricken. Es wäre.gut zu wissen, welcher Sensor bei Dir zum Abbruch führt und wie Deine Topologie aussieht.

**henfri** · 04.10.2018, 14:01

Hallo Bernd,

vielen Dank für deine Antwort. Ich habe gerade eclipse, pydev und remote-debugging ans Laufen gebracht. Dank deiner Anleitung ganz ohne Rotwein.
Ich habe mehrere problematische Sensoren:

Code:

grep problem ./var/log/ow_busmonitor.log | cut -d' ' -f 5 |sort -u
26.A1B930010000
28.14B51B030000
28.1FCE1B030000
28.88A61B030000
28.D2C61B030000
28.F412E0020000
3A.778F07000000

Aber nur drei kommen vor, wenn keine Verbindung besteht:

Code:

grep "No connection" ./var/log/ow_busmonitor.log | cut -d' ' -f 5 |sort -u
28.88A61B030000
28.D2C61B030000
28.F412E0020000

Allerdings glaube ich, dass dies die Sensoren sind, die *nach* dem sensor abgefragt werden, der zum Abbruch führt, denn die "No connection" logzeile kommt hier her:

Code:

    def _request(self, path, cmd=10, value=None):
...
        if not self.connected:
            raise owex("No connection to owserver.")

Dass die Verbindung nicht besteht, kommt eher hierher:

Code:

                except Exception as e:
                    self.logger.warning("1-Wire: problem reading {} {}: {}".format(addr, path, e))
                    if not self.connected:
                        return
                    else:
                        #pydevd.settrace("192.168.177.20")
                        self.close()
                        break

Die Topologie ist so, wie sie nicht sein sollte. Ein Stern.

Was schwebt dir vor bezüglich der lib/network.py?

Gruß,
Hendrik

**Haiphong** · 04.10.2018, 15:10

Wieviel Meter hat denn Dein Stern ???

Also bis 2x 17 Meter, ca. 20 DS18B20 und BM DS9490 hatte ich absolut keine Probleme.

Kannst Du keinen zweiten Busmaster bzw. den großen Busmaster nehmen ??

**henfri** · 04.10.2018, 16:59

Hallo,

ich weiß nicht, wie lang die Kabel sind. Sie gehen durch das ganze Haus.
Ich habe schon den Professional BM mit zwei Kanälen.
Ich bin recht sicher, dass ich kein Problem hätte, wenn sh.py den Sensor-Cycle nicht beim ersten Problem abbrechen würde.

Gruß,
Hendrik

**henfri** · 04.10.2018, 17:24

Hallo,

ich denke, dies sollte mein Problem lösen:

Code:

    def _sensor_cycle(self):
        if not self.connected:
            return
        start = time.time()
        for addr in self._sensors:
            if not self.alive:
                self.logger.info("1-Wire: Self not alive".format(addr))
                break
            for key in self._sensors[addr]:
                item = self._sensors[addr][key]['item']
                path = self._sensors[addr][key]['path']
                if path is None:
                    self.logger.info("1-Wire: path not found for {0}".format(item.id()))
                    continue
                try:
                    value = self.read('/uncached' + path).decode()
                    if key.startswith('T') and value == '85.0000':
                        self.logger.info("1-Wire: problem reading {0}. Wiring problem?".format(addr))
                        temperror=True
                        continue
                    else:
                        temperror=False
                    value = float(value)
                except Exception as e:
                    self.logger.warning("1-Wire: problem reading {} {}: {}. Trying to continue with next sensor".format(addr, path, e))
                    #if not self.connected:
                    #    return
                    #else:
                    #    self.close()
                    #    break
                else:  #only if no exception
                    if key == 'L':  # light lux conversion
                        if value > 0:
                            value = round(10 ** ((float(value) / 47) * 1000))
                        else:
                            value = 0
                    elif key == 'VOC':
                        value = value * 310 + 450
                    if not temperror: item(value, '1-Wire', path)
                    
        cycletime = time.time() - start
        self.logger.debug("1-Wire: sensor cycle takes {0} seconds".format(cycletime))

spricht da etwas gegen?
Ich würde gerne etwas mehr Statistik machen.
Dazu würde ich gerne drei Sub-Items hinzufügen:
OW_Item habe ich heute.
Zusätzlich gäbe es:
OW_Item.successful
OW_Item.not_successful
OW_Item.successrate

Jetzt muss ich natürlich prüfen, ob es das Item gibt.
Mit get_children() finde ich alle Children. Aber wie prüfe ich den Namen des Child?
Oder gibt es eine bessere option?

Was mir vorschwebt:

Code:

try:
  ...
except:
  if exists(Item.not_successful):
     Item.not_successful(Item.not_successful()+1)
else:
  if exists(Item.successful):
     Item.successful(Item.successful()+1)

Gruß,
Hendrik

**bmx** · 04.10.2018, 20:36

Läuft das owfs lokal auf dem gleichen Rechner wie SmartHomeNG oder auf dem Prof.Busmaster?

**henfri** · 04.10.2018, 21:40

Hallo Bernd,

owfs läuft auf dem Rechner, an dem der Prof-Busmaster angeschlossen ist. Smarthome-ng läuft auf dem gleichen Rechner -aber in einem Docker-Container.

Was hältst du von meiner Anpassung oben?

Gruß,
Hendrik

**Haiphong** · 05.10.2018, 11:31

Ist das Problem auf beiden Kanälen des Professional BM ??

**bmx** · 05.10.2018, 14:22

Den Code kann ich mir derzeit nicht anschauen, nächste Woche vielleicht.

**henfri** · 05.10.2018, 15:23

Hallo,

ja, das Problem habe ich auf beiden Kanälen des Busmasters.

@Bernd: Das wäre nett. Ich würde wohl auch einen PR machen. Ich denke, das sollte so besser sein als vorher. Ich kann mir zumindest kein Scenario vorstellen, bei dem es ein Problem machen würde.

Gruß,
Hendrik

**henfri** · 07.10.2018, 20:35

Hier mein Pull Request:
https://github.com/smarthomeNG/plugins/pull/169

Ankündigung

OneWire Plugin - Probleme mit einigen Sensoren

OneWire Plugin - Probleme mit einigen Sensoren

Kommentar

Kommentar

Kommentar

Kommentar

Kommentar

Kommentar

Kommentar

Kommentar

Kommentar

Kommentar

Kommentar

Kommentar