[IE][VPU]: Enables Loop Support#4157
Merged
Maxim-Doronin merged 6 commits intoopenvinotoolkit:masterfrom Feb 5, 2021
ggladilov:vpu/gg/enables-loop-support
Merged
[IE][VPU]: Enables Loop Support#4157Maxim-Doronin merged 6 commits intoopenvinotoolkit:masterfrom ggladilov:vpu/gg/enables-loop-support
Maxim-Doronin merged 6 commits intoopenvinotoolkit:masterfrom
ggladilov:vpu/gg/enables-loop-support
Conversation
inference-engine/src/legacy_api/src/convert_function_to_cnn_network.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/vpu/common/src/ngraph/operations/static_shape_loop.cpp
Outdated
Show resolved
Hide resolved
inference-engine/src/vpu/graph_transformer/src/stages/tensor_iterator.cpp
Outdated
Show resolved
Hide resolved
Signed-off-by: Gladilov, Gleb <gleb.gladilov@intel.com>
Signed-off-by: Gladilov, Gleb <gleb.gladilov@intel.com>
Signed-off-by: Gladilov, Gleb <gleb.gladilov@intel.com>
Signed-off-by: Gladilov, Gleb <gleb.gladilov@intel.com>
In general approach is to keep index of data which shape contains dimension of length equal to iterations count. Current version is fixed to use batch of such dimension in shape. Previous version index for input data object for both LoopStart and LoopEnd. While for LoopStart it is correct behavior, for LoopEnd it is wrong. The reason is LoopStart has "big" data on input and single slice on output, LoopEnd has slice on input and "big" data on output. As a result, LoopEnd saw iterations count as 1 (taking batch from slice instead of complete chunk) in dynamic case and copied each iteration result to first slice of output leaving all others uninitialized. Signed-off-by: Gladilov, Gleb <gleb.gladilov@intel.com>
Signed-off-by: Gladilov, Gleb <gleb.gladilov@intel.com>
andrejsokolov
approved these changes
Feb 5, 2021
Maxim-Doronin
approved these changes
Feb 5, 2021
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
6 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.
Descirption
StaticShapeLoop-> CNN conversion asTensorIteratorvisit_attributesAPI forStaticShapeLoopLoopsupport inTensorIteratorparserLoopStart/LoopEndserializationLoopEndLoopfixesTask
#-45342