Test: Adapter Log Parser v0.x.x GitHub

[Kuddel]

@Mic vielen Dank:

[Dutchman]

Sehr nice mic ! ich hänge mich mal mit dran den als Entwickler ist der adapter sehr praktisch !

In der basis klappt es ziemlich ok, muss natürlich noch ausgiebig testen.
Par kleine Sachen

• Der interface überschlaget mich mit langen texten, da verliert man sehr schnell de faden
• Die Tabellen sind leider sehr unuebersichtlich, auf nem 4k schirm hat man platz aber normaler PC/tabblet fast nicht mit zu arbeiten :
• Es gibt keine uebersetzungen (wen du damit Hilfe brauchst sag bescheid hab das in mehreren Adaptern drin)
• Hab dir n PR geschickt fuer issue/enhancement templates auf git
• es waehre praktisch das alle adapter bereits in der Tabelle stehen wuerde, jetzt muss man alles eintragen das ist bij 40 adapter bissl mies

[Beliar_666]

Mit welchem Widget lässt sich das nun am besten darstellen? Mit dem Json table hab ich immer das Problem, das die Spaltenbreite nie passt.

[Glasfaser]

@Beliar_666

Hier auch ein fertiges View von Mic Link Text , muss nur auf die neuen Datenpunkte geändert werden.

oder nimm mein eingestelles Widget materialdesign-Table

Spoiler

[{"tpl":"tplVis-materialdesign-Table","data":{"oid":"logparser.0.filters.Error.json","g_fixed":true,"g_visibility":false,"g_css_font_text":true,"g_css_background":true,"g_css_shadow_padding":false,"g_css_border":true,"g_gestures":false,"g_signals":false,"g_last_change":false,"countCols":"3","tableLayout":"standard","showHeader":true,"headerTextSize":"small","showColumn0":"true","colType0":"text","textAlign0":"left","showColumn1":"true","colType1":"text","textAlign1":"left","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,"showColumn2":"true","colType2":"text","textAlign2":"left","showColumn3":"true","colType3":"text","textAlign3":"left","label0":"Datum","label2":"Quelle","label1":"Meldung","padding_left0":"10","label3":"Einträge","dataJson":"","padding_left2":"","class":"mdui-table mdui-table-ascard-r250-l mdui-table-striped noSwipe","textSize3":"small","textColor2":"#8f8f8f","textSize2":"small","padding_right3":"","rowHeight":"","textSize0":"small","textSize1":"small","padding_left1":"","textColor0":"#8f8f8f","textColor1":"#8f8f8f","textColor3":"#8f8f8f","imageSize1":"","prefix3":"<span style=\"display: inline-block; word-wrap:break-word;\"></span>","suffix3":"","colorHeaderRowText":"linear-gradient(0deg, #2c2a3f, #4f344c, #2c2a3f)","colorHeaderRowBackground":"#202020","colorRowBackground":"linear-gradient(0deg, #2c2a3f, #4f344c, #2c2a3f)","colorRowText":"#8f8f8f","dividers":"","colorBackground":"linear-gradient(0deg, #2c2a3f, #4f344c, #2c2a3f)","prefix2":"<span style=\"display: inline-block; word-wrap:break-word;\">","prefix1":"<span style=\"display: inline-block; word-wrap:break-word;\">","suffix2":"</span>","suffix1":"</span>","columnWidth0":"70","columnWidth1":"40","columnWidth3":"400","columnWidth2":"80","fontFamily3":"Jura-DemiBold","fontFamily2":"Jura-DemiBold","fontFamily1":"Jura-DemiBold","fontFamily0":"Jura-DemiBold","showColumn4":"true","colType4":"text","textAlign4":"center","name":"Tab_Error","colNoWrap0":true,"visibility-cond":"==","visibility-val":1,"visibility-groups-action":"hide"},"style":{"left":"8px","top":"71px","width":"970px","height":"650px","z-index":"3","background":"linear-gradient(0deg, #2c2a3f, #4f344c, #2c2a3f)","background-color":"#2e2e2e","color":"","overflow-y":"auto","border-radius":"10px","font-family":"Jura-DemiBold"},"widgetSet":"materialdesign"}]

[Negalein]

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

oder nimm mein eingestelles Widget materialdesign-Table

wie schaltest du zwischen den einzelnen Logstufen um?

Derzeit verwende ich beim Script den View von Mic.

[Glasfaser]

@Negalein

Ich nutze das umschalten mit einem Widget nicht !

Habe einzelne Widget´s mit Sichtbarkeit geschaltet , da bin ich mit den Spalteneinstellungen pro Widget variabler .....

Kann dir gerade da nicht helfen , da ich es nicht aktiviert habe !

Bin selber noch am Testen .

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

...suche selber noch ...vielleicht ein Einstellungsfehler von mir

[Negalein]

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

Habe einzelne Widget´s mit Sichtbarkeit geschaltet , da bin ich mit den Spalteneinstellungen pro Widget variabler .....

So wie hier, dass Widgets bei true eines DP sichtbar werden?

[Mic]

@Dutchman
Cool, vielen Dank für dein Feedback und Ideen, die Issues und den PR
Auf die Issues habe ich entsprechend schon auf Github geantwortet, bei Issue 4 habe ich noch eine Frage an dich.

Auf die Punkte, für die du Issues eröffnet hast, bin ich dort auf Github bereits eingegangen.
Hier die weiteren:

Der interface überschlaget mich mit langen texten, da verliert man sehr schnell de faden

Der Aufbau gefällt mir auch noch nicht sonderlich. Mit den "langen Texten" dachte ich eigentlich, ich mach was gutes, d.h. ich erkläre die Felder hier direkt, damit z.B. nicht nur die Option "Remove PID" da steht, und keiner checkt, was damit gemeint ist.
Klar muss das dann in der Doku erklärt sein, aber so sieht man halt die Erklärung direkt.
Ich gebe ein anderes Beispiel:

Das ist ein Screenshot vom Parser-Adapter. Was bedeutet da denn nun "Num", "Rolle", "Alt", "Ersatz", etc.? Klar, steht in der Doku, aber wäre es nicht besser, man sieht die Erklärung gleich im User Interface?

Ich werfe diese Frage mal so in den Raum, mir ist es egal, die Pflege für mich ist sogar leichter und einfacher in der Adapter-Readme als in der admin/index_m.html, aber ich dachte ich tue den Usern was gutes damit.

Die Tabellen sind leider sehr unuebersichtlich, auf nem 4k schirm hat man platz aber normaler PC/tabblet fast nicht mit zu arbeiten

Da bin ich auch nicht glücklich damit. Außer kürzen (wie Term "Logs zusammenfassen") und Spaltenbreiten begrenzen fällt mir da aber nichts zu ein. Evtl. hat ein HTML/CSS Experte hier noch Tipps. Eigentlich bräuchte ich die Konfig dreidimensional

Hab dir n PR geschickt fuer issue/enhancement templates auf git

Nice, danke dafür.

[Glasfaser]

@Negalein

Jipp ..

.

@Mic

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 !?

LOG siehe oben ...

[Mic]

@Glasfaser
@Alle

Hier mal eine Übersicht für die Log-Script-Umsteiger, wie sich die Datenpunkte geändert haben. Exemplarisch für den Filter "Info". Im Log-Script wurde da immer noch "log..." vorangestellt.

Im VIS am besten die Widgets oder die View exportieren, im Texteditor suchen/ersetzen, und dann wieder importieren.

Hilft das?

Log-Script	Adapter Log Parser
0_userdata.0.Log-Script.logInfo.logJSON	logparser.0.filters.Info.json
0_userdata.0.Log-Script.logInfo.logJSONcount	logparser.0.filters.Info.jsonCount
0_userdata.0.Log-Script.logInfo.clearJSON	logparser.0.filters.Info.emptyJson
0_userdata.0.Log-Script.All.clearAllJSON	logparser.0.emptyAllJson
0_userdata.0.Log-Script.All.lastTimeUpdated	logparser.0.lastTimeUpdated
0_userdata.0.Log-Script.All.visView1.clearJSON	logparser.0.visualization.table0.emptyJson
0_userdata.0.Log-Script.All.visView1.outputJSON	logparser.0.visualization.table0.json
0_userdata.0.Log-Script.All.visView1.outputJSONcount	logparser.0.visualization.table0.jsonCount
0_userdata.0.Log-Script.All.visView1.whichFilter	logparser.0.visualization.table0.selection

[Mic]

@Glasfaser
Was ist denn das für ein Datenpunkt "Loganzeige" aus deinem Screenshot? Denn kenne ich nicht.

[Negalein]

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

Was ist denn das für ein Datenpunkt "Loganzeige" aus deinem Screenshot?

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

[Glasfaser]

@Negalein
Richtig ....

[Glasfaser]

@Mic
Hast du vielleicht ein Tip !!

siehe meine Anfrage oben Beitrag 18 ! ( Editieren +++ und Zusammenfassung )

[Mic]

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

siehe meine Anfrage oben Beitrag 18 ! ( Editieren +++ und Zusammenfassung )

Bitte poste den Link des Beitrags, den du meinst, direkt, ich sehe leider weder die Beitragsnummern, noch funktioniert die Beitragssuche nach diesem Text. Der Link ist in der Datumsangabe, neben dem Usernamen...

[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