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

UI: show real battery buffer start value #16582

Merged
merged 1 commit into from
Oct 10, 2024
Merged

Conversation

naltatis
Copy link
Member

@naltatis naltatis commented Oct 9, 2024

fixes #16575

Shows actual bufferStartSoc even if the values is not present in the select range. Useful when setting values through API.

98 %

Bildschirmfoto 2024-10-09 um 16 45 40

available options

Bildschirmfoto 2024-10-09 um 16 45 45

@naltatis naltatis added enhancement New feature or request ux User experience/ interface labels Oct 9, 2024
@VolkerK62
Copy link
Contributor

Supi .... noch schneller geht nicht 😄

@naltatis
Copy link
Member Author

naltatis commented Oct 9, 2024

@andig verstehst du wieso der Test fehlschlägt?

@VolkerK62
Copy link
Contributor

war hier auch. #16546
hilft dir evtl weiter

@andig
Copy link
Member

andig commented Oct 10, 2024

Sehe den Test nicht- flaky?

@andig andig merged commit 184a95d into master Oct 10, 2024
6 checks passed
@andig andig deleted the fix/buffer_start_value branch October 10, 2024 11:21
@naltatis
Copy link
Member Author

@andig ja flaky. Rerun hat geholfen.

Geht um --- FAIL: TestOcpp (17.55s)
https://github.com/evcc-io/evcc/actions/runs/11257466856/job/31301764925#step:5:91

@michiproep
Copy link

Hi @naltatis ,
noch eine Kleinigkeit: Das wording "wenn über 99%" scheint tatsächlich zu stimmen - was aber gleichbedeutend mit "wenn auf 100" ist.
Sollten wir drüber nachdenken aus dem ">" ein ">=" zu machen?

@naltatis
Copy link
Member Author

naltatis commented Oct 11, 2024

Von der Logik ist es bewusst ein >. Ist bei den anderen Optionen ja auch so. Die Denke dahinter ist, dass du bspw. 90% für den Hausverbrauch behalten möchtest und nicht 89%.

"wenn über 99%" und "wenn auf 100%" sind jetzt in der Tat gleich, ich sehe hier aber gerade keine schöne bzw. einfache Lösung für diesen Zustand. Der kann ja auch "nur" per API hergestellt werden.

Hast du nen konkreten Vorschlag?

@michiproep
Copy link

Ja. stimmt schon.
Ich schau mal nächste Woche ob man 97/98/98 einstellen kann und wie es sich verhält

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ux User experience/ interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Batterie "starte automatisch" - kleinere Schritte
4 participants