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

[Maps] split Grid aggregation source into Clusters and Heat map #55880

Closed
nreese opened this issue Jan 24, 2020 · 4 comments · Fixed by #60870
Closed

[Maps] split Grid aggregation source into Clusters and Heat map #55880

nreese opened this issue Jan 24, 2020 · 4 comments · Fixed by #60870
Labels
[Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation discuss enhancement New value added to drive a business result

Comments

@nreese
Copy link
Contributor

nreese commented Jan 24, 2020

The existing Grid aggregation source has a few problems

  1. It is a little confusing for new users without backgrounds with Elasticsearch, aggregations, and geotile_grid aggregation.
  2. The UI is very tightly coupled to the implementation. In users minds, Heat map and Cluster vectors are very different. There is no reason to couple them in the UI just because they share the same underlying implementation.

I would like to propose spliting the existing Grid aggregation into two sources, Clusers and Heat map. This solves the above problems

  1. clustering is a pretty universal concept that most GIS users will understand.
  2. Heat maps can be easily found at the top UI level and are not buried under something that is not very related.

So instead of seeing a single source Grid aggregation that is confusing and hides Heat maps...
Screen Shot 2020-01-24 at 1 38 12 PM

Users will see something more like the below where you have Clusters and Heat maps.
Screen Shot 2020-01-24 at 1 33 34 PM

Thoughs? Feedback?

@nreese nreese added discuss [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation enhancement New value added to drive a business result labels Jan 24, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

@nickpeihl
Copy link
Member

👍, but I think we should take care before we add too many more layer options. Or maybe consider grouping layer options in the future.

@alexfrancoeur
Copy link

++ to Nick's comment, this list feels like it could continue to grow. Clustering feels like it's well known enough. How does this split affect existing layers in a map? Will it just rename the source details and split some of the styling options? I might be wrong here but I thought heatmap had some unique properties (I'd normally check by am mobile atm)

@nreese
Copy link
Contributor Author

nreese commented Jan 27, 2020

How does this split affect existing layers in a map? Will it just rename the source details and split some of the styling options?

This will be seamless to the user. Under the covers, a new source heatmap source will be created that just extents the existing geo_grid source.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation discuss enhancement New value added to drive a business result
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants