-
Notifications
You must be signed in to change notification settings - Fork 78
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
Comments
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 . |
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
@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. I think fixing this issue will require some major changes as well as change in the current structure. What do you say? |
Oh, this is really helpful information, though! Thank you!
Hmm, yes, i tihnk we would have to try to refactor this layer
significantly. But perhaps we could start by searching for and linking to
some examples of Mapbox vector tiles being used in Leaflet, as a model to
look to?
…On Fri, Mar 22, 2019 at 1:21 PM Ramakrishna Pattnaik < ***@***.***> wrote:
@jywarren <https://github.com/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
<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?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#147 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABfJ7Dkc8mVollf5UszPS3p_b6zeXrUks5vZRECgaJpZM4b8l6r>
.
|
@jywarren that seems appropriate. |
Can we have this in GCI @jywarren @sagarpreet-chadha @ananyaarun ? |
Yes, but noting that it will involve some investigating and troubleshooting
to find the problem!
…On Wed, Nov 27, 2019 at 12:16 PM Sidharth Bansal ***@***.***> wrote:
Can we have this in GCI @jywarren <https://github.com/jywarren>
@sagarpreet-chadha <https://github.com/sagarpreet-chadha> @ananyaarun
<https://github.com/ananyaarun> ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#147?email_source=notifications&email_token=AAAF6J65DKZCQJLDTWOZGDTQV2TODA5CNFSM4G7SL2V2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFKGFPQ#issuecomment-559178430>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAF6JZXUCBKCVBAFFVXE23QV2TODANCNFSM4G7SL2VQ>
.
|
Cool thanks. Published |
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!!!
The text was updated successfully, but these errors were encountered: