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

PV Opt discharging battery where it will cause expensive import #197

Closed
jacekowski opened this issue Apr 4, 2024 · 12 comments
Closed

PV Opt discharging battery where it will cause expensive import #197

jacekowski opened this issue Apr 4, 2024 · 12 comments

Comments

@jacekowski
Copy link

Describe the bug
I'm running slightly modified version (i have added sunsynk inverter support - but i doubt this causing the issue).

To Reproduce
Run addon with my configuration (consumption history disabled, 35kWh/day) - everything is visible in the logs anyway.

Expected behavior
No discharge on day like today. Or tomorrow/

Screenshots
image

As you can see, the plan it generates is to discharge (at 15p/kWh) and then it would cause me to import at higher rate (and that is ignoring any efficiency losses). Similiar thing tomorrow.

pv_opt.txt

@fboundy
Copy link
Owner

fboundy commented Apr 4, 2024

Are you sure you would be importing at a higher rate? Agile prices are pretty low today and tomorrow. It's basically saying buy overnight at <10p and sell at 15p which is what's driving logic.

One possible issue is that there is "value" in having a final SOC of 30% vs 10% which it is ignoring because it knows nothing about Saturday yet. At 11:00 tomorrow when it get's the day-ahead forecast it will update and may not discharge as much. You can always turn off discharging or (in 3.14) give zero value to exporting.

@fboundy
Copy link
Owner

fboundy commented Apr 4, 2024

PS - are you able to share your sunsynk mods?

@jacekowski
Copy link
Author

It does need some cleanup and my implementation relies on node-red to do final control, but it is here jacekowski@de90248

Because sunsynk inverter does not have charge/discharge time slots like other inverters, but instead just has a schedule, i'm using node red to check which slot is the current one, and adjusting settings for that slot at the right time.

image

@fboundy
Copy link
Owner

fboundy commented Apr 4, 2024

It does need some cleanup and my implementation relies on node-red to do final control, but it is here jacekowski@de90248

Because sunsynk inverter does not have charge/discharge time slots like other inverters, but instead just has a schedule, i'm using node red to check which slot is the current one, and adjusting settings for that slot at the right time.

Thanks - I'm afraid Node Red is a mystery to me so I will leave you to it!. I am adding support via the Solarsynk Add-on: #129

@jacekowski
Copy link
Author

Unfortunately, solarsynk relies on sunsynk cloud that doesn't work very frequently so i don't consider it a viable option. I've done parts of it node red because i was developing my own solution in node red and then i've found this addon.

I'm going to migrate the whole thing to python in near future and then do a PR.

@fboundy
Copy link
Owner

fboundy commented Apr 4, 2024

I presume you have a hard-wired connection then?

@jacekowski
Copy link
Author

Yes, I'm allergic to cloud, especially if it is not a reliable one.

@fboundy
Copy link
Owner

fboundy commented Apr 4, 2024

Same here. At some point I will add cloud support for Solis inverters but controlling via the cloud is barely supported

@jacekowski
Copy link
Author

What is the intention of this line

start = None

rather than using self.charge_start_datetime further down?

@fboundy
Copy link
Owner

fboundy commented Apr 6, 2024 via email

@jacekowski
Copy link
Author

I'm on 3.14.0, and plan for today/tomorrow is definitely not optimal. Charging started today at 5:30 at 0p/kWh, rather than waiting for later in the day to charge at negative rates.

Current schedule discharges the battery until 18:30, which will then cause import from the grid at 20p/kWh (delaying export by 30 minutes would be the solution here), then full charge is scheduled for 23:30 rather than later at negative rates.

image

@fboundy
Copy link
Owner

fboundy commented Apr 6, 2024 via email

@fboundy fboundy closed this as completed Apr 27, 2024
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