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

OpenInfraMap petroleum layer erroring with TIMED_OUT #147

Open
jywarren opened this issue Mar 19, 2019 · 8 comments · May be fixed by #149
Open

OpenInfraMap petroleum layer erroring with TIMED_OUT #147

jywarren opened this issue Mar 19, 2019 · 8 comments · May be fixed by #149
Labels
bug Something isn't working gci-candidate hard help wanted Extra attention is needed reserved-for-gci

Comments

@jywarren
Copy link
Member

jywarren commented Mar 19, 2019

We are preparing to participate in Google Code-in, and have reserved this issue for participants in GCI - but we'd love to have your help with another one! Please check out https://code.publiclab.org to see more.

Failed to load resource: net::ERR_CONNECTION_TIMED_OUT on:

https://publiclab.github.io/leaflet-environmental-layers/example/#8/30.554/-92.606/BL2-OIMPetroleum

I believe the tiles themselves are not loading: https://tiles-c.openinframap.org/petroleum/5/5/12.png

Maybe we were blocked for hitting their tiles too hard? Or maybe their API changed?

See #22 #23 for details on the original implementation. Any help appreciated!!!

@jywarren jywarren added bug Something isn't working help wanted Extra attention is needed labels Mar 19, 2019
@avsingh999
Copy link
Member

@jywarren Yes, there are also others, like this please look this #140 I have opened
thanks : )

@rkpattnaik780
Copy link
Collaborator

Open Infrastructure Maps have switched from old raster tiles to vector tiles, the raster tiles have been deprecated. They have mentioned about this in a tweet.

The new tileJSON is defined in https://openinframap.org/map.json .

rkpattnaik780 added a commit to rkpattnaik780/leaflet-environmental-layers that referenced this issue Mar 21, 2019
The openinframaps have switched to vector tiles and the old raster tiles have been deprecated. The tile had to be updated based on openinframap.org/map.json

fixes publiclab#147
@rkpattnaik780 rkpattnaik780 linked a pull request Mar 21, 2019 that will close this issue
@rkpattnaik780
Copy link
Collaborator

@jywarren I had mailed about this issue to Mr. Russ Garret, a maintainer of OpenInfraMap. I got this reply : -

" Those URLs are for the old raster tiles which are no longer in use.
Openinframap now uses Mapbox vector tiles which are less easy to use
in Leaflet unfortunately. You can use the Leaflet.VectorGrid plugin
with the vector tile details in https://openinframap.org/map.json but
you will need to write your own styling. "

I think fixing this issue will require some major changes as well as change in the current structure. What do you say?

@jywarren
Copy link
Member Author

jywarren commented Mar 22, 2019 via email

@rkpattnaik780
Copy link
Collaborator

@jywarren that seems appropriate.

@SidharthBansal
Copy link
Member

Can we have this in GCI @jywarren @sagarpreet-chadha @ananyaarun ?

@jywarren
Copy link
Member Author

jywarren commented Nov 27, 2019 via email

@SidharthBansal
Copy link
Member

Cool thanks.

Published

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working gci-candidate hard help wanted Extra attention is needed reserved-for-gci
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants