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

HMS-1600 mit Buchstabe in der Seriennummer #1516

Closed
1 task
maikal80 opened this issue Mar 20, 2024 · 20 comments
Closed
1 task

HMS-1600 mit Buchstabe in der Seriennummer #1516

maikal80 opened this issue Mar 20, 2024 · 20 comments
Assignees
Labels
enhancement New feature or request fixed dev fixed

Comments

@maikal80
Copy link

Platform

ESP32

Assembly

I did the assebly by myself

nRF24L01+ Module

No response

Antenna

external antenna

Power Stabilization

nothing

Connection picture

  • I will attach/upload an Image of my wiring

Version

0.8.36

Github Hash

Build & Flash Method

AhoyDTU Webinstaller

Setup

AhoyDTU mit der aktuellsten Firmware.

Debug Serial Log output

None

Error description

Ich habe einen Hoymiles HMS-1600. In der Seriennummer ist an der 5.Stelle ein Buchstabe. Mir wird dann auf der AhoyDTU Weboberfläche der Eintrag rot markiert. Eine Kommunikation mit dem Wechselrichter kommt nicht zustande.
Was kann ich machen oder ist es ein Grundsätzliches Problem?
IMG_6222
IMG_6221

@maikal80 maikal80 added the bug Something isn't working label Mar 20, 2024
@tictrick tictrick added enhancement New feature or request and removed bug Something isn't working labels Mar 20, 2024
@lumapu
Copy link
Owner

lumapu commented Mar 21, 2024

das sollte kein Problem darstellen - kommt nur zum ersten mal vor. Ich werde das fixen. Vielen Dank für den Hinweis

@beegee3
Copy link
Contributor

beegee3 commented Mar 22, 2024

A steht für das Baujahr 2024. 1 = 2015, 2 = 2016, ..., F = 2029. Spätestens 2030 funktioniert diese Zählung also nicht mehr, mal sehen, was HM dann macht. Nach dem Baujahr sollte eigentlich die Produktionswoche (01 bis 52) angegeben sein. Hier steht aber 00; merkwürdig 🤔.
@lumapu: ist evtl. nicht ganz so einfach, zumindest kann es bei JSON Ein- und Ausgaben schnell inkompatibel zu den vorherigen Versionen werden. Hab' die Seriennummer spaßeshalber mal auf String umgestellt und mir prompt beim Einlesen einer alten Datei den ESP abgeschossen. Daher mach ich besser keinen Code-Vorschlag 😄

@lumapu
Copy link
Owner

lumapu commented Mar 22, 2024

schade, wo ich deinen Namen gelesen habe dachte ich gleich "ui schon gelöst". Ich probiere mich auch mal, danke für den Hinweis mit dem JSON sowohl als Setting als auch Export

@lumapu
Copy link
Owner

lumapu commented Mar 22, 2024

ich glaube, dass der Ex- und Import das einzige Problem ist, das bekommen wir auch in den Griff

lumapu added a commit that referenced this issue Mar 22, 2024
* add support for newest generation of inverters with A-F in their serial number
* fix NTP and sunrise / sunset
* set default coordinates to the mid of Germany #1516
@lumapu lumapu added the fixed dev fixed label Mar 22, 2024
@lumapu
Copy link
Owner

lumapu commented Mar 22, 2024

@maikal80 bitte testen 😉

@maikal80
Copy link
Author

Mit der neuen Firmware 0.8.97 funktioniert es ohne Probleme. Die Seriennummer mit Buchstabe wurde akzeptiert. Vielen Dank.

@lumapu
Copy link
Owner

lumapu commented Mar 23, 2024

Danke fürs Feedback, die Kommunikation geht auch?

@beegee3
Copy link
Contributor

beegee3 commented Mar 23, 2024

😆 einfach in JSON die Serial HEX Ausgabe als "note". 👍👍👍

@theunknownstarman
Copy link

Super Sache mit der Änderung. GENAU das Problem habe ich bei dem diese Woche gelieferten WR auch :)

@lumapu
Copy link
Owner

lumapu commented Mar 23, 2024

😆 einfach in JSON die Serial HEX Ausgabe als "note". 👍👍👍

ja, das war smart oder? 😇
Blöd mit beim Import einer solchen JSON wird dieses unnötige Feld dann mitgeschleppt. Man könnte das noch beim Import eliminieren ...

@beegee3
Copy link
Contributor

beegee3 commented Mar 24, 2024

naja, man könnte natürlich auf dieses Feld verzichten. Jeder, der sich die JSON Serial ansieht, kann bei Bedarf selbst die Zahl in HEX umwandeln (z.B. mit dem Windows Rechner in der Programmierer Ansicht). Oder du gibst in der Web Anzeige neben der HEX Serial den uint64_t Wert aus. Die helper.cpp Serial2u64 Funktion wird ja jetzt nur noch für die Umrechnung der 'alten' JSON Serial gebraucht. Vielleicht baust du noch ein, das bei 'alter' Serial automatisch die 'neue' gespeichert wird. Dann kann (spätestens nach einer neuen Release Version) die Serial2u64 entfernt werden.
(off topic zur Webanzeige: kannst du bei Gelegenheit die deutsche Übersetzung der token "BTN_INV_ADD" und "TAB_INPUTS" auf Umlaute umstellen? Und die Texte von "BTN_NEXT", "TEST_CONNECTION" und "ERR_INDEX" enthalten "ü" statt "ü", ist das OK?)

@lumapu
Copy link
Owner

lumapu commented Mar 24, 2024

weiß nicht, ob das mit der Übersetzung so geht, da diese Buttons in C definiert werden und nicht in der HTML Seite. Hatte da anfänglich Probleme und keine weitere Muße tiefer reinzuschauen.
Die Anregungen bzgl. Export finde ich gut, ich denke den HEX Export braucht man, wenn man es manuell wieder eingeben will und die JSON als Vorlage nimmt - leider in letzter Zeit häufiger ein use-case ...

@beegee3
Copy link
Contributor

beegee3 commented Mar 25, 2024

bei "TAB_INPUTS" ist es einfach "Eingänge". Bei "BTN_INV_ADD" funktioniert ü nicht, da auf Großbuchstaben gewandelt wird. Aber Unicode geht immer: "Wechselrichter hinzuf\u00FCgen".

@lumapu
Copy link
Owner

lumapu commented Mar 26, 2024

danke 😉

lumapu added a commit that referenced this issue Mar 26, 2024
* fix captions in `/history #1532
* fix get NTP time #1529 #1530
* fix translation #1516
@MiniOh
Copy link

MiniOh commented Jun 16, 2024

Hallo @lumapu
Ich habe auch einen neuen HMS-1600-4T mit 2 Buchstaben in der SN.
Dazu habe ich von der letzten stable auf die 0.8.126 geupdatet.
Danach lässt sich die SN hinzufügen, aber bisher keine Komunikation mit dem Inverter.
Bisher mit 863, 865 und 868MHz getestet.

Inverter "ist gerade nicht verfügbar"

Ideen?

@MiniOh
Copy link

MiniOh commented Jun 17, 2024

@lumapu

Ich nehme meine Frage zurück.
Gestern ne Stunde getestet, ohne Erfolg.
Heute morgen funktionierte die Kommunikation mit dem HMS-1600 wie erwartet.

Danke dennoch.

@lumapu
Copy link
Owner

lumapu commented Jul 11, 2024

wollte schreiben: warte mal auf den nächsten Morgen, aber du hast es ja selbst gemerkt 😅

@stefan123t
Copy link
Collaborator

Jahreszahl jetzt in Hex 😄

@lumapu schau Dir bitte mal analog dazu die beiden HERF Inverter issues / topics bei OpenDTU an:

Aber auch dafür muß die Eingaberoutine aufgebohrt / angepaßt werden, am besten geht das vermutlich auch bei AhoyDTU im Browser / JavaScript.

@dbMerkel
Copy link

Hallo, wollte auch gerade den HMS 1600 4T mit AhoyDTU verbinden. Leider kann ich die Serialnummer nicht eingeben.
1164A00CA84A.
Habe die Version ==> You are using the current stable release: 0.8.83

Wann kann ich mit einem Update rechnen, oder mache ich etwas falsch?

@stefan123t
Copy link
Collaborator

@dbMerkel Du bist auf der Stable, der Entwicklungsbranch development03 ist derzeit bei v0.8.129

Alternativ kannst Du ein Backup Deiner Settings machen und die Serial ID in der json Datei anpassen. Evtl wird die ID beim Hochladen der Config nicht geprüft. Sozusagen ein Workaround.

@lumapu lumapu closed this as completed Aug 16, 2024
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

8 participants