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 bat #1044

Merged
merged 2 commits into from
Aug 4, 2023
Merged

add solarmax bat #1044

merged 2 commits into from
Aug 4, 2023

Conversation

LKuemmel
Copy link
Contributor

ModbusID in den WR verschoben. Zu den WR gibt es keine Modbus-Doku vom Hersteller. Zu den Speichern dagegen schon, dort ist die ModbusID = 1.
https://openwb.de/forum/viewtopic.php?t=7250

@benderl
Copy link
Contributor

benderl commented Aug 1, 2023

Der Aufbau ist auf den ersten Blick nicht logisch. Wenn die Modbus-ID für den Wechselrichter eingestellt werden kann, sollte das auch für andere Komponenten (z.B. Speicher) möglich sein. Als Standardwert kann die "1" gesetzt und ein Hilfetext ergänzt werden. Die Einstellung jedoch ganz weg zu lassen, finde ich nicht richtig.

Weiterhin sollte bei der Speicher-Komponente unbedingt erwähnt werden, dass zusätzlich ein WR konfiguriert werden muss. Der "MAX.STORAGE Ultimate" ist ein Hybrid-System.

@benderl benderl added the enhancement New feature or request label Aug 4, 2023
benderl
benderl previously requested changes Aug 4, 2023
Copy link
Contributor

@benderl benderl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Siehe Kommentar

@LKuemmel LKuemmel merged commit 36b0bb8 into openWB:master Aug 4, 2023
1 check passed
@LKuemmel LKuemmel deleted the solarmax branch August 4, 2023 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants