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.

Sign up for GitHub

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

Jump to bottom

[6.x] [ML] Fix Anomaly Table update trigger (#18769) #18774

Merged
merged 1 commit into from
May 4, 2018
Merged

[6.x] [ML] Fix Anomaly Table update trigger (#18769) #18774

merged 1 commit into from
May 4, 2018

Commits on May 3, 2018

  1. [ML] Fix Anomaly Table update trigger (elastic#18769) 

    Fixes an issue with the anomaly table not correctly updating in the anomaly explorer and the anomaly markers not showing up in the time series viewer.
- moves the initialization of scope.anomalyRecords = []; to the controllers of anomaly explorer and time series viewer. having it in the anomaly table directive caused a race condition to overwrite already fetched records.
- use $watchCollection instead of $watch to track anomalyRecords and focusChartData changes.
replaces $timeouts with scope.$evalAsync
- fixes how focusChartData gets updated correctly to fix missing anomaly markers in the time series viewer.
    @walterra
    walterra committed May 3, 2018
    Configuration menu
    Copy the full SHA
    bb341d0 View commit details
    Browse the repository at this point in the history