From cbdc42af021f898e82d3e78ce7c636d3fb5eece0 Mon Sep 17 00:00:00 2001
From: Russell <rcoh@rcoh.(none)>
Date: Mon, 10 Jan 2011 12:21:19 -0500
Subject: Some performance improvements.  Faster evaluation of range-based
 queries with lambda expressions.  Faster exp with approximated fastexp.  Some
 changes to the component registry.

---
 pixelmappers/SimpleMapper.py | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

(limited to 'pixelmappers')

diff --git a/pixelmappers/SimpleMapper.py b/pixelmappers/SimpleMapper.py
index 4d12fe4..2df24e0 100644
--- a/pixelmappers/SimpleMapper.py
+++ b/pixelmappers/SimpleMapper.py
@@ -14,17 +14,21 @@ class SimpleMapper(PixelMapper):
                     bestPixel = pixel
                     bestDist = pixelDist
             return [(bestPixel,1)]
-        elif type(type(str)):
-            #[{x}>5,{y}<k]
+        else:
+            #{x}>5,{y}<k
             #TODO: we should probably encapsulate this somewhere
             ret = []
             eventLocation = eventLocation.replace('{x}', 'pixel.location[0]')
             eventLocation = eventLocation.replace('{y}', 'pixel.location[1]')
+            conditions = eventLocation.split(',')
+            conditionLambdas = [eval('lambda pixel:'+condition) for condition in conditions]
             for pixel in screen:
                 try:
-                    preValid = eval(eventLocation)
-                    pixelValid = sum(preValid) == len(preValid) #TODO: some
-                    #optimizations possible.  This might be slow in the long run
+                    pixelValid = True
+                    for p in conditionLambdas:
+                        if p(pixel) == False:
+                            pixelValid = False
+                            continue
                     if pixelValid:
                         ret.append((pixel, 1))
                 except Exception as exp:
-- 
cgit v1.2.3