koordinator
koordinator copied to clipboard
proposal: add metric prediction framework
Ⅰ. Describe what this PR does
add resource recommend proposal
Ⅱ. Does this pull request fix one issue?
Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist
- [ ] I have written necessary docs and comments
- [ ] I have added necessary unit tests and integration tests
- [ ] All checks passed in
make test
Codecov Report
Attention: Patch coverage is 67.75087%
with 466 lines
in your changes are missing coverage. Please review.
Project coverage is 67.53%. Comparing base (
ad36a0b
) to head (c4d7644
). Report is 51 commits behind head on main.
:exclamation: Current head c4d7644 differs from pull request most recent head cedff54. Consider uploading reports for the commit cedff54 to get more accurate results
Additional details and impacted files
@@ Coverage Diff @@
## main #1880 +/- ##
==========================================
+ Coverage 67.27% 67.53% +0.25%
==========================================
Files 411 420 +9
Lines 45875 47059 +1184
==========================================
+ Hits 30863 31781 +918
- Misses 12775 12971 +196
- Partials 2237 2307 +70
Flag | Coverage Δ | |
---|---|---|
unittests | 67.53% <67.75%> (+0.25%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
The full list of commands accepted by this bot can be found here.
Approvers can indicate their approval by writing /approve
in a comment
Approvers can cancel approval by writing /approve cancel
in a comment