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

Unexpected error in worker for App pv_opt - charge slots not optimal #132

Closed
solarjones opened this issue Feb 22, 2024 · 18 comments
Closed

Comments

@solarjones
Copy link

solarjones commented Feb 22, 2024

PV Opt v3.8.12

Think there's still a bug lurking somewhere as the optimised charge plan wasn't optimal (maybe it was but represented differently compared to the one below?):
image

I reloaded OE integration, tried to nudge PV_OPT by toggling between readonly and normal modes but the plan remained the same. Next I restarted AppDaemon which aligned to cheapest slots.
image

I can see some errors, but that was from last night. Perhaps it caused the above issue?

pv_opt.log
main.log
error.log

Thanks.

@fboundy
Copy link
Owner

fboundy commented Feb 22, 2024

Do you have the pv_opt.log from 22:30 when the error occurred?

AppDaemon failed to return recent SOC data which is what caused the error in error.log and I suspect this may have stopped it loading correct tariffs after that.

@solarjones
Copy link
Author

1pv_opt.log

@solarjones
Copy link
Author

solarjones commented Feb 23, 2024

v3.8.14 downloaded at about 15:30, restarted AP.

image

Doesn't look like it picked up the OE slots correctly. No errors.

Restarted AD.

image

Looks ok.

However, (and this isn't specific to this version as I noticed this morning) I don't think the PV forecast has been taken into consideration, 26 kWh tomorrow! :) My projected consumption is lowish.

Also toggled between consumption history and fixed. It changed the charge plan by always resulted with 100% SOC around 6am.

image image

pv_opt.log
main.log
error.log

Thanks

@fboundy
Copy link
Owner

fboundy commented Feb 23, 2024

In 3.9.0 I've added a feature to compare the pv_opt tariffs with those from the Octopus Energy automation (so long as you have octopus_auto set to true). At the moment all it does is flag if they are different and also populate a sensor called sensor.pvopt_tariff_import_OK with True or `False'. This should at least help identify when there is a problem and, hopefully, eventually track down the source.

@solarjones
Copy link
Author

In 3.9.0 I've added a feature to compare the pv_opt tariffs with those from the Octopus Energy automation (so long as you have octopus_auto set to true). At the moment all it does is flag if they are different and also populate a sensor called sensor.pvopt_tariff_import_OK with True or `False'. This should at least help identify when there is a problem and, hopefully, eventually track down the source.

Ok, I'll try 3.9.0. Is this correct?
image

What do you think about the PV forecast & SOC with the charge plan?

Thanks

@fboundy
Copy link
Owner

fboundy commented Feb 23, 2024

In 3.9.0 I've added a feature to compare the pv_opt tariffs with those from the Octopus Energy automation (so long as you have octopus_auto set to true). At the moment all it does is flag if they are different and also populate a sensor called sensor.pvopt_tariff_import_OK with True or `False'. This should at least help identify when there is a problem and, hopefully, eventually track down the source.

Ok, I'll try 3.9.0. Is this correct? image

What do you think about the PV forecast & SOC with the charge plan?

Thanks

You should be able to comment it all out as octopus_auto: true is the default

@fboundy
Copy link
Owner

fboundy commented Feb 23, 2024

What do you think about the PV forecast & SOC with the charge plan?
Thanks

I think that's correct. Mine is similar. There are some marginal benefits from charging at <15p and exporting at 15p. My cost saving for tomorrow is only 20p though. If you increase the two thresholds (per slot and per pass) it will probably reduce the amount of forced charging.

@fboundy
Copy link
Owner

fboundy commented Feb 23, 2024

You can see it is reading the Solcast OK because the "Initial SOC" curve reaches 100%.

You can also check that it is reading the forecast correctly by switching to the Solcast_p10 forecast in which case it should charge more and not get to such a high SOC:

image

@solarjones
Copy link
Author

I'd been using the Solcast default (p50?), so have now switched to P10.

image

Thanks.

@fboundy
Copy link
Owner

fboundy commented Feb 23, 2024

v3.8.14 downloaded at about 15:30, restarted AP.

Doesn't look like it picked up the OE slots correctly. No errors.

One thing I don't understand in your comparison is that Octopus don't release the next day's tariffs until ~16.00 so when I look at the log file for 15.30 PV_Opt doesn't have them - it is using a guesstimate based on the published wholesale day-ahead prices which is only hourly so it won't be 100% correct. Once the Octopus prices are available (after 16.30) it all looks OK.

@fboundy
Copy link
Owner

fboundy commented Feb 23, 2024

I'd been using the Solcast default (p50?), so have now switched to P10.
Thanks.

This will be more conservative in that it's likely you will charge your battery more than you needed to but you are very unlikely to charge too little.

@fboundy
Copy link
Owner

fboundy commented Feb 23, 2024

3.9.1 tidies up a few things related to these marginal charging slots and makes it less likely to charge unnecessarily,

@solarjones
Copy link
Author

3.9.2
I've got some more errors and just checked the charging plan, which doesn't look like its updated or got he OE slots correct.

image

Restarting AppD, and the charging plan looks correct now.
image

Logs:
pv_opt.log
main.log
error.log
1pv_opt.log

Just noticed you have version 3.9.3, I'll update to this. Does this have any changes in this area?

Thanks.

@fboundy
Copy link
Owner

fboundy commented Feb 25, 2024 via email

@solarjones
Copy link
Author

v3.9.3
Unexpected error in worker for App pv_opt:

pv_opt.log
main.log
error.log

@fboundy
Copy link
Owner

fboundy commented Feb 26, 2024

Added a check on this in 3.9.4

@fboundy
Copy link
Owner

fboundy commented Feb 28, 2024

Think I may have finally found the root cause! Please check out 3.9.5

@solarjones
Copy link
Author

Happy to report a lovely stable version for me, zero errors and good time slots! Truly very grateful for you being so responsive and what a brilliant optimiser!

@fboundy fboundy closed this as completed Mar 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants