-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[APM] Charts Y-axis doesn't show maxValue in the ticks #84482
Comments
Pinging @elastic/apm-ui (Team:apm) |
@formgeist the mentioned elastic-charts ticket is about providing a niced ending tick, in your example, it basically expands the Y domain to contain the next niced tick on the domain like:
Your use case instead is to show always the maximum value on the existing series instead, independently if it's is nicely rounded or not. |
The values you suggest (80s, 4.0 rpm, 60%) would be perfect for our use case I think. So extending the Y domain to include the next niced tick would indeed solve our problem. |
@markov00 As @sqren said that's all we need to solve this 👍
Not sure what you mean here. Are you thinking of the average values in the legends that we have displayed? |
Hey @formgeist @sqren we have introduced the possibility to add nice rounding to the axis: elastic/elastic-charts#1087 |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
@dannycroft @sqren I've updated the description to have the solution as recommended by the charts team since they've implemented a nice y-value option a few months ago. Maybe we can prioritize this issue and move it into implementation. |
Kibana version: master
Summary
Extracted from #84178
Blocked by elastic/elastic-charts#397Most of the dynamic value-driven charts don't show the
maxValue
meaning that the chart doesn't have an "upper ceiling" shown, making it quite hard to see where those spikes in the charts are landing. I think this in many ways a regression of our existing charts, and I would recommend we take a look at fixing this visualization issue.Solution
Elastic Charts implemented a nice y-value option in elastic/elastic-charts#1087 as described by @markov00;
cc @markov00 @nickofthyme for your consideration
The text was updated successfully, but these errors were encountered: