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

WFS Feature / Zoom to filtered results. #3701

Merged

Conversation

fxprunayre
Copy link
Member

@fxprunayre fxprunayre commented Mar 28, 2019

When indexing features, add 4 fields to store the coordinate of the
bounding box of each features. On client side use min/max aggregations
to retrieve the bounding box of all features matching filter.

When ES support for geo_bounds aggregation will be implemented for geo_shape type
See https://github.com/elastic/elasticsearch/issues/7574
The following would be better probably:

```
 "viewport" : {
   "geo_bounds" : {
     "field" : "location",
     "wrap_longitude" : true
   }
 }
```

See in action https://youtu.be/SKwnwN5RPKA

    When indexing features, add 4 fields to store the coordinate of the
    bounding box of each features. On client side use min/max aggregations
    to retrieve the bounding box of all features matching filter.

    When ES support for geo_bounds aggregation will be implemented for geo_shape type
    See elastic/elasticsearch#7574
    The following would be better probably:

    ```
     "viewport" : {
       "geo_bounds" : {
         "field" : "location",
         "wrap_longitude" : true
       }
     }
    ```
@fxprunayre fxprunayre added this to the 3.8.0 milestone Mar 28, 2019
@fxprunayre fxprunayre requested a review from jahow April 8, 2019 08:23
@fxprunayre
Copy link
Member Author

Has been waiting long for a review. It has been live tested for 2 months on projects. Merging.

@fxprunayre fxprunayre merged commit 24ee908 into geonetwork:master May 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants