energychain / AnkerSolixE1600

AnkerSolixE1600 Node.JS
Apache License 2.0
8 stars 3 forks source link

Lauffähigkeit #4

Open dtrywets opened 2 months ago

dtrywets commented 2 months ago

Hallo zusammen, danke für dieses Projekt.

Ich bin nun schon seit ein paar Tagen daran, die NodeRED Node zum Laufen zu bekommen und scheitere immer wieder. Nachdem verschiedene Anpassungen in den Dateien nötig waren um überhaupt eine Response zu bekommen, scheitere ich nun immer wieder.

Würde mich über eine Info freuen :-)

LG

mrohmer commented 2 months ago

hi,

nachdem ich gerade selbst an einem fork dieser lib arbeite, kann ich gerne versuchen ein bisschen mit zu helfen dein Problem zu lösen. 😉

Meinen fork habe ich hauptsächlich erstellt, da der genutzte API-Endpoint nicht mehr die notwendigen Daten meiner Solix bereit hielt (SoC) und ich generell einen starken Drang hatte den Code auf zu räumen.

Allerdings wäre ein Mindestmaß an Informationen ganz nett:

dtrywets commented 2 months ago

Hi, ich habe inzwischen ebenfalls selbst geforked und mir einen passenden NodeRED Flow gebaut. Meinen Fork findest du hier: https://github.com/dtrywets/AnkerSolixE1600-APIServices

@mrohmer: Welche Anpassungen hast du denn an deinem Fork gemacht und wie nutzt du die API?

mrohmer commented 2 months ago

Meine Änderungen haben sich in dieser lib abgespielt weil für mich eine Nnative Einbindung schlicht besser handlebar ist als noch ein weiterer Service neben dran.

Die Hauptänderung, die ich gebraucht habe, war eine Änderung des Endpoints, da auf Seite Ankers anscheinend eine Änderung an den ausgespielten Daten statt gefunden hat. Ich habe schlicht über den site homepage Endpoint keine Infos zu meiner Solarbank mehr bekommen (solarbank_list war leer).

Da ich dann schon dran war, hab ich den Code erst ein Mal grundlegend aufgeräumt und mir komplett neue Nodes aufgebaut, die meinem Verständnis nach die Sache besser abdecken.

Hatte kurzzeitig einen Draft PR offen, daher hier mal ein kurzer Auszug mit den zusammengefassten Änderungen:

Cleanup

Improvements

dtrywets commented 2 months ago

Dein Ansatz klingt sehr nüchtern und super strukturiert. Ist das in deinen Fork gepushed?


Von: Matthias Rohmer @.> Gesendet: Montag, 9. September 2024 12:12 An: energychain/AnkerSolixE1600 @.> Cc: Sebastian Ladwein @.>; Author @.> Betreff: Re: [energychain/AnkerSolixE1600] Lauffähigkeit (Issue #4)

Meine Änderungen haben sich in dieser lib abgespielt weil für mich eine Nnative Einbindung schlicht besser handlebar ist als noch ein weiterer Service neben dran.

Die Hauptänderung, die ich gebraucht habe, war eine Änderung des Endpoints, da auf Seite Ankers anscheinend eine Änderung an den ausgespielten Daten statt gefunden hat. Ich habe schlicht über den site homepage Endpoint keine Infos zu meiner Solarbank mehr bekommen (solarbank_list war leer).

Da ich dann schon dran war, hab ich den Code erst ein Mal grundlegend aufgeräumt und mir komplett neue Nodes aufgebaut, die meinem Verständnis nach die Sache besser abdecken.

Hatte kurzzeitig einen Draft PR offen, daher hier mal ein kurzer Auszug mit den zusammengefassten Änderungen:

Cleanup

Improvements

— Reply to this email directly, view it on GitHubhttps://github.com/energychain/AnkerSolixE1600/issues/4#issuecomment-2337700526, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATI2EDQEWU2G6R5E7HI2MM3ZVVYAVAVCNFSM6AAAAABNG4TOY6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZXG4YDANJSGY. You are receiving this because you authored the thread.Message ID: @.***>

mrohmer commented 2 months ago

Danke, aber gerade strukturiert würde ich es nicht nennen. Aber es sollte alles so umgesetzt sein 👍