Test: Adapter Log Parser v0.x.x GitHub

[Glasfaser]

@Mic
Genau über dein @all

https://forum.iobroker.net/topic/31969/test-adapter-log-parser-v0-x-x-github/18

[Dutchman]

@Mic sagte in Test: Adapter Log Parser v0.x.x GitHub:

Cool, vielen Dank für dein Feedback und Ideen, die Issues und den PR

haste wieder nen PR diesmal n par translations als Beispiel

zu den texten, mja macht auch sinn überschlage ist denke ich eher auf die Möglichkeiten bezogen.
Die Tabellen z.b. haben viele spalten (timestamp z.b. format währe doch selbe für alle ?) und die Eingabe Felder sind doch die vielen Optionen halte klein

Black list dahingegen ist übersichtlich meiner Meinung nach

die erste Tabelle ist also bissl zu gross, da müsste man vielleicht par Sachen auslagern der usability wegen

[Mic]

@Negalein sagte in Test: Adapter Log Parser v0.x.x GitHub:

ein selbsterstellter DP um die einzelnen Logs über Sichtbarkeit anzuzeigen!

Die Sichtbarkeit von was?

Die VIS-Tabelle zeigt den Inhalt von logparser.0.visualization.table0.json.
Mit Buttons kann man den Datenpunkt logparser.0.visualization.table0.selection setzen. Also als Wert kann man da dann einen Filter setzen, z.B. also:

Beispiel für Button "Info":

(side note: und gebt btw @Scrounger bitte auch wirklich einen Kaffee aus, er macht hier tolle Arbeit mit seinem Adapter )

Drückt man auf den Button "Info", dann wird der State logparser.0.visualization.table0.selection mit "Info" gesetzt und im Datenpunkt logparser.0.visualization.table0.json dann die gefilterten "Info"-Logs angezeigt. Das zeigt dann die JSON-Tabelle auch an.

Diese Buttons legt man dann auch für "Hubschrauberlandeplatz", "Katze-hat-hunger", "warn" oder whatever an
Durch drücken auf den Button wird dann der Filter in der Tabelle angezeigt.

Daher verstehe ich nicht, warum ihr da einen weiteren Datenpunkt habt...

[Mic]

@Dutchman sagte in Test: Adapter Log Parser v0.x.x GitHub:

haste wieder nen PR diesmal n par translations als Beispiel

Cool, vielen Dank hierfür, schaue ich mir an.

zu den texten, mja macht auch sinn überschlage ist denke ich eher auf die Möglichkeiten bezogen.
Die Tabellen z.b. haben viele spalten (timestamp z.b. format währe doch selbe für alle ?) und die Eingabe Felder sind doch die vielen Optionen halte klein
(...)
die erste Tabelle ist also bissl zu gross, da müsste man vielleicht par Sachen auslagern der usability wegen

Mein "Problem" hier ist schlichtweg: Ich brauche halt im Adapter pro Filter die Einstellungen als Objekt (JSON).
Dafür ist diese Tabelle gedacht, aber ich komme da an meine Grenzen (der oben zitierte Parser-Adapter wohl auch).

Ausgelagert habe ich eh schon wie verrückt Also als globale Einstellungen, wie etwa Max. Anzahl Logeinträge, Spalten-Reihenfolge für Tabelle, Sortierung, etc.
Ich habe dabei versucht, abzuwiegen, was wirklich individuell pro Filter konfiguriert werden muss, und wo man ggf. Abstriche machen muss und ggf. weitere Adapter-Instanzen braucht, um individuelle Fälle abzubilden. Manche Spalten, wie "Max. Länge" (also abschneiden nach x Zeichen) sind sicherlich diskussionswürdig, um "global" zu setzen. Ggf. auch "Logs zusammenfassen", wobei ich Use Cases habe, bei denen ich das nicht möchte. Dann bräuchte ich 3-4 Adapter-Instanzen.

timestamp z.b. format währe doch selbe für alle ?)

Ich persönlich brauche verschiedene. Ausgelagert global bräuchte ich auch da dann mehrere Adapter-Instanzen.

Zusammengefasst:
Hat wer einen besseren Vorschlag, wie man das in einem Adapter gut abgebildet bekommt? Kennt wer Adapter, in denen so was besser umgesetzt wurde?

[Dutchman]

@Mic sagte in Test: Adapter Log Parser v0.x.x GitHub:

Zusammengefasst:
Hat wer einen besseren Vorschlag, wie man das in einem Adapter gut abgebildet bekommt? Kennt wer Adapter, in denen so was besser umgesetzt wurde?

ehm, nope

[Negalein]

@Glasfaser sagte in Test: Adapter Log Parser v0.x.x GitHub:

Zusammenfassen ist nicht aktiviert ... und die +++ soll er auch enfernen was noch nicht klappt.

@Mic

So ungefähr gehts mir auch.
Logs zusammenfassen funktioniert nicht überall.

[Negalein]

@Mic sagte in Test: Adapter Log Parser v0.x.x GitHub:

Die Sichtbarkeit von was?

weil ich zB auf dieser View noch andere Sachen anzeigen lasse. zB das Backitup-Log.

[Glasfaser]

@Mic

So hier nochmal als Zusammnefassung :

Ich kann einsetzen was ich will und ich bekomme es nicht Editiert das +++

Das ist doch das gleiche wie im Script ...oder ist etwas anderes zu beachten
und der Log wird immer zusammengefasst !?

[Mic]

@Glasfaser
Was meinst du mit " ich bekomme es nicht Editiert das +++":

• Willst du die "+++" entfernen?
• Willst du Lgs filtern, die diese 3 Plus in Folge haben?

Wenn du Screenshots machst, am besten bitte mit Tabellenüberschrift, dann ist es einfacher zu verstehen.
Du hast unter "Whitelist 1: UND" die +++ gesetzt. Hier die Erklärung.
Das bedeutet, dass nur dann ein Log gefiltert wird, wenn es "+++" enthält.
Willst du "+++" entfernen, dann nimm die Spalte "Entferne Teile aus der Logzeile".

Sorry, ich werde die nächsten Tage das alles vernünftig in Deutsch und Englisch dokumentieren.

[Mic]

@Glasfaser sagte in Test: Adapter Log Parser v0.x.x GitHub:

und der Log wird immer zusammengefasst !?

Das interessiert mich umso mehr.
Du sagst, du hast "Logs zusammenfassen" deaktiviert, und dennoch werden diese zusammen gefasst.
Ich kann das nicht reproduzieren.
Hast du Log-Beispiele, mit denen sich das so verhält? Am besten direkt aus dem ioBroker-Log in Admin.
Danke dir für deine Mithilfe und Tests

[Glasfaser]

@Mic sagte in Test: Adapter Log Parser v0.x.x GitHub:

Was meinst du mit " ich bekomme es nicht Editiert das +++":

• Willst du die "+++" entfernen?

Richtig , es funktioniert nicht .... das +++ bleibt

• Willst du Lgs filtern, die diese 3 Plus in Folge haben?

Das funktioniert ....

Willst du "+++" entfernen, dann nimm die Spalte "Entferne Teile aus der Logzeile".

Das habe ich gemacht und es funktioniert nicht

Und desweiteren werden die Logs immer noch zusammen gefasst , obwohl ich kein Haken drin habe .

Ich lese mal dein Link --- und teste mal weiter .....

Edit :
du hast oben noch was geschrieben ... werde mal sammeln

[Mic]

@Glasfaser sagte in Test: Adapter Log Parser v0.x.x GitHub:

+++

Wäre super, wenn du dann Github Issues aufmachst. Also hier: https://github.com/Mic-M/ioBroker.logparser/issues

Dank @Dutchman ist hier auch eine Vorlage hinterlegt, welche Infos ich brauche. Das Template ist in Englisch, aber gerne auch dann in Deutsch reinschreiben, verstehe ich auch

Als Entwickler müssen wir Fehler reproduzieren können, daher machen wir das ganze mit den Fragen im Template.

Bitte für jedes Problem (z.B. "+++ wird nicht entfernt, obwohl in Spalte "xxx" eingetragen) ein separates Issue aufmachen.

Danke noch mal für Eure Tests usw. Das ist sehr hilfreich, um hier sehr bald einen stabilen Adapter zu veröffentlichen

[Scrounger]

@Mic
Hab heut schon den ganzen Tag mitgelesen und mich drauf gefreut heute noch den Adapter auszuprobieren. Installiert, läuft, perfekt - vielen Dank super Adapter!

Vorallem die Struktur finde ich sehr durchdacht. Speziell die Datenpunkte für Visualisierung und den selection Datenpunkt, bin begeistert

Hab gleich mal nen MDW Select Wigdet gebastelt für den selection Datenpunkt:

Spoiler

[{"tpl":"tplVis-materialdesign-Select","data":{"oid":"logparser.0.visualization.table0.selection","g_fixed":false,"g_visibility":false,"g_css_font_text":false,"g_css_background":false,"g_css_shadow_padding":false,"g_css_border":false,"g_gestures":false,"g_signals":false,"g_last_change":false,"visibility-cond":"==","visibility-val":1,"visibility-groups-action":"hide","inputType":"text","inputLayout":"regular","showInputMessageAlways":"true","showInputCounter":"true","clearIconShow":"true","listDataMethod":"jsonStringObject","countSelectItems":"1","listPosition":"auto","showSelectedIcon":"prepend-inner","showValue":false,"signals-cond-0":"==","signals-val-0":true,"signals-icon-0":"/vis/signals/lowbattery.png","signals-icon-size-0":0,"signals-blink-0":false,"signals-horz-0":0,"signals-vert-0":0,"signals-hide-edit-0":false,"signals-cond-1":"==","signals-val-1":true,"signals-icon-1":"/vis/signals/lowbattery.png","signals-icon-size-1":0,"signals-blink-1":false,"signals-horz-1":0,"signals-vert-1":0,"signals-hide-edit-1":false,"signals-cond-2":"==","signals-val-2":true,"signals-icon-2":"/vis/signals/lowbattery.png","signals-icon-size-2":0,"signals-blink-2":false,"signals-horz-2":0,"signals-vert-2":0,"signals-hide-edit-2":false,"lc-type":"last-change","lc-is-interval":true,"lc-is-moment":false,"lc-format":"","lc-position-vert":"top","lc-position-horz":"right","lc-offset-vert":0,"lc-offset-horz":0,"lc-font-size":"12px","lc-font-family":"","lc-font-style":"","lc-bkg-color":"","lc-color":"","lc-border-width":"0","lc-border-style":"","lc-border-color":"","lc-border-radius":10,"lc-zindex":0,"jsonStringObject":"[\n\t{\n\t\t\"text\": \"Info\",\n\t\t\"value\": \"Info\"\n\t},\n\t{\n\t\t\"text\": \"Warn\",\n\t\t\"value\": \"Warn\"\n\t},\n\t{\n\t\t\"text\": \"Error\",\n\t\t\"value\": \"Error\"\n\t},\n\t{\n\t\t\"text\": \"WarnAndError\",\n\t\t\"value\": \"WarnAndError\"\n\t},\n\t{\n\t\t\"text\": \"Homematic\",\n\t\t\"value\": \"Homematic\"\n\t}\n]"},"style":{"left":"34px","top":"230px","width":"353px","height":"38px"},"widgetSet":"materialdesign"}]

Jetzt hab ich endlich nen grund gefunden dem Select Widget noch beizubringen mit multistates umgehen zu können.

[sigi234]

@Mic

Hallo, Super Adapter !

Habe mal eine View gebastelt, irgendwo habe ich einen Fehler, die Logs springen wieder zurück.

Kannst du die View mal testen?

VIEW_Log_Mic3_sigi234.txt

[Kuddel]

ich habe das gleiche problem wie @Glasfaser

Ich will alle Logeinträge die +++ enthalten in einer Ansicht zusammenfassen.

Im Log selber sollen die +++ aber nicht auftauchen:

Whitelist: +++
Entferne: /script.js.[^:]*: /, "+++"

Ergebnis:

[Kuddel]

Mir ist noch etwas aufgefallen:

Gestern ist eingestellt und im Log steht yesterday

[Scrounger]

@Scrounger sagte in Test: Adapter Log Parser v0.x.x GitHub:

Jetzt hab ich endlich nen grund gefunden dem Select Widget noch beizubringen mit multistates umgehen zu können.

Zur Info mt der neuen Material Design Widgets Version 0.3.0 geht das jetzt (demnächst im latest).
D.h. beim Select Widget einfach den Datenpunkt selection eintragen

und bei Eingabemethode Objekt hat Werteliste auswählen:

So funktioniert die Auswahl dann out of the box.

[Knallochse]

@Scrounger Ich habe heute mal eine Testview mit dem Select Widget erstellt. Hat super funktioniert.
Wegen Platzmangel habe ich dann in meiner normalen View in einem Popup Fenster zu arbeiten (Widget - jqui -container - icon - view in Dialog).
Da wird das Auswahlmenü nicht angezeit. Als ich dann das Popup Fenster geschlossen hatte war das Auswahlfenster da (lag dahinter). Das spielen mit dem Z-Index brachte leider auch keinen Erfolg.

[Knallochse]

@Mic Echt geile Sache dein Adapter. Funktioniert so weit ich das bis jetzt überblicken kann - Hervorragend .
Wie @Dutchman schon schrieb, wären mehr Adapter zur Auswahl (Parser-Regeln) bestimmt nicht schlecht.
Was man nicht braucht kann man ja deaktivieren / oder löschen.
Bei löschen vielleicht noch einen Button für "Werkszustand "
Die Bedienung ist für mich sehr schlüssig und finde ich toll.

[Mic]

@Kuddel sagte in Test: Adapter Log Parser v0.x.x GitHub:

ich habe das gleiche problem wie @Glasfaser

Ich will alle Logeinträge die +++ enthalten in einer Ansicht zusammenfassen.

Im Log selber sollen die +++ aber nicht auftauchen:

Whitelist: +++
Entferne: /script.js.[^:]*: /, "+++"

Ergebnis:

Danke, damit konnte ich es reproduzieren. War ein Fehler im Adapter. Aber bitte die Strings ohne Anführungszeichen, also /script\.js\.[^:]*: /, "+++" ändern in /script\.js\.[^:]*: /, +++.

--> Sollte behoben sein in aktueller Github-Version 0.4.1
Bitte um dein Test-Feedback

Über folgendes Issue werde ich es später ermöglichen, dass man auch Anführungszeichen (") benutzen kann wenn man will: https://github.com/Mic-M/ioBroker.logparser/issues/11

Mic-M created this issue in Mic-M/ioBroker.logparser

closed Enhancement: allow quotation marks for specifying strings vs. regex #11