search

pelias-deprecated / quattroshapes

(DEPRECATED) Pelias import pipeline for Quattroshapes
https://github.com/pelias/whosonfirst
5 stars 1 forks source link

geometry errors (null bounds) #2

Closed missinglink closed 9 years ago

missinglink commented 10 years ago

Prior to fixing https://github.com/pelias/quattroshapes-pipeline/issues/1 there were 1-2 records which returned a null bounds and therefore caused the import to fail.

Because of better error handling those records no longer cause a fatal error; however they are still not being imported properly.

This ticket is to investigate how many error, why they error and to get them importing correctly. As far as I know this only affects the local_admin import, which used to fail around the time the index stats hit 2500.

missinglink commented 10 years ago 
invalid bounds null
center geolib.getCenter failed undefined
geolib.getCenter failed
invalid bounds null
center geolib.getCenter failed undefined
geolib.getCenter failed
invalid bounds null
center geolib.getCenter failed undefined
geolib.getCenter failed
sevko commented 9 years ago

I'm working on removing null/empty geometries and computing and exporting centroids in the Quattro preprocessing scripts in pelias/quattroshapes-patch, which will mean that we don't need to worry about it in the importer.