cartodb4/Windshaft-cartodb
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix aggregation resolution parameter

Browse Source 
It was implemented as the inverse of the intended value
This commit is contained in:
Javier Goizueta
2017-12-14 12:23:02 +01:00
parent daa3fdca11
commit b0e47ecc62
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/cartodb/models/aggregation/aggregation-query.js
+5 -5
View File
@@ -72,8 +72,8 @@ const aggregateColumns = ctx => {
};
// Notes:
// * ${ctx.res*0.00028/256}*!scale_denominator! is equivalent to
// ${ctx.res/256}*CDB_XYZ_Resolution(CDB_ZoomFromScale(!scale_denominator!))
// * ${256*0.00028/ctx.res}*!scale_denominator! is equivalent to
// ${256/ctx.res}*CDB_XYZ_Resolution(CDB_ZoomFromScale(!scale_denominator!))
// * We need to filter spatially using !bbox! to make the queries efficient because
// the filter added by Mapnik (wrapping the query)
// is only applied after the aggregation.
@@ -83,7 +83,7 @@ const aggregationQueryTemplates = {
'centroid': ctx => `
WITH _cdb_params AS (
SELECT
(${ctx.res*0.00028/256}*!scale_denominator!)::double precision AS res,
(${256*0.00028/ctx.res}*!scale_denominator!)::double precision AS res,
!bbox! AS bbox
)
SELECT
@@ -105,7 +105,7 @@ const aggregationQueryTemplates = {
'point-grid': ctx => `
WITH _cdb_params AS (
SELECT
(${ctx.res*0.00028/256}*!scale_denominator!)::double precision AS res,
(${256*0.00028/ctx.res}*!scale_denominator!)::double precision AS res,
!bbox! AS bbox
),
_cdb_clusters AS (
@@ -127,7 +127,7 @@ const aggregationQueryTemplates = {
WITH _cdb_params AS (
SELECT
(${ctx.res*0.00028/256}*!scale_denominator!)::double precision AS res,
(${256*0.00028/ctx.res}*!scale_denominator!)::double precision AS res,
!bbox! AS bbox
), _cdb_clusters AS (
SELECT