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

"Geometry exceeds allowed extent, reduce your vector tile buffer size" #8992

Closed
dmytro-gokun opened this issue Nov 19, 2019 · 14 comments · Fixed by #9753
Closed

"Geometry exceeds allowed extent, reduce your vector tile buffer size" #8992

dmytro-gokun opened this issue Nov 19, 2019 · 14 comments · Fixed by #9753

Comments

@dmytro-gokun
Copy link
Contributor

dmytro-gokun commented Nov 19, 2019

This was initially reported in #8639, but that issue is closed now.

The thing is that mapbox outputs lots of these warnings to the console which totally screws our e2e tests.

Is there any plans to fix this?

@kkaefer
Copy link
Contributor

kkaefer commented Nov 19, 2019

What version of Mapbox GL JS are you using? Are you using tiles from api.mapbox.com? If so, at what locations do these warnings occur?

@dmytro-gokun
Copy link
Contributor Author

What version of Mapbox GL JS are you using?

1.5.0

Are you using tiles from api.mapbox.com?

Yes. For example: mapbox://styles/mapbox/streets-v9.

If so, at what locations do these warnings occur?

E.g. zoom=9, center=[-74.50, 40].

Another example would be mapbox://styles/mapbox/light-v9, zoom=15.5, center=[-74.0066, 40.7135].
These are by no means the only points.

NB. Here: #8639 (comment), @mourner said: "This seems to be a minor issue with some tiles in Mapbox Streets on our side — you can safely ignore the warning while we investigate a fix". So the question is... is that investigation still in progress? Is there any update for that?

@mourner
Copy link
Member

mourner commented Nov 19, 2019

It's in progress, but we have no updates on when this will be fixed on the data side yet. I think you should fix your e2e testing setup anyway — console warnings shouldn't be a blocker for anything.

@dmytro-gokun
Copy link
Contributor Author

@mourner You are (probably) right about e2e tests. But that's only one side of the problem. The other side is that it spams the console and makes debugging harder. Also, a minor inconvenience, that's true. But nevertheless, it makes sense to have that fixed. The problem is not new, many people suffer from it. Would be really nice to have this fixed.

@awulkan
Copy link

awulkan commented Nov 21, 2019

I'm getting 6 of these warnings every time I load the map or change map style in our app. It's centered on Sweden at zoom level 4. Not a blocker, but as @dmytro-gokun said it clutters the console.

@mourner
Copy link
Member

mourner commented Nov 21, 2019

It outputs one warning per worker thread (but not more) whenever it encounters an affected tile. The data-side fix will take some time, so meanwhile we have two options — either leave the warnings as is for now, or remove them completely but have a risk of some users having rare rendering issues when using badly tiled data that exceeds maximum extent without any indication of what's exactly wrong.

I'll try to come up with some middle ground — perhaps we can suppress the warning if the clamping is within some small threshold (the current warnings with Mapbox Streets are triggered by a 1px-off error).

@derwaldgeist
Copy link

+1

Getting this in combination with react-map-gl, when zooming out. We're using maptiler as a tile service, should this be relevant.

@kkaefer
Copy link
Contributor

kkaefer commented Dec 2, 2019

@derwaldgeist This warning is caused because vector tiles contain geometry that exceeds the allowed extent. It's not a problem with Mapbox GL itself but with the vector data source. That means that if you're using a third-party data source, they have to fix this problem on their side.

@derwaldgeist
Copy link

@kkaefer Ok, thanks, good to know.

@lastzero
Copy link

Using Mapbox GL for one day and instantly came across this issue:
https://demo.photoprism.org/places/s2:80858741f414

San Francisco seems to be especially affected!
Looks like we need to report this to our third party source :D

@FrancescoSaverioZuppichini

Same here

@NickPepper
Copy link

MapBox v1.8.1 still has this annoying issue...

@mapbox mapbox locked and limited conversation to collaborators Mar 2, 2020
@mourner
Copy link
Member

mourner commented Mar 2, 2020

Sorry for not addressing this yet — since this does not affect the map experience in any way, and only surfaces as a couple warnings in the dev console, we've been focusing on more pressing issues. But if this is really as annoying as the comments seem to suggest, perhaps we should prioritize a client-side fix, at least for the Streets source (where the trigger is a tiny off-by-one-pixel issue).

Please do not leave "+1" type of comments — upvote the issues you want fixed instead, this helps us prioritize them.

@mapbox mapbox unlocked this conversation Mar 2, 2020
@saifulkhan
Copy link

I am also getting the same warning. We are using Mapbox in our Ionic/Angular project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants