-
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
[Logs UI/Design] Improve log rate visualization #46399
Comments
Pinging @elastic/infra-logs-ui |
@weltenwort @hbharding @katrin-freihofner @tbragin @Kerry350 I edited these AC slightly to reflect the conversations we had in our meeting today. Please feel free to add comments here if you think I've misrepresented anything. Thanks! We should have a smaller meeting focused on this design issue somewhat soon, @hbharding @katrin-freihofner can you let us know how much time you'd like to consider these AC as written and think through some simple solutions so we can talk them through this week? |
ℹ️ There's a dedicated implementation issue for the ML UI link at #46445, which we should update once the design work has progressed. |
Now that we've had the design handoff I'm going to close this ticket, and open a ticket strictly for the implementation (there are a few smaller points still in discussion / pending finalisation, but they can be facilitated during implementation). |
Summary
The visualization of the log rate and its anomalies should be simplified to increase focus and to accomodate partitioning of the analysis.
Rationale
In the current state the log rate visualization is an incomplete imitation of the anomaly explorer in the ML UI. Instead it would serve the use-cases better if it was tailed towards representing the relevant information in a less generic, more summarized fashion. Additionally the upcoming introduction of partitioning the ML job will require the chart to display multiple rates simultaneously while making sure anomalies are easily spotted.
Acceptance criteria
The chart...
Further notes
The text was updated successfully, but these errors were encountered: