Skip to content
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] Single Metric Viewer: Fix regressions #42909

Closed
7 tasks done
walterra opened this issue Aug 8, 2019 · 1 comment
Closed
7 tasks done

[ML] Single Metric Viewer: Fix regressions #42909

walterra opened this issue Aug 8, 2019 · 1 comment
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Feature:Anomaly Detection ML anomaly detection :ml regression v7.4.0

Comments

@walterra
Copy link
Contributor

walterra commented Aug 8, 2019

Documents follow-up tasks for #41739

  • When jumping from the job list to the Single Metric Viewer (via job row icon), the default reloading interval is set to 0 seconds and the reload is turned off, but it still reloads every 30 seconds (on other tests it seems that it takes the refresh interval that is configured in the job management page) — Fixes in [ML] Anomaly Explorer: Updates dragSelect to 1.13.1. Fixes reload issue. #43779 and [ML] Single metric viewer: Fix top nav refresh behaviour. #44860
  • Usability: A refresh reloads all the elements and brings the user back to the top of the page, which is a bit annoying e.g. when reading details of an anomaly in an expanded row at the bottom of the page. Comparison to job list reload: it doesn’t remove the whole table from the page and creates a new one, so my row expansion and page scroll status stays the same after a refresh. Also the refresh closes the Create Rules flyout, which can be annoying as well as I have to configure it all again in case I haven’t saved it yet. The Add annotation flyout stays open / is re-opened but the time frame for that annotation can not be changed any more. — fixed in [ML] Single metric viewer: Fixes full page refresh behavior. #44884
  • Remove the Refresh-button for the top dropdown and update the whole view just on any dropdown change — fixed in [ML] Single metric viewer: Fix detector/entity selection #44772
  • Regression: When loading a job with 2 detectors that have a different partition field (like for the cloudwatch dataset detector 1 is partitioned by instance and detector 2 is partitioned by region), it only shows the split selection for the first detector. But when changing to the second detector, it should also change the split selection so the user can filter e.g. for region.
  • Fix brush update issues: focus chart moves out of chart, empty reload
  • Single metric viewer gets confused when changing detector [ML] Single metric viewer gets confused when changing detector #43681
  • With refresh turned on (set to 5s) on switching jobs, the entity dropdown(s) for the previous job are sometimes retained, changing jobs could also cause the focus chart to stay empty [ML] Single metric viewer: Fix race condition when changing jobs. #45420
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:Anomaly Detection ML anomaly detection :ml regression v7.4.0
Projects
None yet
Development

No branches or pull requests

2 participants