-
Notifications
You must be signed in to change notification settings - Fork 6
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
Rethink inverter control methodology #68
Comments
Some brainstorm notes when it comes to Fronius:
because there are 2 different sets of registers, I guess we'd need to determine do I can see in the SunSpec you're relying on a few registers currently, pretty sure these are the ones defined in https://github.com/longzheng/open-dynamic-export/blob/main/src/inverter/sunspec/index.ts#L273-L286 From <style type="text/css"></style>
There's also these registers, which according to a Home Assistant forum post are used to control battery charge/discharge:
If using HA this varies though? TBC, see the post for details, may be irrelevant here...
The important part:
The register specs for these from <style type="text/css"></style>
|
Another consideration: if the max inverter output and battery charge/discharge can be managed independently at all - does that open up the need for a new limiter? Or maybe adjusting an existing limiter? Where you specify "desired max battery SoC %"? |
I've been a bit under the weather but just wanted to make a quick comment on this (I'll respond to the other points you raised later), yes I think it would be necessary to have something like "desired battery SOC %" as well as "desired import" and "desired export" to take advantage of market pricing. The term "limiters" is kind of misleading in this case, so I'm thinking maybe renaming it to "setpoints" which is more consistent with proper engineering terminology, and have something like
|
I had a similar thought - once you start doing more than "limiting" a better name makes sense :) |
Had another consideration to add here for calculations: what if multiple inverters are in use? Since right now, if multiple inverters are configured, it will split the output power equally so the export kW remains under the limits provided by the grid operator... (?) If one inverter has a battery behind it and another does not (common), does that involve some math like "split the desired output, but then compensate for the battery on the inverter that has a battery"? 🤷♂️ |
Just making some notes about my thoughts and welcome any feedback/input others may have.
Currently the application applies export/generation limits by actively controlling the inverter active power output.
The desired inverter active power output (as a % to nameplate capacity) is calculated by
The actual values are derived by a time-weighted average of the last 5 seconds (default) of samples from controlled inverters and meter.
The active power output is then applied to all controlled inverters, which would effectively reduce the maximum power output to the target solar value.
The pros of this approach is that it makes it possible to control multiple inverters with different capacities with ease, as it effectively pools all inverters as one resource. It also allows controller inverters which either do not have a meter attached or does not support load following export limits (or a mix of inverters with meters and without).
The cons of this approach is that it requires the software to "take over" any native load following export limit logic in the inverters which is usually more responsive
This approach is also going to be more complex as I need to handle battery control. The intention is that the software would manually control the charge and discharge rate of batteries to take advantage of market pricing.
One challenge is how hybrid inverters handles setting "target inverter output". Does it restrict just the solar power output or does it also affect the battery power output?
Alternatively I'm thinking of allowing some inverters (with meter attached and has export limit capabililty) to be controlled by setting export limits, and other inverters (without meter) to be controlled by active power output.
The text was updated successfully, but these errors were encountered: