CartoDB / crankshaft

CARTO Spatial Analysis extension for PostgreSQL
BSD 3-Clause "New" or "Revised" License
54 stars 20 forks source link

functions should be agnostic to 'id' and `the_geom` where appropriate #199

Open andy-esch opened 6 years ago

andy-esch commented 6 years ago

The original k-means function (https://github.com/CartoDB/crankshaft/blob/b547d35882dafe19870bd0428384bd22c63f87fc/src/py/crankshaft/crankshaft/clustering/kmeans.py#L20-L21) does not take an optional database id -- it must be cartodb_id. This should be flexible so the extension can be used on databases outside of carto more easily.