Repository navigation
Conversation
Signed-off-by: Segran <goran@andersson-stehag.se>
Signed-off-by: Segran <goran@andersson-stehag.se>
This file contains hidden or 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
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.
When an EV draws no power or less than requested, FTW can show “The charger has not said why” even when the charger reports a reason. For example, in #1948 Easee reported “limited by dynamic charger current” while the dynamic setting was 12 A, the pilot was 0 A and measured power was 0 W.
Show the charger-reported reason when FTW considers the EV readings current. Keep the existing shortfall, severity and fallback when the explanation is unavailable. This brings the display change from segran2#3 onto master.
All 18 control-feedback tests passed. The full web suite had 925 passes and three chart-palette failures in tests addressed separately by #2008.
Related to #1948 and #1975.