Test: Adapter Log Parser v0.x.x GitHub

[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

[Mic]

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

Mir ist noch etwas aufgefallen:

Gestern ist eingestellt und im Log steht yesterday

War ein Bug. --> Sollte behoben sein in aktueller Github-Version 0.4.1

[Mic]

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

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

Habe jetzt in Version in aktueller Github-Version 0.4.1 zumindest die Spaltenbreiten optimiert (zumindest für Chrome Browser).

[Mic]

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

@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

Vielen Dank für dein Feedback, schätze ich sehr Ebenso, dass du das direkt aufgegriffen hast für deinen Adapter

Ich bin noch an meinem Adapter beschäftigt (Debugging etc.), daher dauert es noch etwas, bis ich deine Implementierungen testen und Feedback geben kann.

[Mic]

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

@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

Danke für dein Feedback Sigi, schöne View hast du da
Was meinst du mit "springen wieder zurück"? Ich komme nicht gleich dazu, mir das anzusehen, evtl. erst in ein paar Tagen, also nicht wundern

[Mic]

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

@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.

Danke auch für deine Rückmeldung
Das mit Adapter-Auswahl ist schon per Issue adressiert, werde das aber wohl erst angehen, sobald ansonsten alles passt, will ihn erstmal zügig "latest" und dann "stable" hin bekommen Da hab ich noch ein paar Sachen zu tun (u.a. gute Dokumentation, Übersetzungen, etc.).
Meine Idee ist aktuell diese: Now regarding issue #2 - probably it would be better to let the user choose which adapters to add to the filter (so multiple selection by using the ctrl key)....

[Glasfaser]

@Mic

Für was ist diese Einstellung .... ?

.

[Kuddel]

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

@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

jetzt werden die Einträge nicht mehr aufgelistet

javascript.0	2020-04-09 08:44:39.257	info	(63817) script.js.smarthome.z_test.test_2: +++ test +++

Letzter Eintrag:

Konfig:

EDIT:

Die Einträge werden jetzt doch aufgelistet:

Wie entferne ich jetzt noch die +++ am Ende?

EDIT 2:

Habe jetzt folgenden Workaround:
Bereinige Log Message*: /script.js.[^:]*: /,+++ , +++

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

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

[Stephan Schleich]

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

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

@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

Danke für dein Feedback Sigi, schöne View hast du da
Was meinst du mit "springen wieder zurück"? Ich komme nicht gleich dazu, mir das anzusehen, evtl. erst in ein paar Tagen, also nicht wundern

Er meint damit, dass wen du Warn,Error etc. anklickst und ein Info Log kommt er sofort wieder auf Info springt

[Mic]

@sigi234
@Stephan-Schleich sagte in Test: Adapter Log Parser v0.x.x GitHub:

Was meinst du mit "springen wieder zurück"? Ich komme nicht gleich dazu, mir das anzusehen, evtl. erst in ein paar Tagen, also nicht wundern

Er meint damit, dass wen du Warn,Error etc. anklickst und ein Info Log kommt er sofort wieder auf Info springt

Ok, danke euch. Sollte nun behoben sein mit Version 0.4.2.

[sigi234]

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

Ok, danke euch. Sollte nun behoben sein mit Version 0.4.2.

Ja behoben, Danke.

[Mic]

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

Wie entferne ich jetzt noch die +++ am Ende?
EDIT 2:

Habe jetzt folgenden Workaround:
Bereinige Log Message*: /script.js.[^:]*: /,+++ , +++

Dein Feedback zeigt, wie wichtig eure Tests sind
Es gab noch einen Fehler bezüglich "multiple regex/string config values separated by comma". Habe ich nun mit 0.4.3 behoben.
Dadurch solltest du auch deinen Workaround nicht mehr benötigen.