Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Coordinates not being reprojected #2

Open
jflasher opened this issue Jun 3, 2016 · 4 comments
Open

Coordinates not being reprojected #2

jflasher opened this issue Jun 3, 2016 · 4 comments
Labels

Comments

@jflasher
Copy link
Member

jflasher commented Jun 3, 2016

It looks like for some records, the coordinates are not being reprojected. Coordinates are being returned like

{"type":"Feature","geometry":{"crs":{"type":"name","properties":{"name":"urn:ogc:def:crs:EPSG:8.8.1:32623"}},"type":"Polygon","coordinates":[[[499980,7100040],[609780,7100040],[609780,6990240],[499980,6990240],[499980,7100040]]]},"properties":{}}
@scisco
Copy link
Member

scisco commented Jun 9, 2016

@jflasher I haven't been able to find such records, but will look out for it. Do you have the logs by any chance?

@jflasher
Copy link
Member Author

Nope, no logs, sorry. Will keep an eye out if I see it again and try to capture something.

@drewbo
Copy link
Member

drewbo commented Jun 22, 2016

To confirm @jflasher, this causes the errors in this other open source project that we both contribute to because then the mapbox upload fails? https://github.com/AstroDigital/ad-sentinel-vector-tile-updater

Could we also temporarily solve it by ignoring these records?

@jflasher
Copy link
Member Author

@drewbo yep, I think that's correct, we get an out of bounds error for the lat/lon on upload. Could probably just do a check around https://github.com/AstroDigital/ad-sentinel-vector-tile-updater/blob/master/updater/updater.js#L124 and ignore if it's not in proper lat/lon. I may be able to do this at some point next week, but not in the next couple of days.

drewbo pushed a commit that referenced this issue Dec 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants