refactor logger in linef

[imago.git] / imago_pack / linef.py

diff --git a/imago_pack/linef.py b/imago_pack/linef.py
index 03de6bd..667442c 100644 (file)
--- a/imago_pack/linef.py
+++ b/imago_pack/linef.py
@@ -13,80 +13,74 @@ except ImportError, msg:
 import filters
 from hough import Hough
 
 import filters
 from hough import Hough
 

-def prepare(image, show_all, do_something, verbose):
+def prepare(image, show_image, logger):

     # TODO comment
     im_l = image.convert('L')
     # 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"
+    logger("edge detection")

     im_edges = filters.edge_detection(im_l)
     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)
 
     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
  
 
     return im_h
  

+def transform(image, hough, show_image):
+    # TODO comment
+    im_hough = hough.transform(image)
+    show_image(im_hough, "hough transform")

 
 

-def find_lines(image, show_all, do_something, verbose):
-    """Find lines in the *image*."""
-    # TODO refactor into smaller functions
-
-    if verbose:
-        print >> sys.stderr, "preprocessing"
-
-    if show_all:
-        do_something(image, "original image")
+   # im_hough.image = filters.peaks(im_hough.image)
+   # show_image(im_hough.image, "peak extraction")
+               
+    im_h2 = filters.high_pass(im_hough, 96)
+    show_image(im_h2, "second high pass filters")

 
 

-    im_h = prepare(image, show_all, do_something, verbose)
-       
+    im_h2 = filters.components2(im_h2)
+    show_image(im_h2, "components centers")

 
 

-    hough = Hough.default(im_h)
+    return im_h2

 
 

+def find_lines(image, show_image, verbose):
+    """Find lines in the *image*."""
+    # TODO refactor into smaller functions
+    

     if verbose:
     if verbose:

-        print >> sys.stderr, "hough transform"
+        def logger(m):
+            print >> sys.stderr, m
+    else:
+        def logger(m):
+            pass
+        
+    logger("preprocessing")
+    show_image(image, "original image")

 
 

-    im_hough = hough.transform(im_h)
-    if show_all:
-        do_something(im_hough, "hough transform")
+    im_h = prepare(image, show_image, logger)

 
 

-   # 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")
+    hough = Hough.default(im_h)

 
 

-    im_h2 = filters.components2(im_h2)
-    if show_all:
-        do_something(im_h2, "components centers")
+    logger("hough transform")
+    
+    im_h2 = transform(im_h, hough, show_image)

 
 

-    if verbose:
-        print >> sys.stderr, "second hough transform"
+    logger("second hough transform")

 
     # im_hough might be used instead im_h2, but at the moment it brings a lot of
     # noise to the second transform, which later confuses the center-finding
     # mechanism (which is not very robust yet)
     hough2 = Hough.default(im_h2)
     im_hough2 = hough2.transform(im_h2)
 
     # im_hough might be used instead im_h2, but at the moment it brings a lot of
     # noise to the second transform, which later confuses the center-finding
     # 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)
 
     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)
      
     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"
+    logger("finding the grid")

 
     lines_m = hough2.all_lines_h(im_h3)
     lines = []
 
     lines_m = hough2.all_lines_h(im_h3)
     lines = []


@@ -111,15 +105,13 @@ def find_lines(image, show_all, do_something, verbose):
         line_points.sort()
         bounds += [line_points[0], line_points[-1]]
 
         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))
 
     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
 
    
     return lines, lines_m[0][0], lines_m[1][0], bounds, hough