comments for timer

diff --git a/imago_pack/capture.py b/imago_pack/capture.py
index 3971af5..dd7f47f 100755 (executable)
--- a/imago_pack/capture.py
+++ b/imago_pack/capture.py
@@ -2,7 +2,7 @@
 
 """Go image capture.
 
 
 """Go image capture.
 

-This module defines an UI for capturing images with a (web)camera and imediately
+This module defines a UI for capturing images with a (web)camera and imediately

 proccessing them with Imago.
 """
 
 proccessing them with Imago.
 """
 

diff --git a/imago_pack/timer.py b/imago_pack/timer.py
index b5ec3fb..2419563 100755 (executable)
--- a/imago_pack/timer.py
+++ b/imago_pack/timer.py
@@ -1,6 +1,12 @@
 #!/usr/bin/env python
 
 #!/usr/bin/env python
 

-"""Go timer"""
+"""Go timer.
+
+This module defines a UI that combines game clock with automatic game recording.
+When player presses the clock, an image of the board is taken. The sequence of
+images can later be analysed by Imago to produce a game record in abstract
+notation.
+"""

 
 import os
 import sys
 
 import os
 import sys


@@ -11,6 +17,7 @@ import subprocess
 import pygame
 
 class Timer:
 import pygame
 
 class Timer:

+    """Keeps track of time remaining one player."""

     def __init__(self, main_time, byop, byot):
         self._last = 0
         self._elapsed = 0.
     def __init__(self, main_time, byop, byot):
         self._last = 0
         self._elapsed = 0.


@@ -21,6 +28,7 @@ class Timer:
         self._byo = False
 
     def get_time(self):
         self._byo = False
 
     def get_time(self):

+        """Return string representation of remaining time."""

         if self._last > 0: # when running:
             r_time = self._main_t - (time.time() - self._last + self._elapsed)
             if (r_time - int(r_time)) < 0.75:
         if self._last > 0: # when running:
             r_time = self._main_t - (time.time() - self._last + self._elapsed)
             if (r_time - int(r_time)) < 0.75:


@@ -47,19 +55,24 @@ class Timer:
         return "{0:0>2}".format(int(r_time / 60)) + sep + "{0:0>2}".format(int(r_time % 60))
 
     def start(self):
         return "{0:0>2}".format(int(r_time / 60)) + sep + "{0:0>2}".format(int(r_time % 60))
 
     def start(self):

+        """Start the clock."""

         self._last = time.time()
 
     def stop(self):
         self._last = time.time()
 
     def stop(self):

+        """Stop the clock."""

         self._elapsed += time.time() - self._last
         self._last = 0
 
     def is_running(self):
         self._elapsed += time.time() - self._last
         self._last = 0
 
     def is_running(self):

+        """Return True if the clock is running."""

         if self._last > 0:
             return True
         else:
             return False
 
 def main():
         if self._last > 0:
             return True
         else:
             return False
 
 def main():

+    """Parse the arguments and run the clock."""
+    # TODO refactor into smaller functions

     parser = argparse.ArgumentParser(description=__doc__)
     parser.add_argument('-m', type=int, default=10,
                         help="main time in minutes (default is 10)")
     parser = argparse.ArgumentParser(description=__doc__)
     parser.add_argument('-m', type=int, default=10,
                         help="main time in minutes (default is 10)")