From fd9a681b1e6fb94684bbfacea2b933ecae1c68ed Mon Sep 17 00:00:00 2001
From: Tomas Musil <tomik.musil@gmail.com>
Date: Thu, 26 Jun 2014 15:13:36 +0200
Subject: [PATCH 1/1] refactor do_something in linef

---
 imago_pack/linef.py | 53 +++++++++++++++++++++++++----------------------------
 todo_count.sh       |  1 +
 2 files changed, 26 insertions(+), 28 deletions(-)

diff --git a/imago_pack/linef.py b/imago_pack/linef.py
index 5eaa0d1..fb83db4 100644
--- a/imago_pack/linef.py
+++ b/imago_pack/linef.py
@@ -13,40 +13,35 @@ except ImportError, msg:
 import filters
 from hough import Hough
 
-def prepare(image, show_all, do_something, verbose):
+def prepare(image, show_image, verbose):
     # TODO comment
     im_l = image.convert('L')
-    if show_all:
-        do_something(im_l, "ITU-R 601-2 luma transform")
+    show_image(im_l, "ITU-R 601-2 luma transform")
 
     if verbose:
         print >> sys.stderr, "edge detection"
     im_edges = filters.edge_detection(im_l)
-    if show_all:    
-        do_something(im_edges, "edge detection")
+    show_image(im_edges, "edge detection")
 
     im_h = filters.high_pass(im_edges, 100)
-    if show_all:
-        do_something(im_h, "high pass filters")
+    show_image(im_h, "high pass filters")
 
     return im_h
  
-def transform(image, hough, show_all, do_something):
+def transform(image, hough, show_image):
+    # TODO comment
     im_hough = hough.transform(image)
-    if show_all:
-        do_something(im_hough, "hough transform")
+    show_image(im_hough, "hough transform")
 
    # im_hough.image = filters.peaks(im_hough.image)
    # if show_all:
    #     do_something(im_hough.image, "peak extraction")
                
     im_h2 = filters.high_pass(im_hough, 96)
-    if show_all:
-        do_something(im_h2, "second high pass filters")
+    show_image(im_h2, "second high pass filters")
 
     im_h2 = filters.components2(im_h2)
-    if show_all:
-        do_something(im_h2, "components centers")
+    show_image(im_h2, "components centers")
 
     return im_h2
 
@@ -54,20 +49,27 @@ def find_lines(image, show_all, do_something, verbose):
     """Find lines in the *image*."""
     # TODO refactor into smaller functions
 
+    def nothing(a, b):
+        pass
+
+    if show_all:
+        show_image = do_something
+    else:
+        show_image = nothing
+
     if verbose:
         print >> sys.stderr, "preprocessing"
 
-    if show_all:
-        do_something(image, "original image")
+    show_image(image, "original image")
 
-    im_h = prepare(image, show_all, do_something, verbose)
+    im_h = prepare(image, show_image, verbose)
 
     hough = Hough.default(im_h)
 
     if verbose:
         print >> sys.stderr, "hough transform"
     
-    im_h2 = transform(im_h, hough, show_all, do_something)
+    im_h2 = transform(im_h, hough, show_image)
 
     if verbose:
         print >> sys.stderr, "second hough transform"
@@ -77,16 +79,13 @@ def find_lines(image, show_all, do_something, verbose):
     # mechanism (which is not very robust yet)
     hough2 = Hough.default(im_h2)
     im_hough2 = hough2.transform(im_h2)
-    if show_all:
-        do_something(im_hough2, "second hough transform")
+    show_image(im_hough2, "second hough transform")
 
     im_h3 = filters.high_pass(im_hough2, 120)
-    if show_all:
-        do_something(im_h3, "third high pass filter")
+    show_image(im_h3, "third high pass filter")
      
     im_h3 = filters.components(im_h3)
-    if show_all:
-        do_something(im_h3, "half centers")
+    show_image(im_h3, "half centers")
 
     if verbose:
         print >> sys.stderr, "finding the grid"
@@ -114,15 +113,13 @@ def find_lines(image, show_all, do_something, verbose):
         line_points.sort()
         bounds += [line_points[0], line_points[-1]]
 
-    if show_all:
-        do_something(im_c, "hough x lines")
+    show_image(im_c, "hough x lines")
 
     image_g = image.copy()
     draw = ImageDraw.Draw(image_g)
     for line in [l for s in lines for l in s]:
         draw.line(line_from_angl_dist(line, image.size), fill=(120, 255, 120))
-    if show_all:
-        do_something(image_g, "lines")
+    show_image(image_g, "lines")
    
     return lines, lines_m[0][0], lines_m[1][0], bounds, hough
 
diff --git a/todo_count.sh b/todo_count.sh
index ce2ec14..feac328 100755
--- a/todo_count.sh
+++ b/todo_count.sh
@@ -1,3 +1,4 @@
 #!/bin/bash
 
+grep --count TODO imago_pack/*.py pcf.c | cut -f2 -d":" | awk '{sum+=$1}END{printf "total %d\n=====\n", sum}' 
 grep --count TODO imago_pack/*.py pcf.c | sort -nr -t":" -k2 | cut -f2 -d"/"
-- 
2.4.2