energychain/AnkerSolixE1600

Lauffähigkeit

Opened this issue · 5 comments

dtrywets commented

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

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:

  • Welche node bzw node-red version verwenden sie?
  • Welche Probleme sind aufgetreten?
  • Welche Dateien mussten sie verändern und wohin gehend?
dtrywets commented

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?

  • 👍1
mrohmer commented

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

  • Refactored the api file to human readable js/none transpiled ts
  • Removed the usage of node-fetch as node brings it's own fetch implementation since v17
  • Removed unused dependencies (in fact all dependencies)
  • Add JSDoc comments to have some sort of type safety/hints
  • Created sub directories to separate the concerns between the lib & node-red related code
  • Removed the json files from a local execution 🙈

Improvements

  • Switch to getScenInfo for data retrieval
  • Split the nodes to get infos about the system & updating the schedule
  • Add status info to the nodes
  • Use global configuration node to manage credentials across multiple nodes
  • Make tokens survive deployments (to make more tinkering possible before hitting the login limit)
dtrywets commented
  • 👍1
mrohmer commented

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