Closed zoyopei closed 2 months ago
Attention: Patch coverage is 8.72483%
with 136 lines
in your changes missing coverage. Please review.
Flag | Coverage Δ | |
---|---|---|
unittests | 26.99% <8.72%> (-0.61%) |
:arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Files with missing lines | Coverage Δ | |
---|---|---|
pkg/device/hygon/device.go | 17.74% <0.00%> (-0.30%) |
:arrow_down: |
pkg/device/nvidia/device.go | 31.46% <0.00%> (-0.36%) |
:arrow_down: |
pkg/scheduler/scheduler.go | 29.24% <0.00%> (ø) |
|
pkg/device/cambricon/device.go | 14.28% <0.00%> (-0.18%) |
:arrow_down: |
pkg/device/iluvatar/device.go | 18.42% <0.00%> (-0.33%) |
:arrow_down: |
pkg/util/util.go | 25.79% <63.63%> (+1.92%) |
:arrow_up: |
pkg/device/devices.go | 23.21% <20.00%> (-8.27%) |
:arrow_down: |
pkg/device/ascend/vnpu.go | 0.00% <0.00%> (ø) |
|
pkg/device/ascend/device.go | 2.75% <3.92%> (-13.51%) |
:arrow_down: |
What type of PR is this?
/kind feature
What this PR does / why we need it: refactoring ascend device code
Which issue(s) this PR fixes: Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: