Skip to content

Conversation

@Maschga
Copy link
Collaborator

@Maschga Maschga commented Oct 5, 2025

Ref #21162: 55e9860
With the latest improvements, it may be possible to reset the interval to 0.1s.

@naltatis naltatis marked this pull request as ready for review October 5, 2025 08:28
@naltatis naltatis enabled auto-merge (squash) October 5, 2025 08:28
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey there - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

auto-merge was automatically disabled October 5, 2025 08:35

Head branch was pushed to by a user without write access

@Maschga
Copy link
Collaborator Author

Maschga commented Oct 5, 2025

Errors occur again at 0.1s.
Let's test 0.25s, as this duration is already used in another test file.

@Maschga Maschga changed the title chore: reset interval to 0.1s chore: reset interval to 0.25s Oct 5, 2025
@Maschga
Copy link
Collaborator Author

Maschga commented Oct 5, 2025

Looks good.

@naltatis naltatis merged commit 1d74952 into evcc-io:master Oct 5, 2025
7 checks passed
@Maschga Maschga deleted the reset-test-interval branch October 5, 2025 09:30
@andig
Copy link
Member

andig commented Oct 7, 2025

Wissen wir, warum 0.1 nicht funktioniert? Was dauert da so lange- die Übertragung oder das Parsen im UI? Für letzteres hätte ich eine Idee.

@Maschga
Copy link
Collaborator Author

Maschga commented Oct 7, 2025

Nein, leider nicht. Ich weiß auch nicht, wie ich das Nachprüfen kann.
Ich kann nur sagen, dass die UI sehr stark gelaggt hat; das Klicken auf einen Toggle hat Sekunden benötigt, um den Zustand zu verändern.

@naltatis
Copy link
Member

naltatis commented Oct 8, 2025

Ich werd das noch mal mit dem Chrome Debugger quantifizieren und schauen wo hier das größte Bottelneck ist.

mfuchs1984 pushed a commit to mfuchs1984/evcc that referenced this pull request Oct 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants