Skip to content

DepthToSpace - revise reference implementation#6210

Merged
ilyachur merged 15 commits intoopenvinotoolkit:masterfrom
elszkowski:depth_to_space_ref_impl
Jun 28, 2021
Merged

DepthToSpace - revise reference implementation#6210
ilyachur merged 15 commits intoopenvinotoolkit:masterfrom
elszkowski:depth_to_space_ref_impl

Conversation

@elszkowski
Copy link
Copy Markdown

@elszkowski elszkowski commented Jun 17, 2021

Details:

  • move DepthToSpace::evaluate_depth_to_space to separate standalone function in runtime::reference::depth_to_space
  • drop needless reshapes from reference implementation (reshapes doesn't affect elements in tensor, shape was only changed)
  • move backend test to ngraph/test/backend/depth_to_space.in.cpp
  • extend backend test set
  • add serialization test
  • add visitor API test

Tickets:

  • 44624

@elszkowski elszkowski marked this pull request as ready for review June 21, 2021 06:29
@elszkowski elszkowski requested a review from ilyachur June 22, 2021 08:46
@elszkowski elszkowski requested a review from jdanieck June 24, 2021 07:11
@ilyachur ilyachur merged commit 119bd70 into openvinotoolkit:master Jun 28, 2021
@elszkowski elszkowski deleted the depth_to_space_ref_impl branch July 19, 2021 08:55
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
* move evaluate method to runtime/reference

* move backend test from fused_op.in.cpp to depth_to_space.in.cpp

* cleanup test

* Drop unnecessary reshapes

* add some comment

* cleanup names

* add visitor test

* Add backend test for DepthToSpace

* Add serialization test

* add empyt lines to get more readable code

* cleanup headers

* fix used types

* remove unnecessary includes
andrei-cv pushed a commit to andrei-cv/openvino that referenced this pull request Aug 30, 2021
* move evaluate method to runtime/reference

* move backend test from fused_op.in.cpp to depth_to_space.in.cpp

* cleanup test

* Drop unnecessary reshapes

* add some comment

* cleanup names

* add visitor test

* Add backend test for DepthToSpace

* Add serialization test

* add empyt lines to get more readable code

* cleanup headers

* fix used types

* remove unnecessary includes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants