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

featuresIn no longer returns GeoJSON values with 'higher' bearings #1822

Closed
musicformellons opened this issue Dec 10, 2015 · 6 comments · Fixed by #2052
Closed

featuresIn no longer returns GeoJSON values with 'higher' bearings #1822

musicformellons opened this issue Dec 10, 2015 · 6 comments · Fixed by #2052

Comments

@musicformellons
Copy link

@anandthakker @jfirebaugh FeaturesIn works fine with no or small bearing, but when changing the bearing of the map to somewhat higher values it stops returning values. Seems that it stops somewhere around bearing values > 45 degrees.
See:
screenshot from 2015-12-10 08-17-40

And here the fiddle showing this behavior:
http://jsfiddle.net/musicformellons/q1n902vL/

@musicformellons
Copy link
Author

Uh..., any comments on this?

@musicformellons
Copy link
Author

@anandthakker @jfirebaugh Uh?

@lucaswoj
Copy link
Contributor

@musicformellons I'm not sure how this has slipped through the cracks until now. I'll take a look this week!

@musicformellons
Copy link
Author

@lucaswoj Yeah, I was a bit puzzled by the low interest this seemed to raise. Seems pretty relevant to me. Glad you will look into it!

@lucaswoj
Copy link
Contributor

lucaswoj commented Feb 2, 2016

@musicformellons I was not able to reproduce the behavior you are reporting. Could you re-open this issue with a test case? Here is my attempt http://jsfiddle.net/4daxz657/

@lucaswoj lucaswoj closed this as completed Feb 2, 2016
@lucaswoj
Copy link
Contributor

lucaswoj commented Feb 2, 2016

Ah. I see. This seems to be a problem only with the bounds-less (i.e. entire viewport) version of featuresIn

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

Successfully merging a pull request may close this issue.

2 participants