From 6810dc0ff0eae489b155eb12cb8c648b962214f2 Mon Sep 17 00:00:00 2001
From: Rafa de la Torre <rtorre@cartodb.com>
Date: Tue, 5 Jul 2016 20:56:15 +0200
Subject: [PATCH] Code to glue together pg and python (WIP)

---
 server/extension/sql/80_isolines_helper.sql | 27 +++++++++++++++------
 server/extension/sql/90_isochrone.sql       |  2 +-
 2 files changed, 20 insertions(+), 9 deletions(-)

diff --git a/server/extension/sql/80_isolines_helper.sql b/server/extension/sql/80_isolines_helper.sql
index 43ff444..bcb30a5 100644
--- a/server/extension/sql/80_isolines_helper.sql
+++ b/server/extension/sql/80_isolines_helper.sql
@@ -55,7 +55,7 @@ RETURNS SETOF cdb_dataservices_server.isoline AS $$
 $$ LANGUAGE plpythonu SECURITY DEFINER;
 
 
-CREATE OR REPLACE FUNCTION cdb_dataservices_server._cdb_mapzen_routing_isolines(
+CREATE OR REPLACE FUNCTION cdb_dataservices_server._cdb_mapzen_isolines(
    username TEXT,
    orgname TEXT,
    isotype TEXT,
@@ -65,6 +65,7 @@ CREATE OR REPLACE FUNCTION cdb_dataservices_server._cdb_mapzen_routing_isolines(
    options text[])
 RETURNS SETOF cdb_dataservices_server.isoline AS $$
   import json
+  from cartodb_services.mapzen import MatrixClient
   from cartodb_services.mapzen import MapzenIsolines
   from cartodb_services.metrics import QuotaService
   from cartodb_services.here.types import geo_polyline_to_multipolygon # TODO do we use the same types?
@@ -82,24 +83,34 @@ RETURNS SETOF cdb_dataservices_server.isoline AS $$
     mapzen_conf_str = plpy.execute("SELECT * FROM CDB_Conf_Getconf('mapzen_conf') AS mapzen_conf")[0]['mapzen_conf']
     mapzen_conf = json.loads(mapzen_conf_str)
 
-    client = MapzenIsolines(mapzen_conf['routing']['api_key'])
+
+    client = MatrixClient(mapzen_conf['matrix']['api_key'])
+    mapzen_isolines = MapzenIsolines(client)
 
     if source:
       lat = plpy.execute("SELECT ST_Y('%s') AS lat" % source)[0]['lat']
       lon = plpy.execute("SELECT ST_X('%s') AS lon" % source)[0]['lon']
-      source_str = 'geo!%f,%f' % (lat, lon)
+      origin = {'lat': lat, 'lon': lon}
     else:
-      source_str = None
+      raise Exception('source is NULL')
 
     # -- TODO Support options properly
+    isolines = []
     if isotype == 'isodistance':
-      resp = client.calculate_isodistance(source_str, mode, data_range)
+      # -- TODO implement
+      raise 'not implemented'
+      resp = mapzen_isolines.calculate_isodistance(origin, mode, data_range)
     elif isotype == 'isochrone':
-      resp = client.calculate_isochrone(source_str, mode, data_range)
+      for r in data_range:
+          isoline = mapzen_isolines.calculate_isochrone(origin, mode, r)
+          isolines.append(isoline)
 
-    if resp:
+    return [] # -- TODO delete this
+
+    # -- TODO rewrite this block
+    if isolines:
       result = []
-      for isoline in resp:
+      for isoline in isolines:
         data_range_n = isoline['range']
         polyline = isoline['geom']
         multipolygon = geo_polyline_to_multipolygon(polyline)
diff --git a/server/extension/sql/90_isochrone.sql b/server/extension/sql/90_isochrone.sql
index a75ec67..83af69b 100644
--- a/server/extension/sql/90_isochrone.sql
+++ b/server/extension/sql/90_isochrone.sql
@@ -34,7 +34,7 @@ RETURNS SETOF cdb_dataservices_server.isoline AS $$
   #if user_isolines_config.google_services_user:
   #  plpy.error('This service is not available for google service users.')
 
-  mapzen_plan = plpy.prepare("SELECT cdb_dataservices_server._cdb_mapzen_routing_isolines($1, $2, $3, $4, $5, $6, $7) as isoline; ", ["text", "text", "text", "geometry(Geometry, 4326)", "text", "integer[]", "text[]"])
+  mapzen_plan = plpy.prepare("SELECT cdb_dataservices_server._cdb_mapzen_isolines($1, $2, $3, $4, $5, $6, $7) as isoline; ", ["text", "text", "text", "geometry(Geometry, 4326)", "text", "integer[]", "text[]"])
   result = plpy.execute(mapzen_plan, [username, orgname, type, source, mode, range, options])
   isolines = []
   for element in result: