-
Notifications
You must be signed in to change notification settings - Fork 119
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
divert_PV_ratio: adjustable requirement on available PV power before starting charge or incrementing current #117
Conversation
treepleks
commented
Sep 21, 2020
•
edited
Loading
edited
…s this can be done using a sufficient fraction of PV power (possibly less than 100%). Creates a divert_PV_ratio (between 0 and 1.0) that gives the required fraction of PV power that must be available to decide to start or increment the charge current. Also GRID_IE_RESERVE_POWER is set to 0.0.
|
It would be really handy if you could use the |
|
FYI this is what I generated from the current divert module |
|
Thanks a lot Jeremy for the feedback. I must confess I sort of hoped the Open EVSE team would be ready to do the simulation :-). I installed the full ESP32 compilation kit just to add this improvement, I can produce a firmware but am still reluctant to flash it on the wall box before it has been tested. |
|
Ok, looking to the code, I understand divert_sim is just a wrapper around divert. I added the required
in divert_simp.cpp. Also added MicroDebug.h and StreamSpy.h. But make still complains at linking: I must be missing something. Any hint here? |
|
Oh, I will take a look at it, that is s related to some resent changes I made... |
|
It's Ok. I could achieve it with a few changes: RapiSender.cpp: divert_sim.cpp And it worked. Simulations show it works as expected. Attached simulations for various values of PV_divert_ratio (GRID_IE_RESERVE_POWER = 0.0 except last where it is 100 as in the vanilla version)
I would suggest that when divert_PV_ratio >1, the decimal part should go to GRID_IE_RESERVE_POWER. This means that with divert_PV_ratio =
Just tell me what you think of this. |
|
Thanks for doing those, I must admit I have read the description several times and I am not entirely sure I get the logic. The graphs certainly look better from the point of view of not starting/stopping the charge so much, which I guess is the point, to sacrifice using grid power to make sure all solar is used. I think I would prefer the default to be 1.1 do better match the existing logic, but with that change I would be happy for it to be merged. @glynhudson what do you think? |
|
Adding Have you tested this on your ESP32? |
|
Hello Glyn, Yes. After I sent the simulations to Jeremy I actually flashed my Open EVSE with it and it works just fine. It changes a lot for me. I have 3kW panels, with work at home, we easily use 900W for various items (up to 4 computers are on) and on a sunny day it leaves around 1500W export and the open EVSE wallbox... would never start. Now, setting the ratio to 0.5 (which is safe, above my supposedly optimal value of 0.39) my car has charged several hours today, at minimal marginal cost. Just great :-) |
|
@treepleks could something have gone wrong with your PR? I've just checked and I can't see a PV ratio option running the latest version: |
|
Ah, got it. My mistake solar PV ratio is only displayed when |

