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
[ML] Delayed data visualization: ensure y-axis count is visible #148982
[ML] Delayed data visualization: ensure y-axis count is visible #148982
Conversation
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.
LGTM.
With this change, you can see that when the datafeed is running but there are no source docs, we aren't setting the y axis minimum to 0
. Can we fix the y axis minimum to 0? Maybe worth adding to #143702 if it isn't something you want to add to this PR.
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.
LGTM
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
…tic#148982) ## Summary Related issue: elastic#143702 Ensure the y-axis counts are visible in the delayed data chart. Default `Show model snapshots` to be initially unchecked/hidden. <img width="615" alt="image" src="https://user-images.githubusercontent.com/6446462/212722354-59b292d9-06ec-469d-95ea-617a130bc299.png"> ## NOTE Looks like this shouldn't explicitly need to be turned into a String() - a limitation on EUI Charts side - they will be updating on their end so that it's not necessary but for now this work-around is needed so that the y-axis values show up. ### Checklist Delete any items that are not applicable to this PR. - [x] Any UI touched in this PR is usable by keyboard only (learn more about [keyboard accessibility](https://webaim.org/techniques/keyboard/)) - [x] Any UI touched in this PR does not create any new axe failures (run axe in browser: [FF](https://addons.mozilla.org/en-US/firefox/addon/axe-devtools/), [Chrome](https://chrome.google.com/webstore/detail/axe-web-accessibility-tes/lhdoppojpmngadmnindnejefpokejbdd?hl=en-US)) Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Summary
Related issue: #143702
Ensure the y-axis counts are visible in the delayed data chart.
Default
Show model snapshots
to be initially unchecked/hidden.NOTE
Looks like this shouldn't explicitly need to be turned into a String() - a limitation on EUI Charts side - they will be updating on their end so that it's not necessary but for now this work-around is needed so that the y-axis values show up.
Checklist
Delete any items that are not applicable to this PR.