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.

Sign up for GitHub

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

Jump to bottom

L2 shows dashes while L1 and L3 don't #1326

Closed
mpvader opened this issue Jul 14, 2024 · 2 comments · Fixed by #1332
Closed

L2 shows dashes while L1 and L3 don't #1326

mpvader opened this issue Jul 14, 2024 · 2 comments · Fixed by #1332
Assignees
@DanielMcInnes
Labels
bug Something isn't working medium_prio
Milestone
Version 1.0.0

Comments

@mpvader
Copy link
Contributor

mpvader commented Jul 14, 2024

On this system, which is a EM24 energy meter, three phase, plus a single RS on L3, gui-v2 shows -- on L2 while there is data for it.

Note that the system shows dashes for both L1 and L2 in case of a grid outage, which happens often there.

Discussion with user of that system: https://victrondevelopment.slack.com/archives/C04RYU0UMDE/p1717966505212219

I'm assuming this is a small typo or bug, hence v1.0.0 milestone. If it gets all complicated then we can move this to a later milestone.

image
@mpvader mpvader added bug Something isn't working medium_prio labels Jul 14, 2024
@mpvader mpvader added this to the Version 1.0.0 milestone Jul 14, 2024
@DanielMcInnes DanielMcInnes self-assigned this Jul 14, 2024
@DanielMcInnes
Copy link
Contributor

DanielMcInnes commented Jul 14, 2024 •
edited
Loading

Hi @mpvader , I don't have access to this installation, could you add [email protected] please? Adding 'blocked' label for the time being.

@DanielMcInnes DanielMcInnes added the blocked Waiting for venus-platform or other support label Jul 14, 2024
@mpvader
Copy link
Contributor Author

mpvader commented Jul 15, 2024

Hi @DanielMcInnes , yes. Done

@DanielMcInnes DanielMcInnes removed the blocked Waiting for venus-platform or other support label Jul 15, 2024
DanielMcInnes added a commit that referenced this issue Jul 15, 2024
@DanielMcInnes
Update current & power in AcInputPhaseModel when the value is constan…
4797f20 
…tly 0

Prior to this change, if the first valid phase value was 0, and it remained at
0, we would display '--' in the ThreePhaseDisplay, even though we had a valid
value of '0'.

Fixes #1326
@DanielMcInnes DanielMcInnes mentioned this issue Jul 15, 2024
Merged
@DanielMcInnes DanielMcInnes linked a pull request Jul 15, 2024 that will close this issue
Update current & power in AcInputPhaseModel when the value is constan… #1332
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DanielMcInnes DanielMcInnes

Labels
bug Something isn't working medium_prio
Projects
None yet
Milestone
Version 1.0.0
Development

Successfully merging a pull request may close this issue.

Update current & power in AcInputPhaseModel when the value is constan… victronenergy/gui-v2
2 participants
@mpvader @DanielMcInnes