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

Bounds where left > right #25

Open
alpha-beta-soup opened this issue Apr 28, 2017 · 2 comments
Open

Bounds where left > right #25

alpha-beta-soup opened this issue Apr 28, 2017 · 2 comments
Labels

Comments

@alpha-beta-soup
Copy link

alpha-beta-soup commented Apr 28, 2017

If I have a tile source that goes from west of Australia to east right of New Zealand, I would expect to be able to write the bounds as:

[90, -70, -170, 70] // left, bottom, right, top

Unfortunately this doesn't seem to produce a valid tilejson (in Mapbox GL JS no tiles are requested for this source).

It's not possible to workaround this by just switching the left/right longitudes, because that represents all the negative space where there is no data. Leaving the only possible workaround to specify the entire -180°/180° range:

[-180, -70, 180, 70],

This results in a lot of unnecessary requests to the server that just result in 404 responses, and also prevents the ability to draw a border around the bounds. However it works from a user perspective because all data is shown.

Extending the bounds beyond 180° to 190° (below) results in appropriate requests, but nothing between 180° and 190°(with Mapbox GL JS as the implementation).

[90, -70, 190, 70]

Should the spec be able to support bounds where the left longitude is greater than the right longitude?

@nigels-com
Copy link

Agreed, this is a bug. longitude should not be limited to [-180,180].

@mapsam
Copy link
Contributor

mapsam commented Apr 9, 2018

Thanks for the ticket @alpha-beta-soup ! Agree that enforcing bounds to 180/90 has its limitations. We’ll work on clarifying the current limitations in v3 and consider expanding beyond 180/90 for v4.

@mapsam mapsam mentioned this issue May 4, 2018
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