mirror of
https://github.com/OpenEPaperLink/OpenEPaperLink.git
synced 2026-03-21 00:04:28 +01:00
[GH-ISSUE #525] New dayahead refresh stops around 13:45 every day #1992
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @cthulu on GitHub (Nov 16, 2025).
Original GitHub issue: https://github.com/OpenEPaperLink/OpenEPaperLink/issues/525
Hi, based on suggestion from @benv666 I am raising this as a new issue. I've forked the current
masterand tagged it to create my own release using GH Actions. This is thus no different to what is inmasterright now.After installing this on my Yellow AP I was able to get the dayahead pricing showing with 15 mins intervals on my tags. But strangely, this refresh seems to stop every day around 13:45 or 14:15 and I have no idea why this happens. Force refreshes or AP reboot doesn't fix it until the new day starts.
I am not sure why this is happening. The only suspicious fact is that new price data is usually available around 13-14:00 but that didn't cause issues in the past.
Originally posted by @benv666 in #517
@cthulu commented on GitHub (Nov 17, 2025):
This is also present in the latest released firmware 2.84. I have re-configured a tag, made it 'dayahead prices' and it fails to refresh after 14:15. Also, all other tags stay frozen at 14:00 and the AP restarts frequently.
@cthulu commented on GitHub (Nov 17, 2025):
This is on the serial if I force-refresh the dayahead tag after 14:00:
@cthulu commented on GitHub (Nov 18, 2025):
Same issue if I disabled the BLE functionality. Can it be that the amount of data available post 14:00 is too much to fit?
@cthulu commented on GitHub (Nov 19, 2025):
Okay I think the problem is quite tricky:
Display intervalandCheap block hours)When I've opened the Web interface in an incognito window on the desktop, I saw the new config options and I've set them to some value. After saving, the refresh started working.
So I think there was some kind of corruption of the settings - they may have been incompletely saved in the Tag DB, leading to some memory curruption downstream?
@nlimper commented on GitHub (Nov 19, 2025):
That's a great find, thanks!
IMHO, it would be better for the AP to fail gracefully without crashing when the parameter is not set (by using some default value). More people will experience the same situation, causing it to be a repeating support question.
@benv666 if you got a chance, it would be great to have it fixed.
@cthulu commented on GitHub (Nov 19, 2025):
I am not 100% absolutely sure the missing setting was the actual issue, but after setting it things have started to work
¯\_(ツ)_/¯@benv666 commented on GitHub (Nov 19, 2025):
@cthulu Great find indeed, that's something to go on.
@nlimper Will try to make a fix coming weekend, won't have much time before then.
Thought I checked for the existance of these config values and had defaults, but might have missed one - to be continued.
@benv666 commented on GitHub (Nov 23, 2025):
I've been trying to come up with a few scenarios that might have triggered the stack overflow posted by @cthulu:
There were 3 settings added by my code,
updatefreq,interval, andcheapblockupdatefreqis unlikely to be your issue, it would only manifest in weird update intervals.cheapblockI'm also combing over, but this one has bound checks in place, so probably not the issue.The ~14:00 issue indicates having a lot of samples, so most likely the averaging code had to deal with an interesting value for the
intervalsetting is my guess, especially if parsing the absent value led to a high number in targetIntervalMinutes. The code expects either 0, 30 or 60 there - and assumed it wasn't anything else. That said, the usual "load setting" checking was in place there, but it's possible that an empty value or some other unexpected thing got stored regardless, and then parsed in interesting ways.I'm going to add a few sanity and bounds checks to make this more robust, won't be finished today though. WIP.
(also would like to be able to reproduce the new settings load error to confirm this was the problem)
@cthulu commented on GitHub (Nov 23, 2025):
Definitely an interesting find! Thanks for checking that out.
On Sun, 23 Nov 2025 at 14:47, BenV @.***> wrote: