Divera Plugin #415

grosj · 2019-10-15T06:44:09Z

Plugin to call the Divera REST-Api

Fixes #413

HOTFIX pocsag decoder to solve #413 ; definition of var in both functions.

HOTFIX pocsag decoder

Schrolli91 · 2019-10-15T06:57:28Z

LGTM

flothi · 2019-10-15T07:22:04Z

Schaut gut aus, bis auf die Zeileneinzüge - entweder hab ich nen Knick in der Optik oder die sind teilweise inkonsistent

Schrolli91 · 2019-10-15T07:45:58Z

@flothi jetzt wo du es sagst, das ist mir vorhin gar nicht aufgefallen ...

@grosj schau dir das bitte nochmal kurz an - evtl bietet deine IDE ja ein auto "auto-format" der sollte das schnell gerade ziehen.

grosj · 2019-10-15T08:09:13Z

Jetzt wo ihr es sagt, in Zeile 78 und 80 passt es nicht ganz.
Komisch, wenn ich es mir hier im Pull-Request ansehe, dann sind ein paar mehr Einzüge schief
Und meine Entwicklungsumgebung ist der nano :-)

correcting line indents

flothi · 2019-10-15T08:18:46Z

Ich war mal so frei, schaut bitte nochmal drüber, ob es so passt.

plugins/Divera/Divera.py

Schrolli91 · 2019-10-15T08:32:32Z

@grosj Bitte noch einen kurzen Changelog Eintrag hinzufügen

Schrolli91 · 2019-10-15T09:34:35Z

Hab gerade gesehen das dein PR gegen unseren master stand - habe das eben auf develop geändert und einen kleineren merge Konflikt behoben.

Schrolli91

@grosj
Funktioniert das Plugin bei dir wie gewünscht?
Testen wird schwierig wenn es so weiter im Moment niemand nutzt.

grosj · 2019-10-17T11:50:05Z

@grosj
Funktioniert das Plugin bei dir wie gewünscht?
Testen wird schwierig wenn es so weiter im Moment niemand nutzt.

Ich habe es bei mir konfiguriert. Allerdings mangels Alarmierungen konnte ich es noch nicht testen. Am Sonntag stehen bei uns ein 2 Probealarme an. Dann werde ich es sehen, ob es funktioniert.

Schrolli91 · 2019-10-21T07:23:37Z

Am Sonntag stehen bei uns ein 2 Probealarme an. Dann werde ich es sehen, ob es funktioniert.

Hat es denn wie erwartet funktioniert? ;-)

grosj · 2019-10-21T08:04:57Z

Am Sonntag stehen bei uns ein 2 Probealarme an. Dann werde ich es sehen, ob es funktioniert.

Hat es denn wie erwartet funktioniert? ;-)

Jepp hat es. Sehr gut sogar.
Allerdings hat Divera die API leicht geändert. Das würde ich noch auf den aktuellen Stand bringen. Es hat zwar auch noch mit den "alten" Parametern funktioniert, aber ich würde es trotzdem im Laufe des Tages anpassen wollen.

Schrolli91 · 2019-10-21T09:23:43Z

Kein Problem, einfach bescheid geben wenn du durch bist

Die Divera-API ist unter https://api.divera247.com beschrieben. Anpassungen an diese Schnittstellenbeschreibung. Aus type wurde title.

grosj · 2019-10-21T13:20:40Z

Anpassung habe ich anhand der API-Beschreibung von https://api.divera247.com durchgeführt.

Schrolli91

Aus meiner Sicht ok - wenns dir Recht ist würde ich nochmal einen Alarm bei dir abwarten ob weiterhin alles funktioniert, bevor ich merge.

grosj · 2019-10-21T13:24:29Z

Aus meiner Sicht ok - wenns dir Recht ist würde ich nochmal einen Alarm bei dir abwarten ob weiterhin alles funktioniert, bevor ich merge.

Von mir aus gerne. Sollte ja auch alles richtig und ohne Fehler funktionieren.
Aber wie gesagt, wir sind allerdings nicht die Einheit, die am häufigsten alarmiert wird.

Schrolli91 · 2019-11-04T06:23:35Z

@grosj bisher keine weiteren Probleme? Dann würde ich nämlich einen merge forcieren.

Ist kein Wert oder ein anderer Wert als true/false für priority gesetzt, wird ein Divera-Alarm ausgelassen.

Beschreibung zum Divera-Plugin angepasst. Typo korrigiert.

plugins/Divera/Divera.py

grosj · 2019-11-04T07:28:01Z

@grosj bisher keine weiteren Probleme? Dann würde ich nämlich einen merge forcieren.

Bisher funktioniert es ohne Probleme. Hab gerade noch eine kleine Änderung durchgeführt und in der Config noch etwas Doku nachgetragen.
Es gibt nun keine Default-Priority. Wird diese nicht angegeben oder ein anderer Wert, dann wird keine Divera-Alarmierung ausgeführt.

Schrolli91 · 2019-11-04T07:30:03Z

Ok sieht gut aus, dann würde ich das ganze jetzt mergen

grosj · 2019-11-04T07:31:15Z

Ok sieht gut aus, dann würde ich das ganze jetzt mergen

Prima. Danke Dir.

c-bob · 2019-12-02T12:01:56Z

Moin! Ich weiß jetzt nicht, ob das der richtige Weg zur Kontaktaufnahme ist. Bin neu bei GitHub. Wie sieht es mit dem divera Plugin aus? Offiziell wird es ja noch nicht erwähnt. Wie müsste ich das denn einbinden?

grosj · 2019-12-02T12:24:45Z

Moin! Ich weiß jetzt nicht, ob das der richtige Weg zur Kontaktaufnahme ist. Bin neu bei GitHub. Wie sieht es mit dem divera Plugin aus? Offiziell wird es ja noch nicht erwähnt. Wie müsste ich das denn einbinden?

Hallo, es ist bereits in dem Developer-Branch enthalten und würde (nach meinem Kenntnisstand) bei einem neuen Release in den Master mit überführt.

Schrolli91 · 2019-12-02T13:25:08Z

Das ist soweit korrekt, beim nächsten Release wandert es direkt in den Master Branch

c-bob · 2019-12-02T16:31:06Z

Ah, okay, danke! Wie ist denn der Stand für's nächste Release? Ist developer schon stabil genug? Konntet ihr divera schon genügend testen?

Schrolli91 · 2019-12-02T18:17:44Z

Bisher sind keine Probleme bekannt...
Ich denke das ich gegen Ende des Jahres das nächste Release bringe, aber du kannst den Dev relativ bedenkenlos installieren - es gibt sowieso nur noch kleinere Neuerungen und Bugfixes (Entwicklung von BW3 wird forciert) - also keine Änderungen, die die generelle Stabilität beeinträchtigen würden.

c-bob · 2019-12-02T21:25:48Z

War schon dabei, hab mal ein diff über alles gemacht - Änderungen halten sich in Grenzen. Gibt's zu bw3 schon irgendwo etwas zum reinschnuppern?

grosj · 2019-12-03T06:42:04Z

Ah, okay, danke! Wie ist denn der Stand für's nächste Release? Ist developer schon stabil genug? Konntet ihr divera schon genügend testen?

Ich hab das Plugin bei mir am Laufen. Keine Probleme ...

Schrolli91 · 2019-12-03T08:31:05Z

https://github.com/BOSWatch/BW3-Core/tree/develop
Allerdings noch sehr early-stage - bisher keine Plugins portiert. Es existiert also nur das Grundgerüst ohne Funktionalität am Endpunkt.

unter http://boswatch.de gibts immer mal wieder News zu BW3 wenn sich was tut - leider bin ich zeitlich momentan wieder etwas mehr gebunden.

papierkorb27 · 2020-01-07T11:04:06Z

Schönen Dank schonmal für das Plugin! Wäre es möglich, ein kurze Beschreibung zur Konfiguration zu bekommen? Ich habe mit Divera nicht ganz so viel zu tun, würde es aber gerne einmal testen.

Meine Ideen:

In der Config den Divera API Key eintragen
den poc_title bzw. poc_text anpassen
jetzt einen regex-Filter anlegen, z.B.: divera = POC;ric;divera;*;0123456
jetzt das Plugin aktivieren

Verständnisfrage: wo kann ich denn einstellen, dass er bei verschiedenen RICs verschiedene Gruppen alarmiert?

grosj · 2020-01-07T12:02:21Z

Schönen Dank schonmal für das Plugin! Wäre es möglich, ein kurze Beschreibung zur Konfiguration zu bekommen? Ich habe mit Divera nicht ganz so viel zu tun, würde es aber gerne einmal testen.

Meine Ideen:

In der Config den Divera API Key eintragen

den poc_title bzw. poc_text anpassen

jetzt einen regex-Filter anlegen, z.B.: divera = POC;ric;divera;*;0123456

jetzt das Plugin aktivieren

Verständnisfrage: wo kann ich denn einstellen, dass er bei verschiedenen RICs verschiedene Gruppen alarmiert?

Hallo,
Du kannst den API Key in der config.ini genauso wie den poc_title und poc_text eintragen. Ebenfalls kannst Du dort das PlugIn aktivieren. RegEx-Filter können dort ebenfalls konfiguriert werden.
Dies ist in der config.template.ini bereits enthalten.

Die Einstellung bzgl. der Alarmierung verschiedener Gruppen bei unterschiedlichen RICs erfolgt, ist über das PlugIn nicht möglich.
Aber soweit ich weiß lässt sich dies doch in der Schnittstellenkonfiguration von Divera hinterlegen.

papierkorb27 · 2020-01-07T20:38:35Z

Hallo,
Du kannst den API Key in der config.ini genauso wie den poc_title und poc_text eintragen. Ebenfalls kannst Du dort das PlugIn aktivieren. RegEx-Filter können dort ebenfalls konfiguriert werden.
Dies ist in der config.template.ini bereits enthalten.

Die Einstellung bzgl. der Alarmierung verschiedener Gruppen bei unterschiedlichen RICs erfolgt, ist über das PlugIn nicht möglich.
Aber soweit ich weiß lässt sich dies doch in der Schnittstellenkonfiguration von Divera hinterlegen.

Die Konfiguration habe ich ja so wie ich beschrieben hatte bereits durchgeführt, nur die Umsetzung wie man über verschiedene RICs verschiedene Gruppen alarmiert, war mir unklar. Also würde man die RIC-Differenzierung auf der Divera-Seite durchführen und nur ggf. mit einem Regex-Filter die RICs vorfiltern. Wie gesagt, mit der Divera API habe ich mich bisher nicht beschäftigt, dann schaue ich mir das mal an.

Dank und Grüße!

grosj · 2020-01-08T12:39:43Z

Die Konfiguration habe ich ja so wie ich beschrieben hatte bereits durchgeführt, nur die Umsetzung wie man über verschiedene RICs verschiedene Gruppen alarmiert, war mir unklar. Also würde man die RIC-Differenzierung auf der Divera-Seite durchführen und nur ggf. mit einem Regex-Filter die RICs vorfiltern. Wie gesagt, mit der Divera API habe ich mich bisher nicht beschäftigt, dann schaue ich mir das mal an.

Dank und Grüße!

Über die Divera-Schnittstelle kann auf jeden Fall die RIC mitgegeben werden. Dies ist allerdings noch nicht im Plugin enthalten.

Das eine Zuordnung von RICs zu Gruppen über Divera möglich wäre ist nur eine Vermutung von mir. Wir setzen die Free-Version von Divera ein. Da ist nicht alles möglich. Und man sieht auch nicht, was möglich sein könnte.

Bastian Schroll and others added 6 commits October 13, 2019 17:56

HOTFIX pocsag decoder

7904fc3

Fixes #413

HOTFIX pocsag decoder

a994381

HOTFIX pocsag decoder to solve #413 ; definition of var in both functions.

Merge pull request #414 from Schrolli91/hotfix_poc-decoder

18bde9d

HOTFIX pocsag decoder

Update CHANGELOG.md

ca26f1d

Add Section for Divera-Plugin

4b8f647

Plugin for Divera

656fe48

Schrolli91 added Plugin Review needed labels Oct 15, 2019

Update Divera.py

9f251bd

correcting line indents

Schrolli91 reviewed Oct 15, 2019

View reviewed changes

plugins/Divera/Divera.py Outdated Show resolved Hide resolved

plugins/Divera/Divera.py Outdated Show resolved Hide resolved

grosj added 2 commits October 15, 2019 10:46

Update line feeds

6b638c5

Changlog zum Divera-Plugin

3d76997

Schrolli91 changed the base branch from master to develop October 15, 2019 09:32

Merge branch 'develop' into master

c3589a9

Schrolli91 removed the Review needed label Oct 17, 2019

Schrolli91 self-requested a review October 17, 2019 10:58

Schrolli91 approved these changes Oct 17, 2019

View reviewed changes

grosj added 2 commits October 21, 2019 15:15

Update config.template.ini

0181075

Anpassung auf Divera-Schnittstelle

0dfe1ee

Die Divera-API ist unter https://api.divera247.com beschrieben. Anpassungen an diese Schnittstellenbeschreibung. Aus type wurde title.

Schrolli91 approved these changes Oct 21, 2019

View reviewed changes

grosj added 2 commits November 4, 2019 08:19

Keinen Default für priority

ddbddbb

Ist kein Wert oder ein anderer Wert als true/false für priority gesetzt, wird ein Divera-Alarm ausgelassen.

Update config.template.ini

3fc0e6d

Beschreibung zum Divera-Plugin angepasst. Typo korrigiert.

Schrolli91 reviewed Nov 4, 2019

View reviewed changes

plugins/Divera/Divera.py Show resolved Hide resolved

Schrolli91 merged commit 4321234 into Schrolli91:develop Nov 4, 2019

This was referenced Apr 16, 2020

Prepare Release for BW 2.5 #442

Closed

Prepare Release for BW 2.5 #443

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Divera Plugin #415

Divera Plugin #415

grosj commented Oct 15, 2019

Schrolli91 commented Oct 15, 2019

flothi commented Oct 15, 2019

Schrolli91 commented Oct 15, 2019

grosj commented Oct 15, 2019 •

edited

Loading

flothi commented Oct 15, 2019

Schrolli91 commented Oct 15, 2019

Schrolli91 commented Oct 15, 2019

Schrolli91 left a comment •

edited

Loading

grosj commented Oct 17, 2019

Schrolli91 commented Oct 21, 2019

grosj commented Oct 21, 2019

Schrolli91 commented Oct 21, 2019

grosj commented Oct 21, 2019

Schrolli91 left a comment

grosj commented Oct 21, 2019

Schrolli91 commented Nov 4, 2019

grosj commented Nov 4, 2019

Schrolli91 commented Nov 4, 2019

grosj commented Nov 4, 2019

c-bob commented Dec 2, 2019

grosj commented Dec 2, 2019

Schrolli91 commented Dec 2, 2019

c-bob commented Dec 2, 2019

Schrolli91 commented Dec 2, 2019

c-bob commented Dec 2, 2019

grosj commented Dec 3, 2019

Schrolli91 commented Dec 3, 2019

papierkorb27 commented Jan 7, 2020

grosj commented Jan 7, 2020

papierkorb27 commented Jan 7, 2020

grosj commented Jan 8, 2020

Divera Plugin #415

Divera Plugin #415

Conversation

grosj commented Oct 15, 2019

Schrolli91 commented Oct 15, 2019

flothi commented Oct 15, 2019

Schrolli91 commented Oct 15, 2019

grosj commented Oct 15, 2019 • edited Loading

flothi commented Oct 15, 2019

Schrolli91 commented Oct 15, 2019

Schrolli91 commented Oct 15, 2019

Schrolli91 left a comment • edited Loading

Choose a reason for hiding this comment

grosj commented Oct 17, 2019

Schrolli91 commented Oct 21, 2019

grosj commented Oct 21, 2019

Schrolli91 commented Oct 21, 2019

grosj commented Oct 21, 2019

Schrolli91 left a comment

Choose a reason for hiding this comment

grosj commented Oct 21, 2019

Schrolli91 commented Nov 4, 2019

grosj commented Nov 4, 2019

Schrolli91 commented Nov 4, 2019

grosj commented Nov 4, 2019

c-bob commented Dec 2, 2019

grosj commented Dec 2, 2019

Schrolli91 commented Dec 2, 2019

c-bob commented Dec 2, 2019

Schrolli91 commented Dec 2, 2019

c-bob commented Dec 2, 2019

grosj commented Dec 3, 2019

Schrolli91 commented Dec 3, 2019

papierkorb27 commented Jan 7, 2020

grosj commented Jan 7, 2020

papierkorb27 commented Jan 7, 2020

grosj commented Jan 8, 2020

grosj commented Oct 15, 2019 •

edited

Loading

Schrolli91 left a comment •

edited

Loading