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 2eae68a commit e14a5fc
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 e14a5fc

Please sign in to comment.