-
Notifications
You must be signed in to change notification settings - Fork 19.6k
[OpenVINO Backend] support ops.slice_update #21362
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
base: master
Are you sure you want to change the base?
[OpenVINO Backend] support ops.slice_update #21362
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #21362 +/- ##
==========================================
- Coverage 82.73% 82.73% -0.01%
==========================================
Files 565 565
Lines 55215 55264 +49
Branches 8606 8617 +11
==========================================
+ Hits 45682 45722 +40
- Misses 7424 7429 +5
- Partials 2109 2113 +4
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:
|
c968548
to
004626b
Compare
004626b
to
aef0743
Compare
let us wait until |
@rkazants |
@@ -810,9 +810,83 @@ def prepare_slice_index(val): | |||
|
|||
|
|||
def slice_update(inputs, start_indices, updates): |
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.
please add comments for each block
799ecfa
to
7ef40d5
Compare
@rkazants |
7ef40d5
to
307110e
Compare
307110e
to
a4188f1
Compare
Hi @rkazants ,
I've supported
ops,silce_update
for my GSoC project, but I can't enable tests for it until getting__getitem__
merge.It could be implemented more easily if we use
opset15
instead ofopset14
.