-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[XY] Fixes percentage mode behavior when multiple metrics are placed on the same axis #121610
Conversation
Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors) |
@elasticmachine merge upstream |
src/plugins/vis_types/xy/public/utils/compute_percentage_data.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested the PR against the previous 7.15.1 version and it's now fixed 👍
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@stratoula Not sure whether that's a new bug or not, but in case of percentiles the percentage formatting is dropped (low priority issue): |
@stratoula That would be great, thanks. Besides that it seems to work just fine 👍 |
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: |
…on the same axis (elastic#121610) * Fixes the stacked part for percentage mode * Fixes the percentages mode * Add unit tests * Address performance related comment Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit 2d41a34)
…on the same axis (elastic#121610) * Fixes the stacked part for percentage mode * Fixes the percentages mode * Add unit tests * Address performance related comment Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit 2d41a34)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…on the same axis (#121610) (#122546) * Fixes the stacked part for percentage mode * Fixes the percentages mode * Add unit tests * Address performance related comment Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit 2d41a34) Co-authored-by: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
…on the same axis (#121610) (#122547) * Fixes the stacked part for percentage mode * Fixes the percentages mode * Add unit tests * Address performance related comment Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit 2d41a34) Co-authored-by: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
…on the same axis (elastic#121610) * Fixes the stacked part for percentage mode * Fixes the percentages mode * Add unit tests * Address performance related comment Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Summary
Closes #121508
Closes #122496
This PR fixes the ratios that are computed for the percentage mode if multiple metrics are placed on the same axis.
With small multiples
With split series
Checklist