Still fixing small bugs in stepper, now tested

kiln/stepper.py

@@ -1,6 +1,7 @@
 import time
 import atexit
 import threading
+import warnings
 import Queue
 
 from RPi import GPIO
@@ -78,7 +79,7 @@ def _step_noblock(self, step, speed):
         target = self.phase + step
         while self.phase != target:
             now = time.time()
-            self.phase += 1 if target > self.phase else 1
+            self.phase += 1 if target > self.phase else -1
             output = self.pattern[self.phase%len(self.pattern)]
             for pin, out in zip(self.pins, output):
                 GPIO.output(pin, out)
@@ -89,6 +90,7 @@ def _step_noblock(self, step, speed):
                 target += step
                 if block:
                     self._step(target - self.phase, speed)
+                self.finished.set()
 
             diff = ispeed - (time.time() - now)
             if (diff) > 0:
@@ -117,7 +119,7 @@ def _step(self, step, speed):
 
 
 class Regulator(object):
-    def __init__(self, maxsteps=4500, minsteps=2500, speed=200, ignite_pin=26):
+    def __init__(self, maxsteps=4500, minsteps=2500, speed=150, ignite_pin=26):
         """Set up a stepper-controlled regulator. Implement some safety measures
         to make sure everything gets shut off at the end
 
@@ -163,7 +165,7 @@ def off(self, block=True):
         self.current = 0
 
     def set(self, value, block=False):
-        if not 0 < value < 1:
+        if not 0 <= value <= 1:
             raise ValueError("Must give fraction between 0 and 1")
         target = int(value * (self.max - self.min) + self.min)
         nsteps = target - self.current

kiln/test_stepper.py

@@ -2,12 +2,12 @@
 import stepper
 
 def test_noblock():
-	reg = Regulator(ignite_pin=None)
+	reg = stepper.Regulator(ignite_pin=None)
 
 	reg.ignite()
 	reg.set(.5)
 	time.sleep(.5)
 	reg.set(.1)
 	time.sleep(.5)
 	reg.set(.5, block=True)
-	return reg
+	return reg