Skip to content

[CPU] fixed strides filling in blocking desc#2353

Merged
dmitry-gorokhov merged 1 commit intoopenvinotoolkit:masterfrom
antonvor:fix/antonvor/strides_filling_in_blocking_desc
Nov 10, 2020
Merged

[CPU] fixed strides filling in blocking desc#2353
dmitry-gorokhov merged 1 commit intoopenvinotoolkit:masterfrom
antonvor:fix/antonvor/strides_filling_in_blocking_desc

Conversation

@antonvor
Copy link
Copy Markdown
Contributor

@antonvor antonvor commented Sep 21, 2020

Issue 39150

@antonvor antonvor requested review from a team September 21, 2020 13:32
@antonvor antonvor self-assigned this Sep 21, 2020
@antonvor antonvor force-pushed the fix/antonvor/strides_filling_in_blocking_desc branch from 064558d to 973d453 Compare September 21, 2020 13:34
@openvino-pushbot openvino-pushbot added the category: CPU OpenVINO CPU plugin label Sep 21, 2020
@dmitry-gorokhov
Copy link
Copy Markdown

@antonvor Is it possible to cover this case with tests?

@antonvor antonvor force-pushed the fix/antonvor/strides_filling_in_blocking_desc branch from 973d453 to d16af45 Compare September 22, 2020 08:56
@ilya-lavrenov ilya-lavrenov added the pr: needs tests PR needs tests updating label Sep 23, 2020
@ilya-lavrenov ilya-lavrenov added this to the 2021.2 milestone Sep 23, 2020
@antonvor antonvor force-pushed the fix/antonvor/strides_filling_in_blocking_desc branch from d16af45 to 82ed234 Compare November 9, 2020 13:08
@dmitry-gorokhov
Copy link
Copy Markdown

@antonvor Tests part should be implemented separately. Please create corresponding ticket and leave the number here.

@dmitry-gorokhov dmitry-gorokhov merged commit be065c0 into openvinotoolkit:master Nov 10, 2020
@antonvor
Copy link
Copy Markdown
Contributor Author

Issue with adding a test for changes in this PR - 43015

mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: CPU OpenVINO CPU plugin pr: needs tests PR needs tests updating

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants