Cycle Start Time Not working

[Whynolove]

I've just put everything together and the winder works with the control buttons. However, the "Cycle Start Time" function doesn't seem to be functioning. I have it enabled, but it does not start the winder at the designated time. I've triple checked my wiring and made sure everything else was correct. I think it may be a software issue? Please help.

[mwood77]

Hi mate, apologies for not answering sooner. I didn’t see a notification for this.

That’s interesting. What time did you have it set for? I noticed a bug in the code for the cycle start timer earlier this week, and pushed a fix for it yesterday.

Besides that, it could be that it isn’t detecting your locale correctly. It determines your time offset with an IP geo lookup… could’ve gone wrong there.

It highly likely it’s a software bug, as there’s no wiring dependency for this to work.

• First question, which ESP32 did you buy? Do you have a link?

---

This is where it gets your local time and syncs itself; the ESP32 has a notoriously inaccurate RTC:

winderoo/src/platformio/osww-server/src/main.cpp

Lines 122 to 139 in 3895e75

	void getTime()
	{
	http.begin(client, timeURL);
	int httpCode = http.GET();
	if (httpCode > 0)
	{
	DynamicJsonDocument doc(2048);
	deserializeJson(doc, http.getStream());
	const unsigned long epoch = doc["unixtime"];
	const unsigned long offset = doc["raw_offset"];
	rtc.offset = offset;
	rtc.setTime(epoch);
	}
	http.end();
	}

• Second question, can you make sure the timezone and raw_offset fields are correct for your locale, when you click on this link:
  http://worldtimeapi.org/api/ip

---

Lastly, this is where it determines whether to begin winding, when the timer is enabled:

winderoo/src/platformio/osww-server/src/main.cpp

Lines 537 to 546 in 3895e75

	if (userDefinedSettings.timerEnabled == "1")
	{
	if (rtc.getHour(true) == userDefinedSettings.hour.toInt() &&
	rtc.getMinute() == userDefinedSettings.minutes.toInt() &&
	!routineRunning &&
	userDefinedSettings.winderEnabled == "1")
	{
	beginWindingRoutine();
	}
	}

In the above, it’s asking

1. Is the timer enabled?
2. Is the current hour == to the alarm hour desired?
3. Is the current minute == to the alarm minute desired?
4. Is the routine not currently running?
5. Is the winder enabled? (That’s the on/off toggle at the top of the screen - it must be set to ON)

If all of these evaluate correctly, the winder should begin at your desired time.

[Whynolove]

Hey, Thanks for the reply. I actually figured it out. It was the latter: the program seems to think it's in a different time zone. It is 17 hours ahead for some reason. Right now I've just adjusted the time accordingly and it's worked for the past 3 days. To use your new code, should I re-flash the board?

…

________________________________ From: mwood77 ***@***.***> Sent: Monday, August 14, 2023 3:23:36 PM To: mwood77/winderoo ***@***.***> Cc: Whynolove ***@***.***>; Author ***@***.***> Subject: Re: [mwood77/winderoo] Cycle Start Time Not working (Discussion #10) Hi mate, apologies for not answering sooner. I didn’t see a notification for this. That’s interesting. What time did you have it set for? I noticed a bug in the code for the time earlier this week, and pushed a fix for it yesterday. Besides that, it could be that it isn’t detecting your locale correctly. It determines your time offset with an IP geo lookup… could’ve gone wrong there. — Reply to this email directly, view it on GitHub<#10 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BB2MM2ETGGKZC2UVS5DNNNDXVKQORANCNFSM6AAAAAA3KOWHZY>. You are receiving this because you authored the thread.Message ID: ***@***.***>

[mwood77]

Are you behind a VPN?

---

To fix the bug, pull the new code, then re-flash the code & file system to the board. The bug was in the frontend code, which is stored in the filesystem.

[Whynolove]

Nope, no VPN. What was the bugfix?

…

________________________________ From: mwood77 ***@***.***> Sent: August 15, 2023 12:12 AM To: mwood77/winderoo ***@***.***> Cc: Whynolove ***@***.***>; Author ***@***.***> Subject: Re: [mwood77/winderoo] Cycle Start Time Not working (Discussion #10) That's very strange - are you behind a VPN or something?

________________________________ To fix the bug, pull the new code, then re-flash the code & file system to the board. The bug was in the frontend code, which is stored in the filesystem. — Reply to this email directly, view it on GitHub<#10 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BB2MM2D7ENABTNJYWCDIDHTXVMOMFANCNFSM6AAAAAA3KOWHZY>. You are receiving this because you authored the thread.Message ID: ***@***.***>

[mwood77]

When selecting the hour 23, it would show as 3.

[mwood77]

This was an issue with instantiating the rtc using an epoch/offset. I've moved this to using local datestamp time in #49