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

Line rendering problem #2854

Closed
jingsam opened this issue Jul 12, 2016 · 12 comments
Closed

Line rendering problem #2854

jingsam opened this issue Jul 12, 2016 · 12 comments

Comments

@jingsam
Copy link
Contributor

jingsam commented Jul 12, 2016

Recently I upgraded mapbox-gl-js from 0.11.0 to 0.20.0, some wired artifacts appear with line feature rendering.
ENV: Windows 10, Chrome 51, vector tile v2(created by tilelive-bridge)

Uploading QQ截图20160712151214.png…

I have created a jsFiddle. Please help me, thanks.

@mourner
Copy link
Member

mourner commented Jul 12, 2016

What's the buffer size on those tiles?

@lucaswoj
Copy link
Contributor

I wonder if this could be a true positive of the "Geometry exceeds allowable extent" error message.

ref #2568

@jingsam
Copy link
Contributor Author

jingsam commented Jul 13, 2016

@mourner I use tilelive-bridge to generate vector tiles. I use the default configuration, which the extent should be 4096 and buffer is 128, I guess.

Here is one of my tile: http://218.244.250.70:8090/api/v1/tilesets/foxgis/admin2/8/210/99.vector.pbf

@mourner
Copy link
Member

mourner commented Jul 13, 2016

I remember seeing a very similar issue earlier but can't recall the number...

@jingsam
Copy link
Contributor Author

jingsam commented Jul 13, 2016

checkout this:
#1679
#1680

@jingsam
Copy link
Contributor Author

jingsam commented Jul 13, 2016

I tried to render my style using mapbox-gl-native, same problem happens. Maybe it is the new earcut engine causing the problem?

@jingsam
Copy link
Contributor Author

jingsam commented Jul 14, 2016

I get the warning:

Geometry exceeds allowed extent

So confirmed it is the problem of buffer size

@andrewharvey
Copy link
Collaborator

possibly the same issue as #2676 (though I only notice it for large a canvas of 4096px)

@lucaswoj
Copy link
Contributor

@jingsam are you still seeing the "Geometry exceeds allowed extent" warning message using v0.21.0?

@jingsam
Copy link
Contributor Author

jingsam commented Jul 15, 2016

@lucaswoj Yes, v0.21.0 has the warning: Geometry exceeds allowed extent, reduce your vector tile buffer size http://jsfiddle.net/jingsam/9g7otrp5/1/

@lucaswoj
Copy link
Contributor

@jingsam can you try reducing the vector tile buffer size?

On Jul 14, 2016, at 6:47 PM, jingsam [email protected] wrote:

@lucaswoj https://github.com/lucaswoj Yes, v0.21.0 has the warning: Geometry exceeds allowed extent, reduce your vector tile buffer size


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub #2854 (comment), or mute the thread https://github.com/notifications/unsubscribe-auth/AARK2nhyt0PytsBfRc_bYJR21rgdAheDks5qVubLgaJpZM4JKGph.

@lucaswoj
Copy link
Contributor

This problem should be fixed by reducing your buffer size.

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

No branches or pull requests

4 participants