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

Geo: Geohash_cell produces bad neighbors #7226

Closed
clintongormley opened this issue Aug 11, 2014 · 0 comments

Comments

@clintongormley
Copy link
Member

commented Aug 11, 2014

GET /_validate/query?explain
{
  "query": {
    "filtered": {
      "filter": {
        "geohash_cell": {
          "location": {
            "lat": 51.521568,
            "lon": -0.141257
          },
          "precision": "100km",
          "neighbors": true
        }
      }
    }
  }
}

Returns geohashes:

Only gcpt, gpcv, and u10j are in the right place. The others are in the Gulf of Guinea.

@colings86 colings86 self-assigned this Aug 12, 2014
@colings86 colings86 removed the adoptme label Aug 12, 2014
@colings86 colings86 added v1.4.0 and removed v1.3.0 labels Aug 12, 2014
@colings86 colings86 closed this in cd4aea8 Aug 13, 2014
colings86 added a commit that referenced this issue Aug 13, 2014
The geohash grid it 8 cells wide and 4 cells tall. GeoHashUtils.neighbor(String,int,int.int) set the limit of the number of cells in y to < 3 rather than <= 3 resulting in it either not finding all neighbours or incorrectly searching for a neighbour in a different parent cell.

Closes #7226
@colings86 colings86 changed the title Geohash_cell produces bad neighbors Geo: Geohash_cell produces bad neighbors Aug 13, 2014
@colings86 colings86 added the v1.3.2 label Aug 13, 2014
colings86 added a commit that referenced this issue Aug 13, 2014
The geohash grid it 8 cells wide and 4 cells tall. GeoHashUtils.neighbor(String,int,int.int) set the limit of the number of cells in y to < 3 rather than <= 3 resulting in it either not finding all neighbours or incorrectly searching for a neighbour in a different parent cell.

Closes #7226
colings86 added a commit that referenced this issue Sep 8, 2014
The geohash grid it 8 cells wide and 4 cells tall. GeoHashUtils.neighbor(String,int,int.int) set the limit of the number of cells in y to < 3 rather than <= 3 resulting in it either not finding all neighbours or incorrectly searching for a neighbour in a different parent cell.

Closes #7226
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
The geohash grid it 8 cells wide and 4 cells tall. GeoHashUtils.neighbor(String,int,int.int) set the limit of the number of cells in y to < 3 rather than <= 3 resulting in it either not finding all neighbours or incorrectly searching for a neighbour in a different parent cell.

Closes elastic#7226
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.