search

DrozmotiX / ioBroker.coronavirus-statistics

Adapter to show Global Corona Virus information and current reports
MIT License
18 stars 19 forks source link

History Logging wird deaktiviert #308

Closed mest2 closed 3 years ago

mest2 commented 3 years ago

Hallo, ich benutze die Aktuelle Version des Adapter. Es fiel mir auf, dass nach jedem Abholen von Daten des Adapters die History-Einträge im Coronavirus-Adapter "deaktiviert" werden. Steht so im Log. Ich kann diese neu anlegen, dann steht auch "enabled" im Log. Mit dem nächsten Datenupdate werden diese allerdings wieder deaktiviert.

Danke fürs Entwickeln! VG Markus

log.pdf

DutchmanNL commented 3 years ago

ich benutze die Aktuelle Version des Adapter.

welche JS-Controller version ? (Host)

mest2 commented 3 years ago

Sorry, Die Version 3.3.15 Admin: 5.1.23 NodeJS: 12.22.4 NPM: 6.14.14

DutchmanNL commented 3 years ago

und welche adapter version? ich bekomme es nicht reproduziert ;/

DutchmanNL commented 3 years ago

habe es mit der stable (0.6.3) und rezenten (0.8.0) probiert bekomme es nicht reproduziert.

@ Apollon77 ich verwende diese 2 Funktionen um Objekte zu erstellen/ergaenzen das dürfte aber nicht zum deaktivieren von History führen ?

            await this.setObjectNotExistsAsync(state, {
                type: 'state',
                common: {
                    name: state_name,
                    role: role,
                    type: type,
                    unit: unit,
                    read: true,
                    write: writable
                },
                native: {},
            });

            // Ensure name changes are propagated
            await this.extendObjectAsync(state, {
                type: 'state',
                common: {
                    name: state_name,
                    type: type, // Also update types t solve log error's and  attribute changes
                },
            });

https://github.com/DrozmotiX/ioBroker.coronavirus-statistics/blob/main/main.js#L781-L801

Apollon77 commented 3 years ago

beides sollte das nicht verursachen

DutchmanNL commented 3 years ago

und welche adapter version? ich bekomme es nicht reproduziert ;/

Bitte auch Version des history Adapter hab da ne Vermutung

mest2 commented 3 years ago

Hi, der History Adapter ist Version 1.9.13, Coronavirus Version 0.8.0. Es wir immer exakt zu der Zeit "deaktiviert" wenn der Coronavirus Adapter Daten zieht.

DutchmanNL commented 3 years ago

Ich kan es nicht reproduzieren .... @Apollon77 hast du nen Ansatz eventuell?

Apollon77 commented 3 years ago

nope. Ausser silly logging beim user und log senden

DutchmanNL commented 3 years ago

nope. Ausser silly logging beim user und log senden

@mest2 koenntest dus das bitte mal machen, sowohl den corona adapter als history auf loglevel silly ?

mest2 commented 3 years ago

Hallo, ich werde mich heute Abend mal dran probieren.

mest2 commented 3 years ago

Ich glaube ich konnte es lösen indem ich beide Instanzen und Adapter mal gelöscht und wieder installiert habe. Nachdem ich anscheinend der Einzigste mit dem Problem bin hielt ich diese Idee für logisch. Bislang scheint es zu passen und die History bleibt erhalten.

DutchmanNL commented 3 years ago

Ich glaube ich konnte es lösen indem ich beide Instanzen und Adapter mal gelöscht und wieder installiert habe. Nachdem ich anscheinend der Einzigste mit dem Problem bin hielt ich diese Idee für logisch. Bislang scheint es zu passen und die History bleibt erhalten.

sehr interessant, sollt sich doch was ändern bitte issue Wiede aufmachen :)

Net-Felix commented 3 years ago

Bei mir leider gleiches problem, wenn ich die Objekte beobachte sieht es so aus als ob der Datenpunkt erst gelöscht wird (bei stadt/kreis der komplette oberpunkt also z.B. coronavirus-statistics.0.Germany.Kreis.Ravensburg) und danach wieder neu angelegt wird. Ich verwende als History den influx adapter. Werde jetzt mal den Corona adapter löschen und neu anlegen

DutchmanNL commented 3 years ago

Bei mir leider gleiches problem, wenn ich die Objekte beobachte sieht es so aus als ob der Datenpunkt erst gelöscht wird (bei stadt/kreis der komplette oberpunkt also z.B. coronavirus-statistics.0.Germany.Kreis.Ravensburg) und danach wieder neu angelegt wird. Ich verwende als History den influx adapter. Werde jetzt mal den Corona adapter löschen und neu anlegen

