Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Current hour not fully updated with data #51

Closed
85ni opened this issue Jul 6, 2021 · 20 comments
Closed

Current hour not fully updated with data #51

85ni opened this issue Jul 6, 2021 · 20 comments
Labels
bug Something isn't working question Further information is requested

Comments

@85ni
Copy link

85ni commented Jul 6, 2021

Hi,
Auch von mir Danke für den tollen Adapter.
ich habe aktuell v 0.9.7 installiert. Der Adapter läuft alle 30 Minuten.
Mir ist nun aufgefallen, dass die Werte in current_hour nicht korrekt aktualisiert werden. Wenn ich die Werte mit den Inhalten aus dem 60minutes/day0 vergleiche stehen da ganz andere Werte drin als jetzt in current_hour. Bspw. wurde Regenmenge heute 13:57 aktualisiert. Über den Tag verteilt sind die Werte in den einzelnen Stunden ebenfalls unterschiedlich, aber im current_hour Ordner steht seit immer nur 0.1 drin. Irgendwie scheint es die Einträge aus den jeweiligen Stunden-Ordner nicht korrekt auf die current_hour zu kopieren.
Aktuell steht bspw 2.5 im ordner 20:00, current_hour immer noch 0.1. Hoffe ist klar was ich meine :-)

Merci für die Prüfung.

@baerengraben
Copy link
Owner

Hi

Das ist komisch. Current_hour wird bei jeder Forecast-Abfrage aktualisiert oder aber spätestens jede volle Stunde. Habs gerade nochmals bei mir überprüft. Bei mir scheint es zu funktionieren.

Was steht bei dir im attribut "forecast/current_hour/local_date_time"? Da müsste die jeweilig aktuelle Stunde drin stehen.

Kannst du bitte bei deiner Instanz den debug-modus einschalten und mir (nach mind. 2-3h Laufzeit) die Logs hier posten?

PS:
Mir ist beim Test aber aufgefallen, dass beim Tageswechsel (also von 23:00 auf 00:00) die Werte von day0/00:00 statt day1/00:00 übernommen werden. Von 00:00 bis zur nächsten Forecast Abfrage (bei mir um 00:18) werden entsprechend die falschen Werte in current_hour gehalten. => Das ist definitiv ein Fehler. Der fällt aber nur auf, wenn man viel zu spät ins Bett geht. :)
Dazu mache ich noch einen Issue auf.

@85ni
Copy link
Author

85ni commented Jul 7, 2021

Interessanterweise hat es heute nach dem Aktivieren des Debug Modus plötzlich funktioniert. Bin die nächsten Tage unterwegs und kann es nicht im Detail beobachten. Mache ich aber nächste Woche. Allenfalls hat sich das Problem ja von selbst gelöst:-)

@baerengraben baerengraben added the question Further information is requested label Jul 13, 2021
@85ni
Copy link
Author

85ni commented Jul 16, 2021

klingt vielleicht etwas doof, aber wenn ich den Debug Modus aktiviere auf dem Adapter, dann scheint es die Werte korrekt zu aktualisieren. Sobald ich ihn wieder deaktiviere geht es nicht mehr. Habe jetzt noch ein paar Werte mehr historisiert und beobachte noch ein wenig ob ich einen genaueren Zusammenhang feststellen kann

@baerengraben
Copy link
Owner

baerengraben commented Jul 19, 2021

Merci fürs Update. Ich habe den Adapter mal mit dem Log-Level "info" laufen gelassen. Die Current Hour wird bei mir auch so korrekt stündlich übernommen.

Ich kann mir aktuell auch nicht ausmalen, wieso das Log-Level die Funktion

