Improvement: DPL: remove no-restart option #1775
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
schlimmchen
requested changes
Mar 25, 2025
webapp/src/locales/de.json
Outdated
| @@ -736,8 +736,7 @@ | |||
| "VoltageThresholds": "Batterie Spannungs-Schwellwerte ", | |||
| "VoltageLoadCorrectionInfo": "<b>Hinweis:</b> Wenn Leistung von der Batterie abgegeben wird, bricht ihre Spannung etwas ein. Der Spannungseinbruch skaliert mit dem Entladestrom. Damit Wechselrichter nicht vorzeitig ausgeschaltet werden sobald der Stop-Schwellenwert unterschritten wurde, wird der hier angegebene Korrekturfaktor mit einberechnet, um die Spannung zu errechnen, die der Akku in Ruhe hätte. Korrigierte Spannung = DC Spannung + (Aktuelle Leistung (W) * Korrekturfaktor).", | |||
| "InverterRestartHour": "Uhrzeit für automatischen Wechselrichterneustart", | |||
| "InverterRestartDisabled": "Keinen automatischen Neustart planen", | |||
| "InverterRestartHint": "Der Tagesertrag des Wechselrichters wird beim Einsatz an Solarmodulen nachts zurückgesetzt, wenn sich der Wechselrichter mangels Licht abschaltet. Um den Tageserstrag zurückzusetzen obwohl der Wechselrichter dauerhaft von einer Batterie gespeist wird, kann der Inverter täglich zur gewünschten Uhrzeit automatisch neu gestartet werden." | |||
| "InverterRestartHint": "Der Wechselrichter wird täglich zur gewünschten Zeit automatisch neu gestartet. Dies ist notwendig, da sich die Wechselrichter seltsam verhalten, wenn sie nicht neu gestartet oder längere Zeit (Tage/Wochen/Monate) ohne Strom sind, und um den täglichen Ertrag zurückzusetzen." | |||
There was a problem hiding this comment.
"da sich die Wechselrichter seltsam verhalten" => "da sich die Wechselrichter möglicherweise unerwartet verhalten"
ohne Strom? => "längere Zeit (Tage/Wochen/Monate) am Stück in Betrieb sind"
webapp/src/locales/fr.json
Outdated
| @@ -739,8 +739,7 @@ | |||
| "VoltageThresholds": "Battery Voltage Thresholds", | |||
| "VoltageLoadCorrectionInfo": "<b>Hint:</b> When the battery is discharged, its voltage drops. The voltage drop scales with the discharge current. In order to not stop inverters too early (stop threshold), this load correction factor can be specified to calculate the battery voltage if it was idle. Corrected voltage = DC Voltage + (Current power * correction factor).", | |||
| "InverterRestartHour": "Automatic Inverter Restart Time", | |||
| "InverterRestartDisabled": "Do not execute automatic restart", | |||
| "InverterRestartHint": "When used with solar modules, the inverter's daily yield is usually reset at night when the inverter turns off due to lack of light. To reset the daily yield even though the inverter is continuously powered by a battery, the inverter can be automatically restarted daily at the desired time." | |||
| "InverterRestartHint": "The inverter(s) will be automatically restarted daily at the desired time. This is necessary because the inverter(s( behave strangely if not restarted or without power for days/weeks/months and to reset the daily yield." | |||
There was a problem hiding this comment.
Let's not keep untranslated tokens in locales. Remove this. The English token will be used as fallback gracefully anyways.
webapp/src/locales/en.json
Outdated
| @@ -738,8 +738,7 @@ | |||
| "VoltageThresholds": "Battery Voltage Thresholds", | |||
| "VoltageLoadCorrectionInfo": "<b>Hint:</b> When the battery is discharged, its voltage drops. The voltage drop scales with the discharge current. In order to not stop inverters too early (stop threshold), this load correction factor can be specified to calculate the battery voltage if it was idle. Corrected voltage = DC Voltage + (Current power * correction factor).", | |||
| "InverterRestartHour": "Automatic Inverter Restart Time", | |||
| "InverterRestartDisabled": "Do not execute automatic restart", | |||
| "InverterRestartHint": "When used with solar modules, the inverter's daily yield is usually reset at night when the inverter turns off due to lack of light. To reset the daily yield even though the inverter is continuously powered by a battery, the inverter can be automatically restarted daily at the desired time." | |||
| "InverterRestartHint": "The inverter(s) will be automatically restarted daily at the desired time. This is necessary because the inverter(s( behave strangely if not restarted or without power for days/weeks/months and to reset the daily yield." | |||
There was a problem hiding this comment.
"This is necessary because the inverter(s( behave strangely if not restarted or without power for days/weeks/months and to reset the daily yield." => "This is necessary because the inverter may behave unexpectedly if not restarted and powered continuously for days/weeks/months. A restart is also required to reset the daily yield."
There was a problem hiding this comment.
Much better! thanks
To ensure that inverters connected to batteries and smart-buffers work fine they need to be restarted regularly, with this change we remove the option to disable that behaviour, which was the default in the UI and in the configuration.
AndreasBoehm
force-pushed
the
andreasboehm/chore/remove-no-restart-option
branch
from
d2b9b08 to
ebadabd
Compare
schlimmchen
approved these changes
Mar 25, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To ensure that inverters connected to batteries and smart-buffers work fine they need to be restarted regularly, with this change we remove the option to disable that behaviour, which was the default in the UI and in the configuration.