[GitHub] [incubator-pinot] cyy0714 commented on a change in pull request #4423: [TE] add ml auto config into YAML template

[GitBox <gi...@apache.org>]

cyy0714 commented on a change in pull request #4423: [TE] add ml auto config into YAML template
URL: https://github.com/apache/incubator-pinot/pull/4423#discussion_r302684105
 
 

 ##########
 File path: thirdeye/thirdeye-frontend/app/utils/constants.js
 ##########
 @@ -27,6 +27,7 @@ rules:                            # Can configure multiple rules with "OR" relat
         configuration:
           bucketPeriod: P1D       # Use PT1H for hourly and PT5M for minute level (ingraph metrics) data.
           pValueThreshold: 0.05   # Higher value means more sensitive to small changes.
+          mlConfig: true          # The machine learning auto config to select and maintain the configuration with the best performance.
 
 Review comment:
   Thanks for the comment. The current plan is to change the template (Only affects the new alerts, and monitor the performance) and set the default value to true (GA to all alerts). That's why we decide to change the template for evaluation.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org