Skip to content

Commit

Permalink
Changed required minimum intervals in cold start message (#411)
Browse files Browse the repository at this point in the history
Required minimum intervals to finish cold start is related to shingle size. This PR adds the shingle size in the computation and puts the result in the cold start message.

Testing done:
1. verified changing shingle size will change the message.

Signed-off-by: Kaituo Li <kaituo@amazon.com>
(cherry picked from commit e7ac417)
  • Loading branch information
kaituo authored and amitgalitz committed Feb 9, 2023
1 parent 0361c92 commit 8462e0f
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion public/pages/DetectorResults/containers/AnomalyResults.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ import { detectorIsSample } from '../../Overview/utils/helpers';
import { SampleIndexDetailsCallout } from '../../Overview/components/SampleIndexDetailsCallout/SampleIndexDetailsCallout';
import { CoreStart } from '../../../../../../src/core/public';
import { CoreServicesContext } from '../../../components/CoreServices/CoreServices';
import { DEFAULT_SHINGLE_SIZE } from '../../../utils/constants';

interface AnomalyResultsProps extends RouteComponentProps {
detectorId: string;
Expand Down Expand Up @@ -313,7 +314,8 @@ export function AnomalyResults(props: AnomalyResultsProps) {
<p>
Attempting to initialize the detector with historical data.
This initializing process takes approximately 1 minute if
you have data in each of the last 40 consecutive intervals.
you have data in each of the last{' '}
{32+get(detector, 'shingleSize', DEFAULT_SHINGLE_SIZE)}{' '} consecutive intervals.
</p>
</EuiText>
</EuiFlexGroup>
Expand Down

0 comments on commit 8462e0f

Please sign in to comment.