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.

Sign up for GitHub

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

Jump to bottom

query-test failures: symbol/{panned,rotated}-after-insert #3346

Closed
jfirebaugh opened this issue Oct 11, 2016 · 1 comment
Closed

query-test failures: symbol/{panned,rotated}-after-insert #3346

jfirebaugh opened this issue Oct 11, 2016 · 1 comment
Labels
bug 🐞

Comments

@jfirebaugh
Copy link
Contributor

The following related query tests are failing:

These failures need investigation and fixing.

Refs mapbox/mapbox-gl-native#6627 cc @brunoabinader
ChrisLoer added a commit that referenced this issue Nov 6, 2017
@ChrisLoer
Update query tests for viewport collision detection.
0d1854f 
Defining correct behavior for these two tests has gotten easier: in both cases it's just all 121 points from 121points.geojson.
Earlier versions of this test included duplicates in expected.json because the same point would end up in results from multiple tiles.
Fixes issue #3346
@ChrisLoer ChrisLoer mentioned this issue Nov 6, 2017
Merged
ChrisLoer added a commit that referenced this issue Nov 6, 2017
@ChrisLoer
Update query tests for viewport collision detection.
f3d55f6 
Defining correct behavior for these two tests has gotten easier: in both cases it's just all 121 points from 121points.geojson.
Earlier versions of this test included duplicates in expected.json because the same point would end up in results from multiple tiles.
Fixes issue #3346
@ChrisLoer
Copy link
Contributor

Re-enabled with #5603.

@ChrisLoer ChrisLoer mentioned this issue Nov 8, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug 🐞
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mourner @jfirebaugh @ChrisLoer