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

Add SolarMax MAX.STORAGE #4320

Merged
merged 4 commits into from
Sep 8, 2022
Merged

Add SolarMax MAX.STORAGE #4320

merged 4 commits into from
Sep 8, 2022

Conversation

premultiply
Copy link
Member

No description provided.

@premultiply premultiply added the devices Specific device support label Sep 5, 2022
@andig
Copy link
Member

andig commented Sep 5, 2022

@premultiply bitte schau auch nochmal- falls du es testen kannst- ob das uint32s Decoding bei mir überhaupt richtig war :O

@andig
Copy link
Member

andig commented Sep 5, 2022

Sollen wir das Template hier vllt. solarmax-hybrid nennen (Datei und type)? Dann passt es schematisch zu den anderen.

@premultiply
Copy link
Member Author

@premultiply bitte schau auch nochmal- falls du es testen kannst- ob das uint32s Decoding bei mir überhaupt richtig war :O

Richtiges Protokoll gibt es ja leider nirgends aber nach dem Registerscan den jemand hier https://forum.fhem.de/index.php?topic=122419.msg1169983#msg1169983 gemacht hat sieht es nach BE, also uint32 (ohne s) aus.

Dafür in dem Speichersystem sieht es - zumindest nach Anleitung - nach LE aus. Hoffen wir mal dass das auch so stimmt...

@premultiply premultiply changed the title Add Solarmax maxstorage Add SolarMax MAX.STORAGE Sep 5, 2022
@premultiply premultiply self-assigned this Sep 5, 2022
@premultiply premultiply marked this pull request as ready for review September 6, 2022 19:52
@andig
Copy link
Member

andig commented Sep 8, 2022

Wir haben jetzt:

  • ein senergy Tempalte das Senergy und Solarmax enthält
  • ein solarmax Template das nur senergy enthält
  • ein solarmax-maxstorage Template das prinzipiell ein -hybrid WR zu sein scheint

...so richtig schön ist das nicht. Könnten wir das alles unter Solarmax (oder einem anderen Namen) konsolidieren?

Update ich seh grad SolarMax haben wir nicht mehr. Brauchen wir dennoch beide Sorten von Datei/Template-namen?

@premultiply
Copy link
Member Author

Das solarmax-Template wird mit diesem PR gelöscht, da das damit unterstütze Produkt eigentlich nur ein Label von Solarmax hat und ein Senergy-WR ist.

@andig
Copy link
Member

andig commented Sep 8, 2022

…und der Hybrid ist ein echtes Solarmax Prdukt?

@andig
Copy link
Member

andig commented Sep 8, 2022

Und brauchen wir nicht noch ein

covers: [ „solarmax“ ]

Oder hat das eh niemand verwendet? Bestehende Config geht sonst kaputt.

@premultiply
Copy link
Member Author

…und der Hybrid ist ein echtes Solarmax Prdukt?

Sieht zumindest erstmal so aus.

Und brauchen wir nicht noch ein

covers: [ „solarmax“ ]

Oder hat das eh niemand verwendet? Bestehende Config geht sonst kaputt.

Das kam ja erst vor ganz kurzem neu dazu. Daher sehe ich die Gefahr noch nicht so groß.
Notfalls könnte man natürlich dafür auch ein covers einbauen...

@andig andig merged commit 7777603 into master Sep 8, 2022
@andig andig deleted the solarmax-maxstorage branch September 8, 2022 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants