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

Visual builder aggregation help reference #8701

Closed
tbragin opened this issue Oct 15, 2016 · 2 comments
Closed

Visual builder aggregation help reference #8701

tbragin opened this issue Oct 15, 2016 · 2 comments
Labels
Feature:Visualizations Generic visualization features (in case no more specific feature label is available) release_note:enhancement Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@tbragin
Copy link
Contributor

tbragin commented Oct 15, 2016

In Visualize, we typically expose Elasticsearch terminology around which aggregations we're using (e.g. Terms). One notable exception is "Unique Count", which is actually fulfilled by the cardinality aggregation in Elasticsearch. Either way, users get confused -- some don't know what "Terms" refers to, others would really like to know how unique count is calculated.

Proposal is to provide a reference in Visualize to help the user know more about the aggregation they are choosing.

Some ideas:

  • Could selector itself supports basic help text?
  • Once the aggregation is chosen, show a tooltip with more info and a link to ES aggregation reference
@tbragin tbragin added release_note:enhancement Feature:Visualizations Generic visualization features (in case no more specific feature label is available) labels Oct 15, 2016
@sophiec20
Copy link
Contributor

Also concerns ML. We have a function distinct count which effectively maps to unique count which are both approximations, albeit slightly different ones.

We currently have a mix of labelling in the ML UI - which is not ideal. I am curious why unique count is used as I feel that the concept is more commonly referred to as distinct e.g. even in our own documentation..

A single-value metrics aggregation that calculates an approximate count of distinct values.

Also, off the topic of this GitHub issue (but not the one that brought me here #7955), perhaps Average should be renamed to Mean, especially now that we have a Median.

@timroes timroes added the Team:Visualizations Visualization editors, elastic-charts and infrastructure label Sep 16, 2018
@timroes
Copy link
Contributor

timroes commented Oct 27, 2018

We nowadays have help links for all aggregations in the editor, so you can directly jump to the documentation for more explanation. With the new visual editor (#17076), we are trying to get rid of Elasticsearch specific terminology more and rather use common names. I would be closing this issue, since now that we have the links available there are no current plans for improving that behavior in the existing editor.

@timroes timroes closed this as completed Oct 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Visualizations Generic visualization features (in case no more specific feature label is available) release_note:enhancement Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

No branches or pull requests

3 participants