Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

[Time series echart] Need a threshold to show value labels. #17281

Closed
kamalkeshavani-aiinside opened this issue Oct 29, 2021 · 6 comments
Closed
Labels
enhancement:request Enhancement request submitted by anyone from the community need:community-volunteer Requires a community volunteer

Comments

@kamalkeshavani-aiinside
Copy link
Contributor

kamalkeshavani-aiinside commented Oct 29, 2021

Is your feature request related to a problem? Please describe.
A new feature is available in master, where user can select to show only total or all values for a stacked series. When showing all values, we see that the values are very cluttered when many series have close/similar values.

Sample:
image

Describe the solution you'd like
A good solution for this is in Pie chart, where the user can input the minimum threshold in percentage to decide which labels should be seen in chart.
image

I guess that if such threshold value is added by user in Time-series chart, then need to calculate % value of each series at each point on x-axis to decide which labels to show on every point of x-axis.

Describe alternatives you've considered
Another solution would be to set a fixed threshold from code, but I guess giving the option to user would be a better idea.

Additional context
Add any other context or screenshots about the feature request here.

@villebro
Copy link
Member

This is an excellent suggestion! @junlincc @stephenLYZ let's consider how this could best be implemented.

@villebro villebro added the enhancement:request Enhancement request submitted by anyone from the community label Oct 29, 2021
@stephenLYZ
Copy link
Member

Thanks for the suggestion! And the idea that calculating % value of each series are also very valuable.

@junlincc
Copy link
Member

Absolutely agree! thanks for suggesting! @kamalkeshavani-aiinside.

The change seems to be a good first issue. I'm okay with simply adding a text input field.

@junlincc junlincc added the need:community-volunteer Requires a community volunteer label Nov 15, 2021
@corbinrobb
Copy link
Contributor

Do you mind if I give this a try? @junlincc

@eschutho
Copy link
Member

yes, go for it @corbinrobb! Thank you!

@corbinrobb
Copy link
Contributor

No problem! @eschutho I will have a PR up shortly

@apache apache locked and limited conversation to collaborators Feb 2, 2022
@geido geido converted this issue into discussion #18524 Feb 2, 2022

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
enhancement:request Enhancement request submitted by anyone from the community need:community-volunteer Requires a community volunteer
Projects
None yet
Development

No branches or pull requests

6 participants