Skip to content
This repository has been archived by the owner on Jan 9, 2020. It is now read-only.

Better error handling in geojson conversion #253

Open
mgrauer opened this issue Jan 13, 2016 · 4 comments
Open

Better error handling in geojson conversion #253

mgrauer opened this issue Jan 13, 2016 · 4 comments
Milestone

Comments

@mgrauer
Copy link
Contributor

mgrauer commented Jan 13, 2016

@jeffbaumes noticed this today with some dirty data. Add some exception handling in the object conversion code, log any exceptions, and proceed past the bad row.

Either that, or get cleaner data. 馃毧

@aashish24
Copy link
Contributor

@mgrauer I already have created a issue for it #252

@aashish24
Copy link
Contributor

Its not the problem that we are getting bad data

@mgrauer
Copy link
Contributor Author

mgrauer commented Jan 13, 2016

This was a separate issue to #252, there was data that didn't have the mapped "latitude" and "longitude" fields on certain rows, and would cause the entire dataset conversion to fail. Instead, those rows should be skipped during conversion. I think there was an issue of some incorrect formats of lat/long values also for certain rows.

@mgrauer mgrauer reopened this Jan 13, 2016
@aashish24
Copy link
Contributor

his was a separate issue to #252, there was data that didn't have the mapped "latitude" and "longitude" fields on certain rows, and would cause the entire dataset conversion to fail. Instead, those rows should be skipped during conversion. I think there was an issue of some incorrect formats of lat/long values also for certain rows.

Ah okay, sorry for the noise then.

@aashish24 aashish24 added this to the Release 0.2.0 milestone Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants