-
Notifications
You must be signed in to change notification settings - Fork 458
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 README for packages with ML Modules to ensure naming is consistent #11165
Update README for packages with ML Modules to ensure naming is consistent #11165
Conversation
🚀 Benchmarks reportPackage
|
Data stream | Previous EPS | New EPS | Diff (%) | Result |
---|---|---|---|---|
access |
2881.84 | 1855.29 | -1026.55 (-35.62%) | 💔 |
To see the full report comment with /test benchmark fullreport
This reverts commit fd62849.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Quality Gate passedIssues Measures |
💚 Build Succeeded
History
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Proposed commit message
This PR updates the README for the packages containing ML Modules.
Changing ML Modules to 'Anomaly Detection Configurations' to be in line with the way we refer to the UI they link to in the assets tab.
As this is not a super high priority change, this can be published later with the next update to these packages.
These changes coincide with the changes to the way we refer to ML Assets (previously 'ML Modules') and the new ML UI for 'Supplied Configurations'.
Checklist
changelog.yml
file.