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

Update cold start messages #377

Closed
kaituo opened this issue Dec 19, 2022 · 10 comments
Closed

Update cold start messages #377

kaituo opened this issue Dec 19, 2022 · 10 comments
Assignees
Labels
enhancement New feature or request v2.6.0

Comments

@kaituo
Copy link
Collaborator

kaituo commented Dec 19, 2022

Is your feature request related to a problem?
Due to opensearch-project/anomaly-detection#753, we change this message “Attempting to initialize the detector with historical data. This initializing process takes approximately 10 minutes.” It might still take longer than 1 minute if not sufficient historical data but also doesn’t take 10 minutes anymore. Better still, we should add wording to measure how much historical data is enough.

@kaituo kaituo added enhancement New feature or request untriaged v2.5.0 'Issues and PRs related to version v2.5.0' labels Dec 19, 2022
@kaituo kaituo self-assigned this Dec 19, 2022
@kaituo kaituo removed the untriaged label Dec 19, 2022
@ohltyler
Copy link
Member

ohltyler commented Jan 5, 2023

@kaituo is this still targeted for 2.5.0?

@kaituo
Copy link
Collaborator Author

kaituo commented Jan 5, 2023

No, will change to 2.6.

@kaituo kaituo added v2.6.0 and removed v2.5.0 'Issues and PRs related to version v2.5.0' labels Jan 5, 2023
@kaituo
Copy link
Collaborator Author

kaituo commented Jan 17, 2023

old: This initializing process takes approximately x minutes.
new: This initializing process takes approximately 1 minute if you have data in each of last 40 most recent consecutive intervals.

@alicejw1
Copy link

Hi @vagimeli, just fyi. . Kaituo has a screen capture image that shows some new UI text for us to review. Could you please take a look at it? thanks

@kaituo
Copy link
Collaborator Author

kaituo commented Jan 17, 2023

new screenshot:

Screen Shot 2023-01-17 at 10 46 45 AM

@kaituo
Copy link
Collaborator Author

kaituo commented Jan 17, 2023

@alicejw-aws @vagimeli @ohltyler Can you review the wording change?

@alicejw1
Copy link

I would update as follows:
"The initialization process takes approximately one minute if you have data in each of the 40 most recent consecutive intervals."

@kaituo
Copy link
Collaborator Author

kaituo commented Jan 17, 2023

@alicejw-aws Will update.

@vagimeli
Copy link

@kaituo The suggested changes are noted in bold text. "This initializing process takes approximately 1 minute if you have data in each of the last 40 consecutive intervals."

@kaituo kaituo mentioned this issue Jan 19, 2023
4 tasks
@kaituo
Copy link
Collaborator Author

kaituo commented Jan 30, 2023

updated msg in #398

@kaituo kaituo closed this as completed Jan 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request v2.6.0
Projects
None yet
Development

No branches or pull requests

4 participants