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

Improve handling of very small slices in pie charts #16931

Closed
gmoskovicz opened this issue Feb 27, 2018 · 2 comments · Fixed by #83929
Closed

Improve handling of very small slices in pie charts #16931

gmoskovicz opened this issue Feb 27, 2018 · 2 comments · Fixed by #83929
Assignees
Labels
enhancement New value added to drive a business result Feature:ElasticCharts Issues related to the elastic-charts library Feature:Pie Chart Pie chart visualization feature Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@gmoskovicz
Copy link
Contributor

Kibana version: 6.2

Elasticsearch version: 6.2

Server OS version: Any

Browser version: Any

Browser OS version: Any

Original install method (e.g. download page, yum, from source, etc.): Zip

Description of the problem including expected versus actual behavior:

In 5.x you were able to see very small buckets as lines, but in 6.x all very small buckets are just missing, and you see a big white space now.

Steps to reproduce:

  1. Create a Pie Chart visualization with 1000 buckets.
  2. Run and check results.

5.x Results

5 x-screenshot

6.x results

6 x-screenshot

This is just an example, but when you have a relevant chart, in 6.x you can no longer see the very small buckets that we use to be able to see, and just see the following:

renditiondownload 2

Errors in browser console (if relevant):

Provide logs and/or server output (if relevant):

No logs.

@gmoskovicz gmoskovicz added the bug Fixes for quality problems that affect the customer experience label Feb 27, 2018
@lukasolson lukasolson added Feature:Visualizations Generic visualization features (in case no more specific feature label is available) triage_needed labels Feb 27, 2018
@timroes timroes changed the title Pie Chart Visualization: White width is larger in 6.x Improve handling of very small slices in pie charts May 24, 2018
@timroes timroes added enhancement New value added to drive a business result Feature:Pie Chart Pie chart visualization feature and removed bug Fixes for quality problems that affect the customer experience regression labels May 24, 2018
@timroes
Copy link
Contributor

timroes commented May 24, 2018

We have some issues handling very small slices in pie charts and should think about how we properly want to handle them.

Users that create too manys lices might end up with slices below a pixel, that won't even be drawn anymore. It might get worse in cases where you have this very small slices only on the second level of the pie chart. In those cases it might look like there is no second level at all because all slices are below a pixel width.

We should find a generic approach to handle that many/small slices in a good way for users (or limit slice amount, and sizes in a user friendly way).

@timroes timroes added Team:Visualizations Visualization editors, elastic-charts and infrastructure and removed Feature:Visualizations Generic visualization features (in case no more specific feature label is available) labels Sep 16, 2018
@timroes timroes added the Feature:ElasticCharts Issues related to the elastic-charts library label Sep 25, 2019
@stratoula
Copy link
Contributor

The new chart implementation will solve this. For example a multi buckets pie chart will be displayed as:
Screenshot 2020-12-22 at 11 53 51 AM

@stratoula stratoula self-assigned this Dec 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:ElasticCharts Issues related to the elastic-charts library Feature:Pie Chart Pie chart visualization feature Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants