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 errors at various zoom levels #90

Closed
allspatial opened this issue Dec 29, 2016 · 9 comments
Closed

Geometry errors at various zoom levels #90

allspatial opened this issue Dec 29, 2016 · 9 comments
Labels

Comments

@allspatial
Copy link

allspatial commented Dec 29, 2016

I am getting geometry errors in several polygon and line layers at various zoom levels. The higher the zoom level the more errors appear (screenshot 2). I am using tegola_darwin_amd64 (v 0.3.0) from the release page. I can provide sample data in Shapefile format or as PostgreSQL dump if required.

Many thanks,

Michael

error

error2

@ARolek
Copy link
Member

ARolek commented Dec 29, 2016

@allspatial thanks for the report. What client side library are you using? If you're using Mapbox then we have seen this issue and it should be fixed in our 0.4 release which comes with geometry clipping (and is nearing completion!).

If you're using OpenLayers then this is a new issue and your shapefile would be helpful.

@ARolek ARolek added the bug label Dec 29, 2016
@allspatial
Copy link
Author

Hi @ARolek! Many thanks for your swift reply! I am indeed using mapbox (mapbox-gl-js v 0.29.0). I wouldn't mind using OpenLayers but I did a few tests about three months back and the performance of OL when labelling thousands of features was significantly lower than with mabbox-gl-js. However, that might have improved meanwhile. I am very much looking forward to your 0.4 release then.

@ARolek
Copy link
Member

ARolek commented Dec 29, 2016

@allspatial I agree, OL support for labeling with MVT still needs some love. Once we have clipping merged into master, would you mind helping us test the pre release?

@allspatial
Copy link
Author

I would be glad to test the pre-release, I really want to use Tegola for quite an important government project. Are there any instructions on how to build the binary (I haven't done anything with Go so far)?

@ARolek
Copy link
Member

ARolek commented Dec 31, 2016

@allspatial I can put together some instructions in the Readme.md for building. Do you have go installed? If so, you can clone the repo, and then navigate to the cmd/ directory and run the following:

go build -o tegola *.go
./tegola --config=path/to/config.toml

For additional details on the config file check out our docs.

@allspatial
Copy link
Author

allspatial commented Jan 2, 2017

I installed Go via Homebrew. Building the tegola binary went smoothly. I had to cd to cmd/tegola/ though, not just cmd/. Many thanks!!!

@ARolek
Copy link
Member

ARolek commented Jan 2, 2017

@allspatial good catch. I have added the building instructions to the Readme file here. We will let you know when clipping is ready for you to test.

@allspatial
Copy link
Author

@ARolek, perfect. I am looking forward to that. Many thanks again!

@gdey
Copy link
Member

gdey commented Mar 14, 2017

Moving this to #56; as I believe these errors are related to clipping. (As pointed out by @allspatial )

@gdey gdey closed this as completed Mar 14, 2017
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