annotate py/fridge.py @ 484:4a51e2bf689b

a few fixes
author Matt Johnston <matt@ucc.asn.au>
date Thu, 06 Feb 2014 20:35:43 +0800
parents d05b1612e19e
children 3f187baa3439
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
1 # -*- coding: utf-8 -*-
462
a91adc95543d less verbose logging
Matt Johnston <matt@ucc.asn.au>
parents: 461
diff changeset
2 from utils import L,W,E,EX,D
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
3 import config
443
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
4 import gevent
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
5
443
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
6 class Fridge(gevent.Greenlet):
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
7
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
8 OVERSHOOT_MAX_DIV = 1800.0 # 30 mins
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
9
443
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
10 def __init__(self, server):
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
11 gevent.Greenlet.__init__(self)
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
12 self.server = server
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
13 self.setup_gpio()
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
14 self.wort_valid_clock = 0
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
15 self.fridge_on_clock = 0
448
fe729664a5e6 working better. logging works properly, cleanup fridge.off() happens.
Matt Johnston <matt@ucc.asn.au>
parents: 447
diff changeset
16 self.fridge_off_clock = server.now()
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
17
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
18 def setup_gpio(self):
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
19 dir_fn = '%s/direction' % config.FRIDGE_GPIO
444
c5629d79b4ac few more bits
Matt Johnston <matt@ucc.asn.au>
parents: 443
diff changeset
20 with open(dir_fn, 'w') as f:
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
21 f.write('low')
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
22 val_fn = '%s/value' % config.FRIDGE_GPIO
447
8fdf86ea41e7 few updates, seems to run
Matt Johnston <matt@ucc.asn.au>
parents: 444
diff changeset
23 self.value_file = open(val_fn, 'r+')
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
24
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
25 def turn(self, value):
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
26 self.value_file.seek(0)
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
27 if value:
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
28 self.value_file.write('1')
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
29 else:
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
30 self.value_file.write('0')
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
31 self.value_file.flush()
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
32
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
33 def on(self):
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
34 self.turn(True)
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
35
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
36 def off(self):
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
37 self.turn(False)
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
38
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
39 def is_on(self):
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
40 self.value_file.seek(0)
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
41 buf = self.value_file.read().strip()
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
42 if buf == '0':
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
43 return False
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
44 if buf != '1':
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
45 E("Bad value read from gpio '%s': '%s'"
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
46 % (self.value_file.name, buf))
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
47 return True
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
48
443
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
49 # greenlet subclassed
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
50 def _run(self):
461
1eb68df9f8ab Add daemon mode with locking, add "disabled" parameter
Matt Johnston <matt@ucc.asn.au>
parents: 457
diff changeset
51 if self.server.params.disabled:
1eb68df9f8ab Add daemon mode with locking, add "disabled" parameter
Matt Johnston <matt@ucc.asn.au>
parents: 457
diff changeset
52 L("Fridge is disabled")
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
53 while True:
443
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
54 self.do()
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
55 gevent.sleep(config.FRIDGE_SLEEP)
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
56
444
c5629d79b4ac few more bits
Matt Johnston <matt@ucc.asn.au>
parents: 443
diff changeset
57 def do(self):
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
58 """ this is the main fridge control logic """
443
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
59 wort, fridge = self.server.current_temps()
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
60
444
c5629d79b4ac few more bits
Matt Johnston <matt@ucc.asn.au>
parents: 443
diff changeset
61 params = self.server.params
c5629d79b4ac few more bits
Matt Johnston <matt@ucc.asn.au>
parents: 443
diff changeset
62
469
de4abcbe8f46 make fridge air range a parameter
Matt Johnston <matt@ucc.asn.au>
parents: 467
diff changeset
63 fridge_min = params.fridge_setpoint - params.fridge_range_lower
de4abcbe8f46 make fridge air range a parameter
Matt Johnston <matt@ucc.asn.au>
parents: 467
diff changeset
64 fridge_max = params.fridge_setpoint + params.fridge_range_upper
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
65
463
f2cca8062128 - use the fridge temperature for control too, keep it in a 6deg band
Matt Johnston <matt@ucc.asn.au>
parents: 462
diff changeset
66 wort_min = params.fridge_setpoint
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
67 wort_max = params.fridge_setpoint + params.fridge_difference
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
68
443
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
69 off_time = self.server.now() - self.fridge_off_clock
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
70
476
06de0f2ffa26 workaround for invalid wort time
Matt Johnston <matt@ucc.asn.au>
parents: 469
diff changeset
71 if wort is not None:
06de0f2ffa26 workaround for invalid wort time
Matt Johnston <matt@ucc.asn.au>
parents: 469
diff changeset
72 self.wort_valid_clock = self.server.now()
06de0f2ffa26 workaround for invalid wort time
Matt Johnston <matt@ucc.asn.au>
parents: 469
diff changeset
73
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
74 if off_time < config.FRIDGE_DELAY:
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
75 L("fridge skipping, too early")
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
76 return
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
77
461
1eb68df9f8ab Add daemon mode with locking, add "disabled" parameter
Matt Johnston <matt@ucc.asn.au>
parents: 457
diff changeset
78 if params.disabled:
1eb68df9f8ab Add daemon mode with locking, add "disabled" parameter
Matt Johnston <matt@ucc.asn.au>
parents: 457
diff changeset
79 if self.is_on():
1eb68df9f8ab Add daemon mode with locking, add "disabled" parameter
Matt Johnston <matt@ucc.asn.au>
parents: 457
diff changeset
80 L("Disabled, turning fridge off")
1eb68df9f8ab Add daemon mode with locking, add "disabled" parameter
Matt Johnston <matt@ucc.asn.au>
parents: 457
diff changeset
81 self.off()
1eb68df9f8ab Add daemon mode with locking, add "disabled" parameter
Matt Johnston <matt@ucc.asn.au>
parents: 457
diff changeset
82 return
1eb68df9f8ab Add daemon mode with locking, add "disabled" parameter
Matt Johnston <matt@ucc.asn.au>
parents: 457
diff changeset
83
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
84 # handle broken wort sensor
476
06de0f2ffa26 workaround for invalid wort time
Matt Johnston <matt@ucc.asn.au>
parents: 469
diff changeset
85 if wort is None:
443
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
86 self.wort_valid_clock = self.server.now()
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
87 W("Invalid wort sensor")
443
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
88 invalid_time = self.server.now() - self.wort_valid_clock
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
89 if invalid_time < config.FRIDGE_WORT_INVALID_TIME:
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
90 W("Has only been invalid for %d, waiting" % invalid_time)
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
91 return
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
92
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
93 if fridge is None:
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
94 W("Invalid fridge sensor")
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
95
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
96 if self.is_on():
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
97 turn_off = False
443
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
98 on_time = self.server.now() - self.fridge_on_clock
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
99
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
100 overshoot = 0
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
101 if on_time > params.overshoot_delay:
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
102 overshoot = params.overshoot_factor \
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
103 * min(self.OVERSHOOT_MAX_DIV, on_time) \
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
104 / self.OVERSHOOT_MAX_DIV
462
a91adc95543d less verbose logging
Matt Johnston <matt@ucc.asn.au>
parents: 461
diff changeset
105 D("on_time %(on_time)f, overshoot %(overshoot)f" % locals())
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
106
477
d05b1612e19e a few fixes
Matt Johnston <matt@ucc.asn.au>
parents: 476
diff changeset
107 if not params.nowort and wort is not None:
d05b1612e19e a few fixes
Matt Johnston <matt@ucc.asn.au>
parents: 476
diff changeset
108 if wort - overshoot < params.fridge_setpoint:
463
f2cca8062128 - use the fridge temperature for control too, keep it in a 6deg band
Matt Johnston <matt@ucc.asn.au>
parents: 462
diff changeset
109 L("wort has cooled enough, %(wort)f" % locals() )
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
110 turn_off = True
463
f2cca8062128 - use the fridge temperature for control too, keep it in a 6deg band
Matt Johnston <matt@ucc.asn.au>
parents: 462
diff changeset
111 elif fridge is not None and fridge < fridge_min:
f2cca8062128 - use the fridge temperature for control too, keep it in a 6deg band
Matt Johnston <matt@ucc.asn.au>
parents: 462
diff changeset
112 W("fridge off fallback, fridge %(fridge)f, min %(fridge_min)f" % locals())
476
06de0f2ffa26 workaround for invalid wort time
Matt Johnston <matt@ucc.asn.au>
parents: 469
diff changeset
113 if wort is None:
06de0f2ffa26 workaround for invalid wort time
Matt Johnston <matt@ucc.asn.au>
parents: 469
diff changeset
114 W("wort has been invalid for %d" % (self.server.now() - self.wort_valid_clock))
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
115 turn_off = True
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
116
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
117 if turn_off:
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
118 L("Turning fridge off")
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
119 self.off()
443
bca470d153fd a bit more, and some tests
Matt Johnston <matt@ucc.asn.au>
parents: 442
diff changeset
120 self.fridge_off_clock = self.server.now()
439
31ac84425a2d python raspberry pi rewrite
Matt Johnston <matt@ucc.asn.au>
parents:
diff changeset
121
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
122 else:
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
123 # fridge is off
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
124 turn_on = False
467
fa79666fb375 add nowort mode
Matt Johnston <matt@ucc.asn.au>
parents: 463
diff changeset
125 if not params.nowort \
fa79666fb375 add nowort mode
Matt Johnston <matt@ucc.asn.au>
parents: 463
diff changeset
126 and wort is not None \
fa79666fb375 add nowort mode
Matt Johnston <matt@ucc.asn.au>
parents: 463
diff changeset
127 and wort >= wort_max:
462
a91adc95543d less verbose logging
Matt Johnston <matt@ucc.asn.au>
parents: 461
diff changeset
128 L("Wort is too hot %f, max %f" % (wort, wort_max))
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
129 turn_on = True
463
f2cca8062128 - use the fridge temperature for control too, keep it in a 6deg band
Matt Johnston <matt@ucc.asn.au>
parents: 462
diff changeset
130 elif fridge is not None and fridge >= fridge_max:
f2cca8062128 - use the fridge temperature for control too, keep it in a 6deg band
Matt Johnston <matt@ucc.asn.au>
parents: 462
diff changeset
131 W("frdge on fallback, fridge %(fridge)f, max %(fridge_max)f" % locals())
442
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
132 turn_on = True
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
133
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
134 if turn_on:
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
135 L("Turning fridge on")
02318c9660cd copied fridge logic from main.c
Matt Johnston <matt@ucc.asn.au>
parents: 439
diff changeset
136 self.on()
457
176fc5b68a3f fix fridge_on_time
Matt Johnston <matt@ucc.asn.au>
parents: 448
diff changeset
137 self.fridge_on_clock = self.server.now()