Ich hätte gerne 80%=80%

[mrinas]

Ich habe mal diese Texte in den Einstellungen angepasst, sowohl für Sofortladen als auch PV Laden. Habt ihr das an anderere Stelle auch noch gesehen? Sonst schicke ich den PR heute so ab.

[hhoefling]

Für den der Zugriff hat.....


Zuerst habe ich es im Nachtladen module gefunden.
Für LP1 in File nachtladen.sh, Zeile 78, aus <= < machen, also das = weg
Also ein Zeichen entfernt.

Jetzt bekomme ich 80% für 80%.
Was bei 100% passiert interessiert mich erst mal nicht, denn das macht das "Morgenladen" nicht das "Nachladen".

PS: Pull Request spare ich mir auf.

[hhoefling]

Für LP2 in nachtladen.sh Zeile 289 ebenfalls das "=" weg
also

Code: Alles auswählen

if (( soc1 <= diesersocs1 )); then

zu

Code: Alles auswählen

if (( soc1 < diesersocs1 )); then

[openWB]

Macht es nicht Sinn das SoCModul mit teils manueller Berechnung anzupassen (wie bei anderen auch (psa) die während dem Laden nicht liefern?

Würde man das hier ändern macht man im Gegenzug andere Probleme auf (100% sind dann keine 100%).

[Jarry]

Ich würde hier auch davon absehen, das "einfach mal eben wegzufixen" nur weil das eine oder andere Auto keine 1% Schritte liefert.
ICH hätte schon was dagegen wenn plötzlich meine Autos nicht mehr bei 100% zum balancen kommen....
Also bitte NICHT einfach aus > ein = machen.
Das hat so wie es ist durchaus seinen Sinn und Daseinsberechtigung...

[mrinas]

PR für die Anpassung des Textes ist eingestellt. Somit ist zumindest sichergestellt dass Konfiguration, Beschreibungstext und Verhalten im Einklang sind.

Ich denke auch dass die Regellogik so wie sie heute ist 'nicht einfach so' entstanden ist, sondern sich alle Beteiligten durchaus Gedanken dazu gemacht haben.

[tensing2]

Und wie wäre es einfach mit?

Code: Alles auswählen

if (( (( soc1 < diesersocs1 )) || (( diesersoc1 == 100 )) )) ; then

[hhoefling]

Ja mit Sicherheit besser,
aber Ich wollte es einfach halten

[hhoefling]

Langsam versteh ich warum openWB so ist wie es ist.....
EInfach ist einfach zu einfach.

[hhoefling]

@mrinas,
Nutze doch bitte die Gelegenheit dazu in den Setup-Seiten wo du den Text angepasst hast,
auch die Prozent Regler auf Schrittweise 1 zu setzen.
Sonst kann ich nicht 79% einstellen wenn ich 80% haben will.