[VPU][GT] Trivial permute optimization#571
Merged
ggladilov merged 8 commits intoopenvinotoolkit:masterfrom Jun 10, 2020
AntonDudchenko:ad/trivial_permute_optimization
Merged
[VPU][GT] Trivial permute optimization#571ggladilov merged 8 commits intoopenvinotoolkit:masterfrom AntonDudchenko:ad/trivial_permute_optimization
ggladilov merged 8 commits intoopenvinotoolkit:masterfrom
AntonDudchenko:ad/trivial_permute_optimization
Conversation
Contributor
Author
|
@AndrewBakalinIntel could you take a look please |
AntonDudchenko
requested review from
a user and
andrew-zaytsev
and removed request for
andrew-zaytsev
Contributor
Author
|
@gladilov-gleb take a look please |
Contributor
Get 2+ approves from our team members first, please :) |
rvyunov
reviewed
Jun 4, 2020
inference-engine/src/vpu/graph_transformer/src/middleend/passes/merge_permute_stages.cpp
Outdated
Show resolved
Hide resolved
rvyunov
reviewed
Jun 4, 2020
inference-engine/src/vpu/graph_transformer/src/middleend/passes/merge_permute_stages.cpp
Outdated
Show resolved
Hide resolved
rvyunov
reviewed
Jun 4, 2020
inference-engine/src/vpu/graph_transformer/src/middleend/passes/merge_permute_stages.cpp
Outdated
Show resolved
Hide resolved
rvyunov
reviewed
Jun 4, 2020
inference-engine/src/vpu/graph_transformer/src/middleend/passes/merge_permute_stages.cpp
Outdated
Show resolved
Hide resolved
rvyunov
reviewed
Jun 4, 2020
inference-engine/src/vpu/graph_transformer/src/middleend/passes/merge_permute_stages.cpp
Outdated
Show resolved
Hide resolved
rvyunov
reviewed
Jun 4, 2020
inference-engine/src/vpu/graph_transformer/src/middleend/passes/merge_permute_stages.cpp
Outdated
Show resolved
Hide resolved
ghost
reviewed
Jun 4, 2020
inference-engine/src/vpu/graph_transformer/src/middleend/passes/merge_permute_stages.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/vpu/graph_transformer/src/middleend/passes/merge_permute_stages.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/vpu/graph_transformer/src/middleend/passes/merge_permute_stages.cpp
Outdated
Show resolved
Hide resolved
Contributor
|
Check other conversations and reply to it, please, also feel free to resolve conversations. Also need to rebase branch to latest master and rerun internal CI. |
ghost
approved these changes
Jun 8, 2020
Contributor
|
Please, let me know when internal validation will end. |
rvyunov
approved these changes
Jun 10, 2020
admitriev-gna
pushed a commit
to admitriev-gna/openvino
that referenced
this pull request
Jun 11, 2020
* Transformation to eliminate trivial permute * Minor changes in unit tests * Replace trivial permutation with copy if input and output dims is equal * Fix mergePermuteStages tests * Small changes in the loop * Add const modifier, change dimsVector type to SizeVector * Change loop condition, rename valiable * To reverse dimsVector
redradist
pushed a commit
to redradist/openvino
that referenced
this pull request
Oct 6, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#-28950
Changes: