Code: Select all
(2018, 11, 3, 13, 30, 1, 5, 307)
Traceback (most recent call last):
File "main.py", line 41, in <module>
File "ntptime.py", line 30, in settime
File "ntptime.py", line 22, in time
OSError: [Errno 110] ETIMEDOUT
Code: Select all
import machine
import network
import time
import ntptime
import utime
OFF = 1
ON = 0
sta_if = network.WLAN(network.STA_IF)
led = machine.Pin(2, machine.Pin.OUT)
led.on()
is_updated = False
################################################
#Methods
def write_to_file(status, time):
file = open('log.txt','w')
file.write(status)
file.write(str(time))
file.close()
#setup network
def connect_on_network():
# sta_if = network.WLAN(network.STA_IF)
if not sta_if.isconnected():
sta_if.active(True)
sta_if.connect('SSID', 'PASSWD')
while not sta_if.isconnected():
pass
print('net cfg:',sta_if.ifconfig())
time.sleep(10)
print(ntptime.settime())
def check_if_time_is_to_turn_on_heater(t):
print(t)
print("LED: " + str(led.value()))
if t[3]>=23 or t[3]<7 and led.value() == OFF:
# if t[3] % 2 == 0:
write_to_file("ON", utime.localtime())
led.off()
print("ON")
utime.localtime()
elif t[3]>=7 and t[3]<23 and led.value() == ON:
led.on()
print("OFF")
utime.localtime()
write_to_file("OFF", utime.localtime())
def update_time_from_server():
if utime.localtime()[4]%15==0 and not is_updated:
try:
ntptime.settime()
except:
print(EOFError)
isupdated = True
time.sleep(0.5)
print("time update...")
elif utime.localtime()[3]%15 != 0 and is_updated:
isupdated = False
#######################################################################
#application
connect_on_network()
time.sleep(2)
while True:
check_if_time_is_to_turn_on_heater(utime.localtime())
time.sleep(1)
if not sta_if.isconnected():
connect_on_network()
update_time_from_server()
Code: Select all
def write_to_file(status, time):
with open('log.txt','w') as file:
file.write(status)
file.write(str(time))
Thankspythoncoder wrote: ↑Mon Nov 05, 2018 6:29 amYou can't argue with code that works
But the (correct) logic looks wrong: you have defined OFF = 1 and ON = 0. So when you issue led.off() the Pin instance produces a value of 0 which you define as ON. This had me foxed for a moment.
You might like to look at the Signal class which is intended to deal with cases like this.
Another small point: when using files it's worth using a Python context manager. This ensures that the file is closed even if the program fails or you interrupt it with ctrl-c while the file is open.
But this is nitpicking. Well done for getting it to workCode: Select all
def write_to_file(status, time): with open('log.txt','w') as file: file.write(status) file.write(str(time))
Code: Select all
import gc
import machine
from sys import platform
gc.collect()
class WDT:
def __init__(self, id=0, timeout=120, use_rtc_memory=True):
self._timeout = timeout / 10
self._counter = 0
self._timer = machine.Timer(id)
self._use_rtc_memory = use_rtc_memory
self.init()
try:
with open("watchdog.txt", "r") as f:
if f.read() == "True":
print("Reset reason: Watchdog")
except Exception as e:
print(e) # file probably just does not exist
try:
with open("watchdog.txt", "w") as f:
f.write("False")
except Exception as e:
print("Error saving to file: {!s}".format(e))
if use_rtc_memory and platform == "esp8266":
rtc = machine.RTC()
if rtc.memory() == b"WDT reset":
print("Reset reason: Watchdog")
rtc.memory(b"")
def _wdt(self, t):
self._counter += self._timeout
if self._counter >= self._timeout * 10:
try:
with open("watchdog.txt", "w") as f:
f.write("True")
except Exception as e:
print("Error saving to file: {!s}".format(e))
if self._use_rtc_memory and platform == "esp8266":
rtc = machine.RTC()
rtc.memory(b"WDT reset")
machine.reset()
def feed(self):
self._counter = 0
def init(self, timeout=None):
timeout = timeout or self._timeout
self._timeout = timeout
self._timer.init(period=int(self._timeout * 1000), mode=machine.Timer.PERIODIC, callback=self._wdt)
def deinit(self): # will not stop coroutine
self._timer.deinit()Code: Select all
wdt=WDT()
while True:
time.sleep(1)
check_time()
wdt.feed()