-
Notifications
You must be signed in to change notification settings - Fork 200
Refactor diagnosis agent #1287
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
Refactor diagnosis agent #1287
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1287 +/- ##
==========================================
+ Coverage 80.19% 80.31% +0.11%
==========================================
Files 218 222 +4
Lines 20336 20429 +93
==========================================
+ Hits 16309 16408 +99
+ Misses 4027 4021 -6 ☔ View full report in Codecov by Sentry. |
dlrover/python/diagnosis/inferencechain/inferenceoperator/collect_metrics_operator.py
Outdated
Show resolved
Hide resolved
dlrover/python/diagnosis/inferencechain/inferenceoperator/collect_metrics_operator.py
Show resolved
Hide resolved
BalaBalaYi
left a comment
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
f10ba6c
into
intelligent-machine-learning:master
What changes were proposed in this pull request?
Refactor diagnosis agent. The agent will observe the problems periodically.
Why are the changes needed?
Complete the framework that the agent could check if predefined problems happen.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Unit test