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 description and battery control for E3/DC RSCP template #13626

Merged
merged 3 commits into from
May 2, 2024

Conversation

docolli
Copy link
Contributor

@docolli docolli commented Apr 28, 2024

I've added some more description for setup, the battery-control flag and a hint how to get a discharge lock.

We should add some hint about the number that should be used for battery parameter. This is normally "0" at single battery storage setups and is mandatory to get battery capacity from the system.

de: Index des Batteriespeichers im System (bei Systemen mit 1 Batteriespeicher ist dieser normalerweise 0)
en: Index of battery storage in the system (for systems with 1 battery storage this is usually 0)

@andig
Copy link
Member

andig commented Apr 28, 2024

What means „normally 0“? Can you have a single battery that is not at index 0? Also, 0 doesn‘t need be configured, its the default.

@andig andig added the devices Specific device support label Apr 28, 2024
@docolli
Copy link
Contributor Author

docolli commented Apr 28, 2024

What means „normally 0“? Can you have a single battery that is not at index 0? Also, 0 doesn‘t need be configured, its the default.

I don't know, but I think they will always start counting at "0". But if "0" is default in code, then it is not mandatory, of course. Missed that in my testing. 😉

@andig
Copy link
Member

andig commented Apr 29, 2024

Wie wäre es- kurz und knapp- mit:

Username and password are identical to Web Portal or My E3/DC App access. Key and RSCP-Password must be set in the E3/DC System at Personalize/User Profile.
NOTE: Active battery control will override Smart-Power/Operating Range settings.

@docolli
Copy link
Contributor Author

docolli commented Apr 29, 2024

Guter Vorschlag!

@andig
Copy link
Member

andig commented May 1, 2024

Magst Du's ändern? Dann kanns rein!

@andig andig merged commit 81d677b into evcc-io:master May 2, 2024
7 checks passed
@andig
Copy link
Member

andig commented May 2, 2024

Top, danke!

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