From c20c85b4743d14419d7af2a07044f8f82de064bd Mon Sep 17 00:00:00 2001
From: John Cupitt <jcupitt@gmail.com>
Date: Mon, 15 Sep 2014 14:55:08 +0100
Subject: [PATCH] getting ready for hough test

---
 TODO                      | 8 +++-----
 python/test_arithmetic.py | 8 ++++++++
 python/vips8/vips.py      | 1 -
 3 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/TODO b/TODO
index 3b17676d..bef49c07 100644
--- a/TODO
+++ b/TODO
@@ -1,13 +1,11 @@
 
 - python:
 
-	- draw_circle() etc. should copy() the image before writing, and
-	  return the copy
+	- maxpos?
 
-	  accidental segvs are too easy otherwise :( 
+	  perhaps:
 
-	  draw.c should declare arg as VIPS_ARGUMENT_REQUIRED_INPUT_OUTPUT 
-	  maybe?
+	  	v, x, y = im.max(x = True, y = True)
 
 	- could import like this:
 
diff --git a/python/test_arithmetic.py b/python/test_arithmetic.py
index 77dc3059..99a8a489 100755
--- a/python/test_arithmetic.py
+++ b/python/test_arithmetic.py
@@ -358,5 +358,13 @@ class TestArithmetic(unittest.TestCase):
             self.assertEqual(hist.height, 1)
             self.assertEqual(hist.bands, 1)
 
+    def test_hough_circle(self):
+        im = Vips.Image.black(100, 100).draw_circle(100, 50, 50, 40)
+
+        hough = im.hough_circle(min_radius = 35, max_radius = 45)
+
+        
+
+
 if __name__ == '__main__':
     unittest.main()
diff --git a/python/vips8/vips.py b/python/vips8/vips.py
index 6daf7a3b..3248d7f2 100644
--- a/python/vips8/vips.py
+++ b/python/vips8/vips.py
@@ -81,7 +81,6 @@ class Argument:
 
         # MODIFY input images need to be copied before assigning them
         if self.flags & Vips.ArgumentFlags.MODIFY:
-            print 'taking copy of MODIFY image arg'
             value = value.copy()
 
         logging.debug('assigning %s' % self.prop.value_type)