-
Notifications
You must be signed in to change notification settings - Fork 279
Add UT for Patches #1766
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
Add UT for Patches #1766
Conversation
03e9597
to
a5563a4
Compare
36fe57f
to
592e5c5
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1766 +/- ##
==========================================
+ Coverage 60.61% 61.41% +0.79%
==========================================
Files 75 77 +2
Lines 8127 8264 +137
==========================================
+ Hits 4926 5075 +149
+ Misses 3201 3189 -12
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
cdf7436
to
57fcc52
Compare
lgtm |
Updated |
6545e87
to
ec6afd3
Compare
Signed-off-by: Pr0Wh1teGivee <calvin_zhu0210@outlook.com>
ec6afd3
to
3173165
Compare
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
What this PR does / why we need it?
Add UT for patches in vLLM Ascend
Does this PR introduce any user-facing change?
No
How was this patch tested?
Irrelevant