CartoDB / observatory-extension

BSD 3-Clause "New" or "Revised" License
6 stars 4 forks source link

Add more countries support to MVT generation #354

Open antoniocarlon opened 6 years ago

antoniocarlon commented 6 years ago

Added multi-country support to MVT generation (currently tested with US, CA, AU and UK).

This updates OBS_GetMCDOMVT functions. Live one (x, y, z version) is used by the tiler (CartoDB/do_tiler/pull/51) for Canada at z 13 and 14, and batch one (z version) is used by the ETL (CartoDB/bigmetadata/pull/578).

Tilesets API MRLI Release

These PRs (together):

Close these issues:

juanignaciosl commented 6 years ago

Blocked until data is already loaded.

juanignaciosl commented 6 years ago

This is no longer blocked.

juanignaciosl commented 6 years ago

First error:

psycopg2.ProgrammingError: column reference "pa_id" is ambiguous
LINE 10:                 the_geom the_geom, pa_id, obs_ca78a5d3a520fc...
                          bbox2d, $1, $2, $3) AS mvtgeom, pa_id as id, pop_0_to_24 pop_0_to_24,pop_25_to_49 pop_25_to_49,pop_50_to_64 pop_50_to_64,pop_65_to_74 pop_65_to_74,pop_75_and_over pop_75_and_over,total_pop total_pop,  area_ratio, area FROM (
                the_geom the_geom, pa_id, obs_ca78a5d3a520fc89e54d3a1a4b268a84dd579fe5.pop_0_to_24,obs_ca78a5d3a520fc89e54d3a1a4b268a84dd579fe5.pop_25_to_49,obs_ca78a5d3a520fc89e54d3a1a4b268a84dd579fe5.pop_50_to_64,obs_ca78a5d3a520fc89e54d3a1a4b268a84dd579fe5.pop_65_to_74,obs_ca78a5d3a520fc89e54d3a1a4b268a84dd579fe5.pop_75_and_over,obs_ca78a5d3a520fc89e54d3a1a4b268a84dd579fe5.total_pop, 

Fixed (I think) with the previous commit.

alrocar commented 5 years ago

@antoniocarlon Can this be closed?