function setCurrentHour(self){
beeinflussen sollte.

Ich denke die Ursache muss woanders liegen.

@85ni
Copy link
Author

85ni commented Jul 27, 2021

Danke für dein Feedback. Spannend. habe es jetzt über die letzten Tage beobachtet. Current_Hour wird bei mir effektiv nur stündlich aktualisiert, wenn der Loglevel Debug ist. Sobald er auf Info oder auch Error steht, dann erfolgt die Aktualisierung nicht mehr regelmässig. Ich habe die für mich relevanten Werte über Influx protokolliert und da kann ich dann schön die Werte mit den Zeiten aus 60minutes vergleichen. Ebenfalls wird der Timestamp irgendwie nach Zufall aktualisiert. Bis jetzt konnte ich nicht feststellen, wann welcher Wert aktualisiert wird. Nur, dass es im Debug-Level so geschieht analog den 60Minutes und ohne eben nicht.

Aber ok, da bei mir der Adapter noch nicht produktiv genutzt wird, ist das jetzt nicht sonderlich kritisch. Wollte eigentlich gerne alles umstellen aber so muss ich wohl oder übel noch bei anderen Adaptern bleiben.

@baerengraben
Copy link
Owner

baerengraben commented Jul 30, 2021

Sehr speziell. Ich kann dieses Verhalten wirklich nicht reproduzieren.
Damit ich besser helfen kann bräuchte ich mehr Infos. Betreibst du den ioBroker auf einem Raspi? Mich würde interessieren, ob du dieses Fehlverhalten auf einem komplett "sauberen" Raspi und fisch installiertem ioBroker reproduzieren kannst.

@85ni
Copy link
Author

85ni commented Jul 30, 2021

Bei mir läuft das auf Proxmox in einer VM. Habe jetzt testhalber eine neue VM mit frischem ioBroker installiert. Werde es da mal beobachten wie es sich verhält. Kann etwas dauern, da ich im Moment nicht viel Zeit habe.

@baerengraben
Copy link
Owner

Bin gespannt auf das Resultat! ...Und kein Problem wegen der Zeit. Das Problem läuft ja nicht weg ;)

@baerengraben
Copy link
Owner

@85ni Evtl. weiss ich nun wo das Problem begraben liegt. Bitte installiere mal Version 1.0.0 mit aktiviertem debug-level.

Wird die Meldung "This should not happen. State is undefined or null. So in this run no data is copied for this value" ausgegeben?

@85ni
Copy link
Author

85ni commented Oct 20, 2021

Also ich habe die neuste Version installiert. Danach Debug Level aktiviert. In den Logs sehe ich deinen Fehler jedoch nicht. Es liefert schön die verschiedenen Abfragen für die Stunden etc. Das wird aktualisiert.

Dann kommt auch folgende Meldung "forecast data is available. State.val is: 190: So updating current_hour...read correspondenting hour forecast from swiss-weather-api.0.forecast.60minutes.day0.actual_hour and write it to swiss-weather-api.0.forecast.current_hour"

Jetzt aber, alle Werte unter forecast.current_hour sind nur noch Initalwerte (0, dummy etc). Sind zwar "grün" mit letzter Aktualisierung wenn die Abfrage gelaufen ist aber es wird gar nichts mehr übertragen. Habe das sowohl mit Debug wie auch Info Level.
Ich habe den Adapter mehrfach neu gestartet um neue Abfragen zu initiieren, leider ohne Veränderung.

@baerengraben
Copy link
Owner

baerengraben commented Oct 20, 2021

Ah, ok. Das war zwar nicht das, was ich erwartet hätte - Aber jetzt weiss ich woran es wohl liegt.

Ich habe mal einen Fix-Versuch getätigt. Kannst du bitte mal deinen aktuellen Adapter komplett löschen (Instanz und Adapter) und anschliessend den swiss-weather-api Adapter aus direkt aus Github installieren, instanzieren und laufen lassen bzw. testen?

@florian1984
Copy link

Habe Adapter Komplett gelöscht und nochmals neu geladen (1.0.1) doch bekomme leider immer noch nicht alle Daten:

image

@Matt77CHE
Copy link

Habe gestern auch alles gelöscht und neu installiert - sieht bei mir leider auch so aus...

@baerengraben
Copy link
Owner

Ok, die Korrektur habe ich bisher nur in Github committed. Habt ihr den Adapter aus Gibhub installiert?

@Matt77CHE
Copy link

Funktioniert;
habe soeben alles noch einmal gelöscht und mittels code kopieren auf github das ganze noch einmal installiert....
jetzt funktioniert es.

@florian1984
Copy link

Habe nichts gesagt :-) herzlichen Dank für dei Korrektur. Einzig mein VIS zickt jetzt noch etwas rum obwohl komplett neu geladen. Wenn ich auf die Tagesansichten klicke passiert nichts mehr:

image

@baerengraben
Copy link
Owner

Stimmt, da war noch etwas krumm. Habe es auf Github korrigiert.
Kannst du es testen?

  • Bitte nochmals den gesamten Adapter komplett entfernen (Instanz und Adapter) und
  • den Adapter dann nochmals aus Github installieren.
  • Anschliessend die Wettervorhersage-View nochmals ganz neu aus Github importieren.

Dann müsste die View wieder funktionieren.

@florian1984
Copy link

Alles so gemacht wie du beschrieben hast. Herzlichen Dank! Jetzt sieht es aus wie vorhin.

@baerengraben
Copy link
Owner

Super, Merci für den Test. Dann release ich diesen Stand jetzt unter Version 1.0.2. Wird dann morgen mit latest Repo sein.

@baerengraben
Copy link
Owner

Gelöst mit Version 1.0.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants