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

build 0.7.13 - Export Settings falsches Datum #1040

Closed
rmayergfx opened this issue Jul 22, 2023 · 5 comments
Closed

build 0.7.13 - Export Settings falsches Datum #1040

rmayergfx opened this issue Jul 22, 2023 · 5 comments
Assignees
Labels
enhancement New feature or request fixed dev fixed

Comments

@rmayergfx
Copy link

rmayergfx commented Jul 22, 2023

Funktioniert leider nicht so wie gewünscht:
grafik

Kurz nach einem Reboot erscheint das:
na_v0.7.13_ahoy_setup.json
Erst wenn ich manuell auf NTP Server SYNC NTP gehe
grafik
Wird das Datum und die Uhrzeit mit abgespeichert, leider total verkehrt:

grafik

Und das obwohl im Interface Datum und Uhrzeit richtig sind:
grafik

@lumapu
Was mir hier noch auffällt, ich bekomme bei SYNC NTP nur command executed, kein wirkliches Feedback ob es auch funktioniert hat, das kenne ich anders von anderen Systemen, da kommt dann auch ein Feedback mit Sync OK in grün, oder Sync failed in rot. Wäre praktisch hier gleich ein Feeback zu bekommen.
Man sollte vielleicht im Bereich NTP Server die Uhrzeit und Datum mit einblenden, um zu sehen ob es funktioniert.

@lumapu
Copy link
Owner

lumapu commented Jul 22, 2023

klar, guter Hinweis. Bei meinen Tests hat's funktioniert, ich teste nochmal.

@lumapu lumapu self-assigned this Jul 22, 2023
@lumapu lumapu added the enhancement New feature or request label Jul 22, 2023
@Ollipop030
Copy link

Ollipop030 commented Jul 24, 2023

Guten Morgen,

gerade die 0.7.15 installiert. Beim Export wird leider immer noch nicht die Uhrzeit mit übernommen (na_v0.7.15_ahoy_setup.json). Zeit ist über NTP synchronisiert und läuft auch richtig.

Nachtrag: Grad gesehen, die Zeit läuft NICHT synchron (hatte das Browserfenster nicht aktualisiert). Ahoy läuft hier 2 Stunden in der Vergangenheit, über NTP mit der fritzbox und 0.debian.pool.ntp.org getestet, und über Sync from Browser.
Die Zeit wird wohl jetzt die UTC sein, dann müssen aber Zeitzonen und Sommer/Winterzeit berücksichtigt werden.

@KoLSMS
Copy link

KoLSMS commented Jul 24, 2023

Guten Morgen,

gerade die 0.7.15 installiert. Beim Export wird leider immer noch nicht die Uhrzeit mit übernommen (na_v0.7.15_ahoy_setup.json). Zeit ist über NTP synchronisiert und läuft auch richtig.

Nachtrag: Grad gesehen, die Zeit läuft NICHT synchron (hatte das Browserfenster nicht aktualisiert). Ahoy läuft hier 2 Stunden in der Vergangenheit, über NTP mit der fritzbox und 0.debian.pool.ntp.org getestet, und über Sync from Browser. Die Zeit wird wohl jetzt die UTC sein, dann müssen aber Zeitzonen und Sommer/Winterzeit berücksichtigt werden.

Tatsächlich läuft auch meine ahoy seit 0.7.15 zwei Stunden in der Vergangenheit 😕

Gruß...

@lumapu
Copy link
Owner

lumapu commented Jul 24, 2023

fix kommt heute noch 😉

lumapu added a commit that referenced this issue Jul 24, 2023
* next attempt to fix yield day for multiple inverters #1016
* fix export settings date #1040
* fix time on WebUI (timezone was not observed) #913 #1016
@lumapu lumapu changed the title DEV 0.7.13 -- fix prometheus build -- Export Settings falsches Datum build 0.7.13 - Export Settings falsches Datum Jul 24, 2023
@rmayergfx
Copy link
Author

Mit Version 0.7.16 habe ich aktuell keine Probleme mehr mit dem Export, Datum und Uhrzeit passt nun bei mir.
@lumapu
Dankeschön.
Kommt bei SYNC FTP noch ein Update des Status in einer späteren Release? Hier wäre es sinnvoll auch das aktuelle Datum/Uhrzeit einzublenden.

@lumapu lumapu added the fixed dev fixed label Jul 28, 2023
lumapu added a commit that referenced this issue Jul 28, 2023
* merge PR #1048 version and hash in API, fixes #1045
* fix: no yield day update if yield day reads `0` after inverter reboot (mostly on evening) #848
* try to fix Wifi override #1047
* added information after NTP sync to WebUI #1040
@lumapu lumapu closed this as completed Aug 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request fixed dev fixed
Projects
None yet
Development

No branches or pull requests

4 participants