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

Add option "system has normal AC loads"; allowing installer to set that to No. #1328

Open
3 of 8 tasks
mpvader opened this issue Jul 27, 2024 · 4 comments
Open
3 of 8 tasks
Assignees

Comments

@mpvader
Copy link
Contributor

mpvader commented Jul 27, 2024

And then on each UI, that box should either be hidden; or if that is complex, then just show 0. My proposal:

Add the new setting, which obviously defaults to yes.

  • systemcalc: make those paths return 0 when the setting is set that there are no normal AC loads.
  • gui-v1: hide if trivial, otherwise do nothing since it will automatically say 0 which is also an improvement.
  • gui-v2: hide if trivial, otherwise do nothing since it will automatically say 0 which is also an improvement.
  • VRM Portal: . Since current dashboard is EOL, do nothing and it will say 0
  • VRM Widgets: hide it. Effort is worth it.
  • VRM Watch App: hide id. Effort is worth it.
  • HTML5: either way is OK
  • EVCS UI: either way is OK

Background of this request is systems in Austria, and possibly elsewhere in the world, where all loads are behind the ESS system, for being backupped. But part of the installed (AC-coupled) PV is not, due to 1:1 rule. And as its now, it technically works but all those UIs all the time show some measurements for AC loads, while there are no AC loads; what is shown is just timing and measurement differences.

Slack

A visual, but note that this won't happen soon since we're not fixing it this way on VRM, read above.

image

@mpvader
Copy link
Contributor Author

mpvader commented Jul 27, 2024

@izak pls let me know what you think of above approach; required effort in systemcalc; and make sure to do a quick test (!). Ie set those loads to 0 hardcoded in systemcalc, and see if my assumption that that is already an improvement is true.

@izak
Copy link
Collaborator

izak commented Jul 30, 2024

I think we need to integrate this with the existing setting, /Settings/SystemSetup/HasAcOutSystem. This already hides AC-out for systems that only has loads on the input side. Perhaps this needs to be an option list where you can choose between the three options: Only input side, only output side, or both.

A crude "make it zero if it isn't invalid" proof of concept reveals:

Before:
Selection_111

Selection_114

After:

Selection_112

Selection_113

@izak
Copy link
Collaborator

izak commented Jul 30, 2024

Additionally, an experiment where instead of setting the consumption to zero, it is set to invalid:

GUI v1:

Selection_115

VRM:

Selection_116

izak added a commit to victronenergy/dbus-systemcalc-py that referenced this issue Sep 12, 2024
Some systems have a grid meter because they want vector registration on
energy counting, but there aren't any actual loads in front of the
inverter/charger. In such systems, showing the AC input loads makes
little sense and can in fact be noisy, since a small amount of power
is almost always reflected there.

This setting skips the publishing of ConsumptionOnInput, causing
the various GUIs to show that correctly.

victronenergy/venus#1328
izak added a commit to victronenergy/dbus-systemcalc-py that referenced this issue Sep 12, 2024
Make option name more specific to Loads rather than entire system,
since a PV-inverter is allowed before the inverter/charger.

victronenergy/venus#1328
@izak
Copy link
Collaborator

izak commented Sep 13, 2024

Screenshots for GUI-v1 proposal.

New setting for specifying position of AC loads. The case for systems with no grid meter is a bit unclear perhaps. I don't want to hide the setting if there is no grid meter, because that makes it impossible to reach the setting when there is a grid outage.

Options

Selection_149

Selection_152

Some toast text to tell the user what is going on:

Selection_150

Selection_151

Behaviour of overview

In both examples here, there is a PV-inverter on the input and some load on the output.

With setting in the "normal" position:

Selection_153

With setting in the "output only" position:

Selection_154

VRM

With setting in the "normal" position:

Selection_156

With setting in "output only" position:

Selection_155

jhofstee added a commit to victronenergy/meta-victronenergy that referenced this issue Sep 13, 2024
Add option to ignore AC-loads before the energy meter
This is for sites with no AC-loads before the meter, but possible
PV-inverters. This is common where the PV-inverter breaks the 1:1 rule
and cannot go on the output, or where the grid meter is used only to
get vector-registration on the energy accounting.

victronenergy/venus#1328
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants