-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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] Explain log rate spikes: Add DualBrush component. #135318
Conversation
Pinging @elastic/ml-ui (:ml) |
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
💚 Build SucceededMetrics [docs]Module Count
Public APIs missing comments
Async chunks
Page load bundle
Unknown metric groupsAPI count
History
To update your PR or re-run it, just comment with: cc @walterra |
Summary
Part of #136265.
Adds the code for the dual brush component for users to be able select log rate spikes in histogram charts.
For this PR a new package
@kbn/aiops-components
was created that also includesProgressControls
from@kbn/aiops-utils
now. The reason for this is: The brush component includes code fromd3
that cannot be imported on the server side whichaiops-utils
was also used for.Note: This PR just adds the component to
@kbn/aiops-components
that we bring over from the PoC, it's not used yet in the UI, this will be done in follow-ups.Checklist