Skip to content

Commit

Permalink
[ML] Explain Log Rate Spikes: Fix loading state for grouping. (elasti…
Browse files Browse the repository at this point in the history
…c#141770)

Fix loading state messages for grouping:
- Fixes the progress bar messages to include the grouping step.
- If progress stalls, it might look to the user like the analysis got stuck when there are steps like the grouping that take a longer time. This updates the progress bar to show an animated background as long as the analysis is running. When the analysis finishes or gets canceled the animated background gets disabled and shows a static background.

(cherry picked from commit 5203f1b)
  • Loading branch information
walterra committed Oct 3, 2022
1 parent e11eb96 commit c876475
Show file tree
Hide file tree
Showing 5 changed files with 108 additions and 17 deletions.
4 changes: 4 additions & 0 deletions x-pack/packages/ml/aiops_components/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,8 @@ RUNTIME_DEPS = [
"@npm//react",
"@npm//@elastic/charts",
"@npm//@elastic/eui",
"@npm//@emotion/react",
"@npm//@emotion/css",
"//packages/kbn-i18n-react",
"//x-pack/packages/ml/aiops_utils",
]
Expand All @@ -78,6 +80,8 @@ TYPES_DEPS = [
"@npm//@types/react",
"@npm//@elastic/charts",
"@npm//@elastic/eui",
"@npm//@emotion/react",
"@npm//@emotion/css",
"//packages/kbn-i18n-react:npm_module_types",
"//x-pack/packages/ml/aiops_utils:npm_module_types",
]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,17 +5,22 @@
* 2.0.
*/

import React from 'react';

import {
useEuiTheme,
EuiButton,
EuiFlexGroup,
EuiFlexItem,
EuiIconTip,
EuiProgress,
EuiText,
} from '@elastic/eui';

import { i18n } from '@kbn/i18n';
import { FormattedMessage } from '@kbn/i18n-react';
import React from 'react';

import { useAnimatedProgressBarBackground } from './use_animated_progress_bar_background';

// TODO Consolidate with duplicate component `CorrelationsProgressControls` in
// `x-pack/plugins/apm/public/components/app/correlations/progress_controls.tsx`
Expand All @@ -37,6 +42,9 @@ export function ProgressControls({
isRunning,
shouldRerunAnalysis,
}: ProgressControlProps) {
const { euiTheme } = useEuiTheme();
const runningProgressBarStyles = useAnimatedProgressBarBackground(euiTheme.colors.success);

return (
<EuiFlexGroup>
<EuiFlexItem>
Expand All @@ -51,7 +59,7 @@ export function ProgressControls({
/>
</EuiText>
</EuiFlexItem>
<EuiFlexItem>
<EuiFlexItem css={isRunning ? runningProgressBarStyles : undefined}>
<EuiProgress
aria-label={i18n.translate('xpack.aiops.progressAriaLabel', {
defaultMessage: 'Progress',
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,53 @@
/*
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
* or more contributor license agreements. Licensed under the Elastic License
* 2.0; you may not use this file except in compliance with the Elastic License
* 2.0.
*/

import { useMemo } from 'react';

import { css } from '@emotion/react';

export const useAnimatedProgressBarBackground = (color: string) => {
return useMemo(() => {
const progressBackground = {
background: `repeating-linear-gradient(
-45deg,
transparent 0 6px,
rgba(0, 0, 0, 0.1) 6px 12px
),
${color}`,
// 0.707 = cos(45deg)
backgroundSize: 'calc(12px / 0.707) 100%, 100% 800%',
backgroundPosition: 'inherit',
};

return css({
'progress[value]': {
animation: 'aiopsAnimatedProgress 4s infinite linear',

'::-webkit-progress-inner-element': {
overflow: 'hidden',
backgroundPosition: 'inherit',
},
'::-webkit-progress-bar': {
backgroundColor: 'transparent',
backgroundPosition: 'inherit',
},

'::-webkit-progress-value': progressBackground,
'::-moz-progress-bar': progressBackground,

'@keyframes aiopsAnimatedProgress': {
'0%': {
backgroundPosition: '0 0',
},
'100%': {
backgroundPosition: 'calc(10 * (12px / 0.707)) 100%',
},
},
},
});
}, [color]);
};
3 changes: 2 additions & 1 deletion x-pack/packages/ml/aiops_components/tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@
"@types/d3-transition",
"jest",
"node",
"react"
"react",
"@emotion/react/types/css-prop"
]
},
"include": [
Expand Down
53 changes: 39 additions & 14 deletions x-pack/plugins/aiops/server/routes/explain_log_rate_spikes.ts
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,10 @@ import {

// Overall progress is a float from 0 to 1.
const LOADED_FIELD_CANDIDATES = 0.2;
const PROGRESS_STEP_P_VALUES = 0.6;
const PROGRESS_STEP_HISTOGRAMS = 0.2;
const PROGRESS_STEP_P_VALUES = 0.5;
const PROGRESS_STEP_GROUPING = 0.1;
const PROGRESS_STEP_HISTOGRAMS = 0.1;
const PROGRESS_STEP_HISTOGRAMS_GROUPS = 0.1;

export const defineExplainLogRateSpikesRoute = (
router: IRouter<DataRequestHandlerContext>,
Expand Down Expand Up @@ -233,7 +235,35 @@ export const defineExplainLogRateSpikesRoute = (
undefined
)) as [NumericChartData];

function pushHistogramDataLoadingState() {
push(
updateLoadingStateAction({
ccsWarning: false,
loaded,
loadingState: i18n.translate(
'xpack.aiops.explainLogRateSpikes.loadingState.loadingHistogramData',
{
defaultMessage: 'Loading histogram data.',
}
),
})
);
}

if (groupingEnabled) {
push(
updateLoadingStateAction({
ccsWarning: false,
loaded,
loadingState: i18n.translate(
'xpack.aiops.explainLogRateSpikes.loadingState.groupingResults',
{
defaultMessage: 'Transforming significant field/value pairs into groups.',
}
),
})
);

// To optimize the `frequent_items` query, we identify duplicate change points by count attributes.
// Note this is a compromise and not 100% accurate because there could be change points that
// have the exact same counts but still don't co-occur.
Expand Down Expand Up @@ -389,6 +419,10 @@ export const defineExplainLogRateSpikesRoute = (
push(addChangePointsGroupAction(changePointGroups));
}

loaded += PROGRESS_STEP_GROUPING;

pushHistogramDataLoadingState();

if (changePointGroups) {
await asyncForEach(changePointGroups, async (cpg, index) => {
const histogramQuery = {
Expand Down Expand Up @@ -445,6 +479,8 @@ export const defineExplainLogRateSpikesRoute = (
}
}

loaded += PROGRESS_STEP_HISTOGRAMS_GROUPS;

// time series filtered by fields
if (changePoints) {
await asyncForEach(changePoints, async (cp, index) => {
Expand Down Expand Up @@ -496,18 +532,7 @@ export const defineExplainLogRateSpikesRoute = (
const { fieldName, fieldValue } = cp;

loaded += (1 / changePoints.length) * PROGRESS_STEP_HISTOGRAMS;
push(
updateLoadingStateAction({
ccsWarning: false,
loaded,
loadingState: i18n.translate(
'xpack.aiops.explainLogRateSpikes.loadingState.loadingHistogramData',
{
defaultMessage: 'Loading histogram data.',
}
),
})
);
pushHistogramDataLoadingState();
push(
addChangePointsHistogramAction([
{
Expand Down

0 comments on commit c876475

Please sign in to comment.