Lösung war den history neu installieren es gab mal ne Version wo dies Probleme gegeben hat wir können es auf jedenfall nicht reproduzieren bin gespannt was bei dir rauskommt

Net-Felix commented 3 years ago

hat leider nichts gebracht, wird immer noch deaktiviert:

...
2021-08-18 20:32:13.715  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Spandau.cases
2021-08-18 20:32:13.724  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Spandau.cases7_per_100k
2021-08-18 20:32:13.751  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Spandau.cases_per_100k
2021-08-18 20:32:13.772  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Spandau.cases_per_population
2021-08-18 20:32:13.798  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Spandau.death_rate
2021-08-18 20:32:13.822  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Spandau.deaths
2021-08-18 20:32:13.845  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Spandau.last_update
2021-08-18 20:32:13.971  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Steglitz-Zehlendorf.BL
2021-08-18 20:32:13.971  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Steglitz-Zehlendorf.cases
2021-08-18 20:32:13.976  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Steglitz-Zehlendorf.cases7_per_100k
2021-08-18 20:32:14.008  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Steglitz-Zehlendorf.cases_per_100k
2021-08-18 20:32:14.033  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Steglitz-Zehlendorf.cases_per_population
2021-08-18 20:32:14.058  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Steglitz-Zehlendorf.death_rate
2021-08-18 20:32:14.080  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Steglitz-Zehlendorf.deaths
2021-08-18 20:32:14.110  - info: influxdb.0 (76224) disabled logging of coronavirus-statistics.0.Germany.Stadt.Berlin_Steglitz-Zehlendorf.last_update
...

ps, da ich mich nicht kaputt klicken will aktiviere ich das logging per script:

$('channel[state.id=coronavirus-statistics.0.*]').each(function (id, i) {
    sendTo('influxdb.0', 'enableHistory', {
        id: id,
        options: {
            enabled: true,
            storageType: "",
            aliasId: "",
            changesOnly: true,
            debounce: 1000,
            changesRelogInterval: 3600,
            changesMinDelta: 0
        }
    }, function (result) {
        if (result.error) {
            console.log(result.error);
        }
        if (result.success) {
            log("Processing "+id+" success", "info")
        }
    });
});

aber auch bei manuell aktivierter history wird das logging wieder deaktiviert wenn neue daten gepollt werden

Letzter edit für heute: es wird anscheinend nur die history für alle objekte unterhalb von coronavirus-statistics.0.Germany.Stadt und coronavirus-statistics.0.Germany.Kreis deaktiviert, alle anderen bleiben aktiv

bei der adapterversion 0.7.1 tritt das phänomen noch nicht auf

Apollon77 commented 3 years ago

@DutchmanNL schau mal in dem code ... du hattest ja nen größeren PR angenommen. nicht das wirklich alle objekte gelöscht und neu angelegt werden!!

DutchmanNL commented 3 years ago

@DutchmanNL schau mal in dem code ... du hattest ja nen größeren PR angenommen. nicht das wirklich alle objekte gelöscht und neu angelegt werden!!

Noor alles gut das macht er richtig, löscht nur wen land nicht ausgewählt die Meldung ist ja auch. Ur incidenteel und können es nicht reproduzieren

@Net-Felix koenntest du mal bitte den history und SA Adapter auf log Stufe Willy stellen und log schicken zum Zeitpunkt wo das passiert?

Net-Felix commented 3 years ago

So, da das ganze etwas zu viel für hier ist hab ich mal nen gist erstellt:

iobroker.2021-08-19.log

hab alle anderen adapter deaktiviert und das log vorher geflusht

DutchmanNL commented 3 years ago

kann es jetzt reproduzieren, das goldene element war das es nur bei de deutschen Städten passiert kümmere mich drum

related : https://github.com/DrozmotiX/ioBroker.coronavirus-statistics/issues/315

DutchmanNL commented 3 years ago

habs gefunden, testet bitte die 0.8.3-0 entweder per git oder NPM installieren (im admin unter adapter Experten modus) sollte jetzt gefixt sein, PS: der bug hat ich nicht mit ner neue version eingeschlichen aber war schon immer drin :/

Net-Felix commented 3 years ago

läuft! Bester Mann!

DutchmanNL commented 3 years ago

läuft! Bester Mann!

super danke ! dan kan ich 0.8.3. als final raushauen heute, aber circa 15Uhr im admin sichtbar :)

DutchmanNL commented 3 years ago

Solved 0.8.3