Skip to content
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

Remove VPU_MYRIAD_PLATFORM from compile tools #6022

Merged

Conversation

DariaMityagina
Copy link
Contributor

@DariaMityagina DariaMityagina commented Jun 3, 2021

it seems that the part where the mentioned option is used is not needed.

IE_SUPPRESS_DEPRECATED_START
config[VPU_MYRIAD_CONFIG_KEY(PLATFORM)] = "VPU_MYRIAD_2480";
IE_SUPPRESS_DEPRECATED_END

Because it was there to set platform = myriad, which is actually redundant since in both compile_tool and myriad_compile device type is defined using

ie.LoadNetwork(network, FLAGS_d, configure()); where FLAGS_d = "MYRIAD" or
ie.LoadNetwork(network, "MYRIAD", configure(FLAGS_c, FLAGS_m));

Without this option model is compiled as well.

Ticket: 48398

@DariaMityagina DariaMityagina requested a review from a team June 3, 2021 12:30
@DariaMityagina DariaMityagina requested a review from a team as a code owner June 3, 2021 12:30
@openvino-pushbot openvino-pushbot added category: inference OpenVINO Runtime library - Inference category: VPU labels Jun 3, 2021
@DariaMityagina DariaMityagina changed the title [WIP] Remove VPU_MYRIAD_PLATFORM from compile tools Remove VPU_MYRIAD_PLATFORM from compile tools Jun 6, 2021
@DariaMityagina DariaMityagina removed the request for review from Maxim-Doronin June 14, 2021 10:15
@DariaMityagina DariaMityagina changed the title Remove VPU_MYRIAD_PLATFORM from compile tools [WIP] Remove VPU_MYRIAD_PLATFORM from compile tools Jun 14, 2021
@DariaMityagina DariaMityagina force-pushed the icv/dm/deprecated_option branch from ee25b13 to 82dfbbe Compare June 15, 2021 09:38
@DariaMityagina DariaMityagina requested a review from a team June 17, 2021 07:44
@DariaMityagina DariaMityagina force-pushed the icv/dm/deprecated_option branch 2 times, most recently from 50dfa3f to ef95552 Compare June 18, 2021 07:23
@DariaMityagina DariaMityagina changed the title [WIP] Remove VPU_MYRIAD_PLATFORM from compile tools Remove VPU_MYRIAD_PLATFORM from compile tools Jun 18, 2021
@DariaMityagina
Copy link
Contributor Author

@AntonDudchenko could you please take a look?

@AntonDudchenko
Copy link
Contributor

LGTM, but please correct the mention of VPU_MYRIAD_2450 key also here:
https://github.com/openvinotoolkit/openvino/blob/master/docs/IE_DG/supported_plugins/MYRIAD.md

@DariaMityagina DariaMityagina requested a review from a team as a code owner June 24, 2021 05:04
@DariaMityagina
Copy link
Contributor Author

LGTM, but please correct the mention of VPU_MYRIAD_2450 key also here:
https://github.com/openvinotoolkit/openvino/blob/master/docs/IE_DG/supported_plugins/MYRIAD.md

done

@DariaMityagina DariaMityagina force-pushed the icv/dm/deprecated_option branch from d7bfc66 to 2206545 Compare June 24, 2021 05:16
@DariaMityagina
Copy link
Contributor Author

@AntonDudchenko do you have other comments?

@DariaMityagina
Copy link
Contributor Author

@Maxim-Doronin could you please take a look?

@DariaMityagina
Copy link
Contributor Author

@Maxim-Doronin can this PR be merged?

@DariaMityagina DariaMityagina force-pushed the icv/dm/deprecated_option branch from 2206545 to 25160d6 Compare June 29, 2021 13:12
@@ -54,8 +54,6 @@ DECLARE_VPU_MYRIAD_CONFIG_KEY(PLATFORM);
* @brief Supported keys definition for VPU_MYRIAD_CONFIG_KEY(PLATFORM) option.
*/
INFERENCE_ENGINE_DEPRECATED("")
DECLARE_VPU_MYRIAD_CONFIG_VALUE(2450);
INFERENCE_ENGINE_DEPRECATED("")
DECLARE_VPU_MYRIAD_CONFIG_VALUE(2480);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why has it remained? Do we need DECLARE_VPU_MYRIAD_CONFIG_KEY(PLATFORM); config if it can take only one value?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, will be removed. Thanks

Copy link
Contributor

@Maxim-Doronin Maxim-Doronin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AntonDudchenko @KorA6, please follow up with my comments and make a review once again

@@ -198,11 +198,6 @@ StageSHAVEsRequirements StageNode::getSHAVEsRequirements() const {
//

// return max for Myriad2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have to remove this comment as well

});
IE_SUPPRESS_DEPRECATED_END

return options;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just return ParsedConfig::getCompileOptions()?

if (availableDevices.empty() && config.platform() != NC_ANY_PLATFORM) {
if (availableDevices.empty()) {
if (config.platform() == NC_ANY_PLATFORM) {
_log->warning("No booted device");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's tell a user that myriad plugin is going to compile model but inference will be unavailable

@@ -376,7 +376,7 @@ void MyriadExecutor::allocateGraph(DevicePtr &device, GraphDesc &graphDesc,
IE_THROW() << "Failed to get output description: " << ncStatusToStr(graphDesc._graphHandle, status);
}

unsigned int fifo_elements = (device->_platform == NC_MYRIAD_2 && executors == 1) ? 4 : 2 * executors;
unsigned int fifo_elements = (executors == 1) ? 4 : 2 * executors;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just 2 * executors

@DariaMityagina DariaMityagina requested a review from ggladilov July 13, 2021 06:53
@DariaMityagina
Copy link
Contributor Author

DariaMityagina commented Jul 13, 2021

@gladilov-gleb could you please take a look? Need one final review to be sure of the changes. Thanks

@DariaMityagina DariaMityagina force-pushed the icv/dm/deprecated_option branch 2 times, most recently from 9840a08 to 6cf98b3 Compare July 16, 2021 11:27
@DariaMityagina DariaMityagina force-pushed the icv/dm/deprecated_option branch from d10af2d to d646d0d Compare July 22, 2021 07:06
@ggladilov
Copy link
Contributor

@gladilov-gleb could you please take a look? Need one final review to be sure of the changes. Thanks

I think it's better to get working (CI is red now) version first. If you think current version is production-ready or you need help - please contact me

@ggladilov
Copy link
Contributor

@Maxim-Doronin @DariaMityagina

Should this request be rebase upon #6176 ?

@DariaMityagina
Copy link
Contributor Author

@gladilov-gleb could you please take a look? Need one final review to be sure of the changes. Thanks

I think it's better to get working (CI is red now) version first. If you think current version is production-ready or you need help - please contact me

Yes, right, I am working on it

@DariaMityagina
Copy link
Contributor Author

@Maxim-Doronin @DariaMityagina

Should this request be rebase upon #6176 ?

I did a rebase after the mentioned PR merge.

@DariaMityagina DariaMityagina force-pushed the icv/dm/deprecated_option branch from d646d0d to 2b61736 Compare July 22, 2021 19:39
@DariaMityagina
Copy link
Contributor Author

@Maxim-Doronin @gladilov-gleb product-configs checks pass. All of the links are in the ticket.

@DariaMityagina DariaMityagina force-pushed the icv/dm/deprecated_option branch 2 times, most recently from ca214c6 to 53e8a71 Compare August 10, 2021 08:57
@DariaMityagina DariaMityagina force-pushed the icv/dm/deprecated_option branch from 53e8a71 to c62aa1f Compare August 16, 2021 09:22
@DariaMityagina
Copy link
Contributor Author

DariaMityagina commented Aug 16, 2021

https://dsp-ci-icv.inn.intel.com/job/MYRIAD/job/IE-MDK/job/manual/job/RunTests-MYRIAD/2404/
Seems to be okay.
Product-config check is in progress.

@azhogov
Copy link

azhogov commented Aug 18, 2021

@azhogov azhogov merged commit ed9b4ad into openvinotoolkit:master Aug 18, 2021
alexander-shchepetov pushed a commit to alexander-shchepetov/openvino that referenced this pull request Sep 5, 2021
ishariko added a commit that referenced this pull request Sep 20, 2021
…ks tests (#7378)

* add multimodel memleaks test runner

* Add support for is_equal_data, get_source_tensor, get_target_tensor methods in ONNX FE API (#6991)

* Change ngraph public api (#6920)

* Moved nGraph function

* Added legacy nGraph function

* Moved export API

* Moved variant

* Added old header for backward compatibility

* Introduce define

* [LPT] LP Transformations refactoring after dynamic shapes support (#6950)

* [LPT] Transformations refactoring after dynamic shapes support

* [LPT] ReshapeTransformation: 2D->2D fix

* [LPT] fixes after review

* [GPU] Fix ScatterNDUpdate unit tests (#7103)

* Deprecate ngraph file utils. Need to have common functions (#7105)

* [GPU] Get rid of memory alloc for input_layout in internal networks (#6897)

* Renamed component name (#7110)

* Propose new MaxPool-8 operation (#5359)

* MaxPool-8: pads_value attribute removal from the operator definition (#7119)

* Remove deprecated option and enable compilation without device (#6022)

* Build openvino wheel package from setup.py (#7091)

* Added ability to build wheel package by executing `setup.py bdist_wheel`

* fix linter issues

* fix formating

* remove blank line

* Support unregistered operations in MO IR Reader (#6837)

* Add support for unregistred operations in MO IR Reader

* Remove commented lines

* Add shapes equality check

* Update comments

* Update groupconv_to_conv function to support case with multiple destinations

* Add ir_data_attrs attribute to restored layers

* Update copy_shape_infer function to new graph api

* Add attribute IE to unsuppurted operations to save their attributes

* Fix wrong attribute name

* Update commentary

* Partially revert updating to new Graph API to fix regression, add appropriate comments

* Update code comments

* Rename copy_shape_infer function and add more comments

* First stage of the AUTO-MULTI merge: redirecting AUTO to the MULTI device plugin (#7037)

* Redirect -d AUTO to MULTI device plugin

Signed-off-by: Shoujiang Ma <[email protected]>

* Modify AUTO tests with MULTI config

Signed-off-by: Shoujiang Ma <[email protected]>

* Fix CI

Signed-off-by: Shoujiang Ma <[email protected]>

* Fix bug: CVS-62424

Signed-off-by: Shoujiang Ma <[email protected]>

* Add some tests for AUTO

Signed-off-by: Shoujiang Ma <[email protected]>

* Add select device logic to MULTI

Signed-off-by: Shoujiang Ma <[email protected]>

* Fix extract device name bug

Signed-off-by: Shoujiang Ma <[email protected]>

* Address reviewer's comment

Signed-off-by: Shoujiang Ma <[email protected]>

* Delete AUTO plugin source code

Signed-off-by: Shoujiang Ma <[email protected]>

* [CPU] Deform. conv. - reference enforced (#6945)

* [CPU] Bump up MKLDNN version to get fix of c26453 warning (#7089)

* OpenVINO ONNX CI Azure - update onnx/models on demand only (#7125)

* [CPU] Added improvements for StridedSlice (#6658)

* Removal of FusedOp inheritance leftovers (#7113)

* Remove FusedOp from v0::Gelu

* Update v0::Gelu NGRAPH_RTTI_DECLARATION

* Enable gelu type_prop tests

* Remove FusedOp from v0::MVN

* Remove FusedOp from HardSigmoid

* Remove FusedOp from LSTMSequence

* Remove supress deprecated

* Add missed NGRAPH_OP_SCOPE to v0 Gelu and HardSigmoid

* Move ngraph::element::Type to ov namespace (#7124)

* Moved ngraph::Type -> ov::Type

* Revert original files

* [GNA] Fix order of SwapMatMulInput transformations (#7137)

* Moved Dimension, PartialShape, Interval, Rank to ov namespace (#7136)

* Mo implementation for If with tf extractor (#6662)

* Add tf2.x impl for If

* Fix ir_engine

* Fix opset

* Fix BOM file

* Added new test

* Fix comments

* Add subgraph_utils

* Fix comments

* Fix transform

* code refactoring

* Fix description

* rewrite support for empty tensor in if

* added onnx extractor

* delete onnx_if

* fix bug with fake_outputs

* Fix test

* Fix control_flow and fix commentaries

* create method results_mapping_and_finding_fake_outputs(output_nodes_in_subgraph,

* [GPU] Fixed 'assigned to self' error in loop_inst.h (#7126)

* [GPU] Fix build for gcc 10 (#7142)

* [GNA] Set input scale factors for imported model (#7139)

* add doc:'Paddle_Support.md' (#7122)

* add doc:'Paddle_Support.md'

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <[email protected]>

* Apply suggestions from code review

* Update docs/IE_DG/Paddle_Support.md

Co-authored-by: Tatiana Savina <[email protected]>

Co-authored-by: Tatiana Savina <[email protected]>

* Remove local configs and it's copying to bin/ for stress tests (#7131)

* Moved DEPRECATION macro and ITT domains to ov namespace (#7153)

* Moved DEPRECATION macro and ITT domains to ov namespace

* Fixed code style

* Enable NormalizeL2Fusion and LeakyReluFusion inside MOC (#7096)

* Enable NormalizeL2Fusion inside MOC

* Fix NormalizewL2 decomposition (KeepDims=True)

* Add support for ONNX Crop operator (#6956)

* Review/update spec for NotEqual operation (#6797)

* Hiding the problem, Validate() changes 'function'

* Review/update spec for NotEqual operation

* Remove unnecessary edits not related to the ticket

* Removing the extra word binary for the short description

* Re-writing detailed description

* Correcting punctuation docs/ops/comparison/NotEqual_1.md

Co-authored-by: Tatiana Savina <[email protected]>

* Specifying auto_broadcast in the short description is similar to Equal spec

* The range of values for auto_brodcast is similar to Equal spec and includes the missing pdpd

Co-authored-by: Tatiana Savina <[email protected]>

* Moved DiscreteTypeInfo to ov namespace (#7127)

* Moved DiscreteTypeInfo to new opset

* Revert old header

* Fixed code style

* Revise CTCLoss OP (#6953)

* Add visitor test to CTCLoss

* Add CTC Loss SSLT

* Add CTC Loss template tests

* Use ngraph rtti macros

* Code style fix

* Enable PriorBoxClustered tests (#7078)

* CumSum spec revision (#6966)

* Update detailed description

* Update exclusive attribute description

* Update Inputs/Outpu description

* Update types

* Update descriptions

* Update data input rank info

* Added common.hpp file with aliases (#7158)

* CumSum reference implementation revision (#6915)

* New CumSum implementation init

* Unified ndim approach

* Move transpose to separate function

* Move transpose to original to separate function

* Move slice_count calculation to function

* Negative axes support

* Refactor redundant copy

* Changed copy to move

* Temp more backend tests

* Add const to shape arg

* Use span for slices calculation

* Remove unused headers

* CumSum new ref tests

* Add more ref tests

* Add all cumsum modes ref tests

* new optimized cum_sum reference

* Add reverse mode

* Optimized cumsum ref

* Remove deprecated cumsum backend tests

* Add more CumSum reference tests

* Simplify CumSum shared layer tests SetUp

* Replace auto to size_t in loop

* Change static_cast to T{}

* [LPT] MarkupCanBeQuantized: handled unsupported concat (#7045)

* [LPT] MarkupCanBeQuantized: added check on unsupported concat

* [LPT] ConcatTransformation: added test-case with unsupported concat and convolution

* [LPT] added test on rtInfo check for unsupported concat

* [LPT] ConcatTransformation: added test-case with unsupported axis to plugin tests

* CVS-56144 Enable all OMZ scope (#7084)

* Install layer tests with CMake (#6892)

* add CMakeLists.txt

* add copyright docstring

* add newline after copyright

* set target name

* change TARGET to DIRECTORY

* Rename layer tests dir to avoid name conflict

* cmakelists.txt final version

* Change destination to tests\layer_tests_openvino

* Add cmake_minimum_required to CMakeLists.txt

* Update CMakeLists.txt

* ReverseSequence specification refactored (#7112)

* ReverseSequence specification refactored

* Change attribute description to avoid confusion

* Allow seq_lenghts input to be of floating-point precision

* MemCheck add INT8 models to pre-commit (#7166)

* updated desktop configs with int8 models

* updated desktop reference configs with actual values

* added commit comments

* parametrize proxy (#7174)

* Updated list of supported operations. (#6981)

* Updated list of supported layers.

* Removed Crop, softsign from Kaldi list.

* Updated limitations.

* Corrected limitations.

* Updated limitations.

* Added Einsum, corrected Where.

* Apply suggestions from code review

Co-authored-by: Anastasiya Ageeva <[email protected]>

Co-authored-by: Anastasiya Ageeva <[email protected]>

* [MO] turn on MarkSubGraphsWithCorrectLayout for TF NCHW (#7150)

* turned on MarkingSubgraphsWithCorrectLayout for TF NCHW

* restricted MarkSubgraphsWithCorrectLayout.py only to TF

* added comments why need to MarkSubgraphsWithCorrectLayout even for TF NCHW models

* [CPU][TESTS][LPT] MatMulTransformations test-cases removed from skip config (#7181)

* Fixed ngraph_onnx_importer compatibility target creation for older cmake (3.10) (#7176)

* [GPU] Fix clBuildProgram failure with ssd_mobilnet_v1_coco and batch=256 (#7121)

* Fix v0::MVN default constructor (#7175)

* [GPU] Fixes for correct MultiDevice plugin and inference request behavior (#7161)

* Fix op category section in operations spec (#7130)

* add ngraph::pass::LSTMCellDecomposition as mandatory (#7028)

* add ngraph::pass::LSTMCellDecomposition as mandatory

* move LSTMCellDecomposition just after CommonOptimizations before all convert opset transformations

* code review fixes: add flag that prevents some legacy transformations if their ngraph-based analogues were executed

* remove isNgraphPassesUsed from  ModelQuantizer

* cleanups

* [IE] Convert to unsigned NMS:0 ->Gather path (#6474)

* inserted Convert to unsigned

* moved declarations from hpp into cpp, specification corrected

* added static const modifier

* updated convert specification

* minor corrections

* split into 3 passes(Init, Propogate, Update), renamed final pass to ConvertNmsGatherPathToUnsigned

* added description why transformation is needed

* added matcher for several NMS versions, removed TRANSFORMATIONS_API macros from cpp

* applied comments:
- used GraphRewrite instead of FunctionPass
- simplified some expressions
- corrected case when Converts output goes to multiple nodes
- added to MOC transformations
- other minor corrections

* removed redundant namespace prefixes

* fixed #include <ngraph/pass/graph_rewrite.hpp>

* removed matcher_scope, debug code, and redundant dynamic_cast

* [nG] [IE] use GatherBase in negative indices resolver (#7145)

* updated pattern matcher into GatherBase in negative indices resolver, so that it is triggered in all versions of operation

* copy_runtime_info fix

* added constant folding

* Nested loop (#6710)

* initial changes to support nested loop

* fixed issues

* fixed nested loop extraction

* added comments

* removed unneeded comments

* review fix

* added tests

* turned off loop tests on GPU

* set xfail for TF tests

* removed TF test to move it in another repo

* fix typo in comment

* move duplicated code to separate functions;
added asserts

* add function for onnx constant creation;
add function to create body of loop
add comments to test

* move main change for nested loop to separate function

* install necessary dirs for tests (#7044)

* install necessary dirs to tests

* rem RUNTIME from install step

* fix paths

* fix install paths

* fix install paths: add destination dirs

* add pandas

* fix requirements conflict - change pytest version to ~5

* remove comment from requirements.txt

* upd numpy version

* Added openvino infer request API (#7151)

* [CPU] Removed eltwise overhead on execution stage (#6760)

* [GNA] For similar records, the pattern length was increased to 4  in the algorithm for determining infinite cycles. (#7165)

* for similar records, the pattern length was increased to 4

* Added comments

* [CPU] Enable direct copy implementation for u8->u8 reorder. (#7043)

* [CPU] Fix not expected No-Preprocess Exception with RGB to BGR conversion (#6954)

* [CPU] Avoid inserting additional transpose + reorder after RNN node. (#5921)

* [MO] Replacing StridedSlice with Squeeze/Unsqueeze (#6693)

* added reinterp_shape parameter to tf ss extractor

* removed reinterp_shape

* added transformation to replace ss

* updated bom

* fix for e2e tests

* updated a case when shrink_axis_mask and new_axis_mask are both initialized

* unittests

* added comments

* updated graph_condition

* comments resolving

* updated the case, when shrink_axis_mask and new_axis_mask are both initialized

* added layer tests for squeeze/unsqueeze cases

* remove case when shrink and new axis masks are both set

* [VPU] Added ConvertGather7ToGather1 pass to frontend (#7183)

This pr adds ConvertGather7ToGather1 pass to frontend before MergeGatherGatherElements pass, to make it so that when MergeGatherGatherElements is ran, any v7::Gather will be replaced with v1::Gather

* [MO] Add transformation for single CTCGreedyDecoder operation (#7023)

* Add transformation for single CTCGreedyDecoder operation

* Fix style in op specification

* Update transformation logic

* refactor old tests and add tests for new transformation

* Move tf specific front transformations to tf folder

* Update transformation logic and comments

* Add run_after function and update comments

* Add output_sparse_format attribute to extractor

* Update transformation conditions and tests

* Fix incorrect comment

* Move sparse_to_dense_replacer to front/tf folder to fix problems with class registration

* Update import

* Update output ports handling in transformation

* Update test

* Fix BOM file

* Update pattern for ctcloss transformation

* Fix and refactor tests for ctcloss transform

* Update transformation conditions

* Add support opset11 for gemm normalizer (#6733)

* Add support opset11 for gemm normolizer

* Add layer test for gemm opset 11

* Fix layer test

* Fix layer test

* Refactoring according to code review

* Fix

* Update biases norm

* Refactoring matmul norm

* Fix accoding to review

* Fix alpha parameter

* Fix variable naming

* Refactoring according to code review

* Add support for ONNX RandomUniform and RandomUniformLike ops (#7190)

* remove adaptive pool2d shape check in ngraph paddle frontend (#7074)

* remove adaptive pool2d shape check in ngraph paddle frontend

* add ngraph paddle frontend dynamic pool2d test

* Revise ReverseSequence reference implementation (#7117)

* ReverseSequence ngraph op shell revision with type_prop tests

* Add attribute count check in visitor test

* Refactor backend tests to template plugin test with reference values

* Rename cpu SLT instances

* Add op to list of trusted operations

* Rewrite validation check for input type due to backward compatibility

* Reference implementation speed up by replacing index function call of CoordinateTransform by precalculated strides

* Moved attribute_adapter, attribute_visitor files to ov namespave (#7179)

* Fixed nGraph build

* Fixed nGraph unit tests

* Fixed func tests

* Fix some operators

* Fixed build

* Try to fix specialization in different namespace

* Try to fix build

* Fixed element_type

* memleaks multimodel supporting

* revert rebase mistake

* add newline at the end of config file

* fix log messages

* refine memelaks test case class

* remove temporary decision designed to  save in memory pipeline functions parameters

* code consistency

* rework example of new memleak tests config format

* oop in testcases

* mistype

* set num of iterations in example test config to previous value

* add multiproc stress unit tests

* Add more cases

* remove unique_ptr test objects saving logic

* switch memleak test configs to new format

* swith weekly memleak test config to new format

* Clarify new get_testdata script arg

* clang-format

* wrong changes

* Add docstring to generateTestsParamsMemLeaks()

* add explanation what update_item_for_name() doing

* Autodetect stress framework while parsing models

* adjust the wording

* Shorten test cases names

* fix get_testdata for memcheck tests

Co-authored-by: Mateusz Bencer <[email protected]>
Co-authored-by: Ilya Churaev <[email protected]>
Co-authored-by: Vladislav Golubev <[email protected]>
Co-authored-by: Sergey Shlyapnikov <[email protected]>
Co-authored-by: Vladimir Paramuzov <[email protected]>
Co-authored-by: Ilya Sharikov <[email protected]>
Co-authored-by: Michał Karzyński <[email protected]>
Co-authored-by: Tomasz Dołbniak <[email protected]>
Co-authored-by: Daria Mityagina <[email protected]>
Co-authored-by: Sergey Lyubimtsev <[email protected]>
Co-authored-by: Anton Chetverikov <[email protected]>
Co-authored-by: Shoujiang Ma <[email protected]>
Co-authored-by: Yury Gaydaychuk <[email protected]>
Co-authored-by: Vitaliy Urusovskij <[email protected]>
Co-authored-by: Rafal Blaczkowski <[email protected]>
Co-authored-by: Alexandra Sidorova <[email protected]>
Co-authored-by: Katarzyna Mitrus <[email protected]>
Co-authored-by: Elizaveta Lobanova <[email protected]>
Co-authored-by: Eugeny Volosenkov <[email protected]>
Co-authored-by: Paul Youngsoo Ahn <[email protected]>
Co-authored-by: Mikhail Ryzhov <[email protected]>
Co-authored-by: Liu Bo <[email protected]>
Co-authored-by: Tatiana Savina <[email protected]>
Co-authored-by: Olesya Martinyuk <[email protected]>
Co-authored-by: Gleb Kazantaev <[email protected]>
Co-authored-by: Nikita Semaev <[email protected]>
Co-authored-by: Bartosz Lesniewski <[email protected]>
Co-authored-by: Anton Pankratv <[email protected]>
Co-authored-by: Anastasiia Urlapova <[email protected]>
Co-authored-by: Gabriele Galiero Casay <[email protected]>
Co-authored-by: Anastasia Popova <[email protected]>
Co-authored-by: Anastasiya Ageeva <[email protected]>
Co-authored-by: Pavel Esir <[email protected]>
Co-authored-by: Maksim Shabunin <[email protected]>
Co-authored-by: Andrew Kwangwoong Park <[email protected]>
Co-authored-by: Mikhail Letavin <[email protected]>
Co-authored-by: Evgeny Kotov <[email protected]>
Co-authored-by: Svetlana Dolinina <[email protected]>
Co-authored-by: Victor Kuznetsov <[email protected]>
Co-authored-by: Dmitrii Khurtin <[email protected]>
Co-authored-by: Ivan Novoselov <[email protected]>
Co-authored-by: Aleksandr Pertovsky <[email protected]>
Co-authored-by: Nikolay Shchegolev <[email protected]>
Co-authored-by: Yegor Kruglov <[email protected]>
Co-authored-by: Polina Brzezinskaya <[email protected]>
Co-authored-by: iliya mironov <[email protected]>
Co-authored-by: mei, yang <[email protected]>
akuporos pushed a commit to akuporos/openvino that referenced this pull request Sep 29, 2021
…ks tests (openvinotoolkit#7378)

* add multimodel memleaks test runner

* Add support for is_equal_data, get_source_tensor, get_target_tensor methods in ONNX FE API (openvinotoolkit#6991)

* Change ngraph public api (openvinotoolkit#6920)

* Moved nGraph function

* Added legacy nGraph function

* Moved export API

* Moved variant

* Added old header for backward compatibility

* Introduce define

* [LPT] LP Transformations refactoring after dynamic shapes support (openvinotoolkit#6950)

* [LPT] Transformations refactoring after dynamic shapes support

* [LPT] ReshapeTransformation: 2D->2D fix

* [LPT] fixes after review

* [GPU] Fix ScatterNDUpdate unit tests (openvinotoolkit#7103)

* Deprecate ngraph file utils. Need to have common functions (openvinotoolkit#7105)

* [GPU] Get rid of memory alloc for input_layout in internal networks (openvinotoolkit#6897)

* Renamed component name (openvinotoolkit#7110)

* Propose new MaxPool-8 operation (openvinotoolkit#5359)

* MaxPool-8: pads_value attribute removal from the operator definition (openvinotoolkit#7119)

* Remove deprecated option and enable compilation without device (openvinotoolkit#6022)

* Build openvino wheel package from setup.py (openvinotoolkit#7091)

* Added ability to build wheel package by executing `setup.py bdist_wheel`

* fix linter issues

* fix formating

* remove blank line

* Support unregistered operations in MO IR Reader (openvinotoolkit#6837)

* Add support for unregistred operations in MO IR Reader

* Remove commented lines

* Add shapes equality check

* Update comments

* Update groupconv_to_conv function to support case with multiple destinations

* Add ir_data_attrs attribute to restored layers

* Update copy_shape_infer function to new graph api

* Add attribute IE to unsuppurted operations to save their attributes

* Fix wrong attribute name

* Update commentary

* Partially revert updating to new Graph API to fix regression, add appropriate comments

* Update code comments

* Rename copy_shape_infer function and add more comments

* First stage of the AUTO-MULTI merge: redirecting AUTO to the MULTI device plugin (openvinotoolkit#7037)

* Redirect -d AUTO to MULTI device plugin

Signed-off-by: Shoujiang Ma <[email protected]>

* Modify AUTO tests with MULTI config

Signed-off-by: Shoujiang Ma <[email protected]>

* Fix CI

Signed-off-by: Shoujiang Ma <[email protected]>

* Fix bug: CVS-62424

Signed-off-by: Shoujiang Ma <[email protected]>

* Add some tests for AUTO

Signed-off-by: Shoujiang Ma <[email protected]>

* Add select device logic to MULTI

Signed-off-by: Shoujiang Ma <[email protected]>

* Fix extract device name bug

Signed-off-by: Shoujiang Ma <[email protected]>

* Address reviewer's comment

Signed-off-by: Shoujiang Ma <[email protected]>

* Delete AUTO plugin source code

Signed-off-by: Shoujiang Ma <[email protected]>

* [CPU] Deform. conv. - reference enforced (openvinotoolkit#6945)

* [CPU] Bump up MKLDNN version to get fix of c26453 warning (openvinotoolkit#7089)

* OpenVINO ONNX CI Azure - update onnx/models on demand only (openvinotoolkit#7125)

* [CPU] Added improvements for StridedSlice (openvinotoolkit#6658)

* Removal of FusedOp inheritance leftovers (openvinotoolkit#7113)

* Remove FusedOp from v0::Gelu

* Update v0::Gelu NGRAPH_RTTI_DECLARATION

* Enable gelu type_prop tests

* Remove FusedOp from v0::MVN

* Remove FusedOp from HardSigmoid

* Remove FusedOp from LSTMSequence

* Remove supress deprecated

* Add missed NGRAPH_OP_SCOPE to v0 Gelu and HardSigmoid

* Move ngraph::element::Type to ov namespace (openvinotoolkit#7124)

* Moved ngraph::Type -> ov::Type

* Revert original files

* [GNA] Fix order of SwapMatMulInput transformations (openvinotoolkit#7137)

* Moved Dimension, PartialShape, Interval, Rank to ov namespace (openvinotoolkit#7136)

* Mo implementation for If with tf extractor (openvinotoolkit#6662)

* Add tf2.x impl for If

* Fix ir_engine

* Fix opset

* Fix BOM file

* Added new test

* Fix comments

* Add subgraph_utils

* Fix comments

* Fix transform

* code refactoring

* Fix description

* rewrite support for empty tensor in if

* added onnx extractor

* delete onnx_if

* fix bug with fake_outputs

* Fix test

* Fix control_flow and fix commentaries

* create method results_mapping_and_finding_fake_outputs(output_nodes_in_subgraph,

* [GPU] Fixed 'assigned to self' error in loop_inst.h (openvinotoolkit#7126)

* [GPU] Fix build for gcc 10 (openvinotoolkit#7142)

* [GNA] Set input scale factors for imported model (openvinotoolkit#7139)

* add doc:'Paddle_Support.md' (openvinotoolkit#7122)

* add doc:'Paddle_Support.md'

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <[email protected]>

* Apply suggestions from code review

* Update docs/IE_DG/Paddle_Support.md

Co-authored-by: Tatiana Savina <[email protected]>

Co-authored-by: Tatiana Savina <[email protected]>

* Remove local configs and it's copying to bin/ for stress tests (openvinotoolkit#7131)

* Moved DEPRECATION macro and ITT domains to ov namespace (openvinotoolkit#7153)

* Moved DEPRECATION macro and ITT domains to ov namespace

* Fixed code style

* Enable NormalizeL2Fusion and LeakyReluFusion inside MOC (openvinotoolkit#7096)

* Enable NormalizeL2Fusion inside MOC

* Fix NormalizewL2 decomposition (KeepDims=True)

* Add support for ONNX Crop operator (openvinotoolkit#6956)

* Review/update spec for NotEqual operation (openvinotoolkit#6797)

* Hiding the problem, Validate() changes 'function'

* Review/update spec for NotEqual operation

* Remove unnecessary edits not related to the ticket

* Removing the extra word binary for the short description

* Re-writing detailed description

* Correcting punctuation docs/ops/comparison/NotEqual_1.md

Co-authored-by: Tatiana Savina <[email protected]>

* Specifying auto_broadcast in the short description is similar to Equal spec

* The range of values for auto_brodcast is similar to Equal spec and includes the missing pdpd

Co-authored-by: Tatiana Savina <[email protected]>

* Moved DiscreteTypeInfo to ov namespace (openvinotoolkit#7127)

* Moved DiscreteTypeInfo to new opset

* Revert old header

* Fixed code style

* Revise CTCLoss OP (openvinotoolkit#6953)

* Add visitor test to CTCLoss

* Add CTC Loss SSLT

* Add CTC Loss template tests

* Use ngraph rtti macros

* Code style fix

* Enable PriorBoxClustered tests (openvinotoolkit#7078)

* CumSum spec revision (openvinotoolkit#6966)

* Update detailed description

* Update exclusive attribute description

* Update Inputs/Outpu description

* Update types

* Update descriptions

* Update data input rank info

* Added common.hpp file with aliases (openvinotoolkit#7158)

* CumSum reference implementation revision (openvinotoolkit#6915)

* New CumSum implementation init

* Unified ndim approach

* Move transpose to separate function

* Move transpose to original to separate function

* Move slice_count calculation to function

* Negative axes support

* Refactor redundant copy

* Changed copy to move

* Temp more backend tests

* Add const to shape arg

* Use span for slices calculation

* Remove unused headers

* CumSum new ref tests

* Add more ref tests

* Add all cumsum modes ref tests

* new optimized cum_sum reference

* Add reverse mode

* Optimized cumsum ref

* Remove deprecated cumsum backend tests

* Add more CumSum reference tests

* Simplify CumSum shared layer tests SetUp

* Replace auto to size_t in loop

* Change static_cast to T{}

* [LPT] MarkupCanBeQuantized: handled unsupported concat (openvinotoolkit#7045)

* [LPT] MarkupCanBeQuantized: added check on unsupported concat

* [LPT] ConcatTransformation: added test-case with unsupported concat and convolution

* [LPT] added test on rtInfo check for unsupported concat

* [LPT] ConcatTransformation: added test-case with unsupported axis to plugin tests

* CVS-56144 Enable all OMZ scope (openvinotoolkit#7084)

* Install layer tests with CMake (openvinotoolkit#6892)

* add CMakeLists.txt

* add copyright docstring

* add newline after copyright

* set target name

* change TARGET to DIRECTORY

* Rename layer tests dir to avoid name conflict

* cmakelists.txt final version

* Change destination to tests\layer_tests_openvino

* Add cmake_minimum_required to CMakeLists.txt

* Update CMakeLists.txt

* ReverseSequence specification refactored (openvinotoolkit#7112)

* ReverseSequence specification refactored

* Change attribute description to avoid confusion

* Allow seq_lenghts input to be of floating-point precision

* MemCheck add INT8 models to pre-commit (openvinotoolkit#7166)

* updated desktop configs with int8 models

* updated desktop reference configs with actual values

* added commit comments

* parametrize proxy (openvinotoolkit#7174)

* Updated list of supported operations. (openvinotoolkit#6981)

* Updated list of supported layers.

* Removed Crop, softsign from Kaldi list.

* Updated limitations.

* Corrected limitations.

* Updated limitations.

* Added Einsum, corrected Where.

* Apply suggestions from code review

Co-authored-by: Anastasiya Ageeva <[email protected]>

Co-authored-by: Anastasiya Ageeva <[email protected]>

* [MO] turn on MarkSubGraphsWithCorrectLayout for TF NCHW (openvinotoolkit#7150)

* turned on MarkingSubgraphsWithCorrectLayout for TF NCHW

* restricted MarkSubgraphsWithCorrectLayout.py only to TF

* added comments why need to MarkSubgraphsWithCorrectLayout even for TF NCHW models

* [CPU][TESTS][LPT] MatMulTransformations test-cases removed from skip config (openvinotoolkit#7181)

* Fixed ngraph_onnx_importer compatibility target creation for older cmake (3.10) (openvinotoolkit#7176)

* [GPU] Fix clBuildProgram failure with ssd_mobilnet_v1_coco and batch=256 (openvinotoolkit#7121)

* Fix v0::MVN default constructor (openvinotoolkit#7175)

* [GPU] Fixes for correct MultiDevice plugin and inference request behavior (openvinotoolkit#7161)

* Fix op category section in operations spec (openvinotoolkit#7130)

* add ngraph::pass::LSTMCellDecomposition as mandatory (openvinotoolkit#7028)

* add ngraph::pass::LSTMCellDecomposition as mandatory

* move LSTMCellDecomposition just after CommonOptimizations before all convert opset transformations

* code review fixes: add flag that prevents some legacy transformations if their ngraph-based analogues were executed

* remove isNgraphPassesUsed from  ModelQuantizer

* cleanups

* [IE] Convert to unsigned NMS:0 ->Gather path (openvinotoolkit#6474)

* inserted Convert to unsigned

* moved declarations from hpp into cpp, specification corrected

* added static const modifier

* updated convert specification

* minor corrections

* split into 3 passes(Init, Propogate, Update), renamed final pass to ConvertNmsGatherPathToUnsigned

* added description why transformation is needed

* added matcher for several NMS versions, removed TRANSFORMATIONS_API macros from cpp

* applied comments:
- used GraphRewrite instead of FunctionPass
- simplified some expressions
- corrected case when Converts output goes to multiple nodes
- added to MOC transformations
- other minor corrections

* removed redundant namespace prefixes

* fixed #include <ngraph/pass/graph_rewrite.hpp>

* removed matcher_scope, debug code, and redundant dynamic_cast

* [nG] [IE] use GatherBase in negative indices resolver (openvinotoolkit#7145)

* updated pattern matcher into GatherBase in negative indices resolver, so that it is triggered in all versions of operation

* copy_runtime_info fix

* added constant folding

* Nested loop (openvinotoolkit#6710)

* initial changes to support nested loop

* fixed issues

* fixed nested loop extraction

* added comments

* removed unneeded comments

* review fix

* added tests

* turned off loop tests on GPU

* set xfail for TF tests

* removed TF test to move it in another repo

* fix typo in comment

* move duplicated code to separate functions;
added asserts

* add function for onnx constant creation;
add function to create body of loop
add comments to test

* move main change for nested loop to separate function

* install necessary dirs for tests (openvinotoolkit#7044)

* install necessary dirs to tests

* rem RUNTIME from install step

* fix paths

* fix install paths

* fix install paths: add destination dirs

* add pandas

* fix requirements conflict - change pytest version to ~5

* remove comment from requirements.txt

* upd numpy version

* Added openvino infer request API (openvinotoolkit#7151)

* [CPU] Removed eltwise overhead on execution stage (openvinotoolkit#6760)

* [GNA] For similar records, the pattern length was increased to 4  in the algorithm for determining infinite cycles. (openvinotoolkit#7165)

* for similar records, the pattern length was increased to 4

* Added comments

* [CPU] Enable direct copy implementation for u8->u8 reorder. (openvinotoolkit#7043)

* [CPU] Fix not expected No-Preprocess Exception with RGB to BGR conversion (openvinotoolkit#6954)

* [CPU] Avoid inserting additional transpose + reorder after RNN node. (openvinotoolkit#5921)

* [MO] Replacing StridedSlice with Squeeze/Unsqueeze (openvinotoolkit#6693)

* added reinterp_shape parameter to tf ss extractor

* removed reinterp_shape

* added transformation to replace ss

* updated bom

* fix for e2e tests

* updated a case when shrink_axis_mask and new_axis_mask are both initialized

* unittests

* added comments

* updated graph_condition

* comments resolving

* updated the case, when shrink_axis_mask and new_axis_mask are both initialized

* added layer tests for squeeze/unsqueeze cases

* remove case when shrink and new axis masks are both set

* [VPU] Added ConvertGather7ToGather1 pass to frontend (openvinotoolkit#7183)

This pr adds ConvertGather7ToGather1 pass to frontend before MergeGatherGatherElements pass, to make it so that when MergeGatherGatherElements is ran, any v7::Gather will be replaced with v1::Gather

* [MO] Add transformation for single CTCGreedyDecoder operation (openvinotoolkit#7023)

* Add transformation for single CTCGreedyDecoder operation

* Fix style in op specification

* Update transformation logic

* refactor old tests and add tests for new transformation

* Move tf specific front transformations to tf folder

* Update transformation logic and comments

* Add run_after function and update comments

* Add output_sparse_format attribute to extractor

* Update transformation conditions and tests

* Fix incorrect comment

* Move sparse_to_dense_replacer to front/tf folder to fix problems with class registration

* Update import

* Update output ports handling in transformation

* Update test

* Fix BOM file

* Update pattern for ctcloss transformation

* Fix and refactor tests for ctcloss transform

* Update transformation conditions

* Add support opset11 for gemm normalizer (openvinotoolkit#6733)

* Add support opset11 for gemm normolizer

* Add layer test for gemm opset 11

* Fix layer test

* Fix layer test

* Refactoring according to code review

* Fix

* Update biases norm

* Refactoring matmul norm

* Fix accoding to review

* Fix alpha parameter

* Fix variable naming

* Refactoring according to code review

* Add support for ONNX RandomUniform and RandomUniformLike ops (openvinotoolkit#7190)

* remove adaptive pool2d shape check in ngraph paddle frontend (openvinotoolkit#7074)

* remove adaptive pool2d shape check in ngraph paddle frontend

* add ngraph paddle frontend dynamic pool2d test

* Revise ReverseSequence reference implementation (openvinotoolkit#7117)

* ReverseSequence ngraph op shell revision with type_prop tests

* Add attribute count check in visitor test

* Refactor backend tests to template plugin test with reference values

* Rename cpu SLT instances

* Add op to list of trusted operations

* Rewrite validation check for input type due to backward compatibility

* Reference implementation speed up by replacing index function call of CoordinateTransform by precalculated strides

* Moved attribute_adapter, attribute_visitor files to ov namespave (openvinotoolkit#7179)

* Fixed nGraph build

* Fixed nGraph unit tests

* Fixed func tests

* Fix some operators

* Fixed build

* Try to fix specialization in different namespace

* Try to fix build

* Fixed element_type

* memleaks multimodel supporting

* revert rebase mistake

* add newline at the end of config file

* fix log messages

* refine memelaks test case class

* remove temporary decision designed to  save in memory pipeline functions parameters

* code consistency

* rework example of new memleak tests config format

* oop in testcases

* mistype

* set num of iterations in example test config to previous value

* add multiproc stress unit tests

* Add more cases

* remove unique_ptr test objects saving logic

* switch memleak test configs to new format

* swith weekly memleak test config to new format

* Clarify new get_testdata script arg

* clang-format

* wrong changes

* Add docstring to generateTestsParamsMemLeaks()

* add explanation what update_item_for_name() doing

* Autodetect stress framework while parsing models

* adjust the wording

* Shorten test cases names

* fix get_testdata for memcheck tests

Co-authored-by: Mateusz Bencer <[email protected]>
Co-authored-by: Ilya Churaev <[email protected]>
Co-authored-by: Vladislav Golubev <[email protected]>
Co-authored-by: Sergey Shlyapnikov <[email protected]>
Co-authored-by: Vladimir Paramuzov <[email protected]>
Co-authored-by: Ilya Sharikov <[email protected]>
Co-authored-by: Michał Karzyński <[email protected]>
Co-authored-by: Tomasz Dołbniak <[email protected]>
Co-authored-by: Daria Mityagina <[email protected]>
Co-authored-by: Sergey Lyubimtsev <[email protected]>
Co-authored-by: Anton Chetverikov <[email protected]>
Co-authored-by: Shoujiang Ma <[email protected]>
Co-authored-by: Yury Gaydaychuk <[email protected]>
Co-authored-by: Vitaliy Urusovskij <[email protected]>
Co-authored-by: Rafal Blaczkowski <[email protected]>
Co-authored-by: Alexandra Sidorova <[email protected]>
Co-authored-by: Katarzyna Mitrus <[email protected]>
Co-authored-by: Elizaveta Lobanova <[email protected]>
Co-authored-by: Eugeny Volosenkov <[email protected]>
Co-authored-by: Paul Youngsoo Ahn <[email protected]>
Co-authored-by: Mikhail Ryzhov <[email protected]>
Co-authored-by: Liu Bo <[email protected]>
Co-authored-by: Tatiana Savina <[email protected]>
Co-authored-by: Olesya Martinyuk <[email protected]>
Co-authored-by: Gleb Kazantaev <[email protected]>
Co-authored-by: Nikita Semaev <[email protected]>
Co-authored-by: Bartosz Lesniewski <[email protected]>
Co-authored-by: Anton Pankratv <[email protected]>
Co-authored-by: Anastasiia Urlapova <[email protected]>
Co-authored-by: Gabriele Galiero Casay <[email protected]>
Co-authored-by: Anastasia Popova <[email protected]>
Co-authored-by: Anastasiya Ageeva <[email protected]>
Co-authored-by: Pavel Esir <[email protected]>
Co-authored-by: Maksim Shabunin <[email protected]>
Co-authored-by: Andrew Kwangwoong Park <[email protected]>
Co-authored-by: Mikhail Letavin <[email protected]>
Co-authored-by: Evgeny Kotov <[email protected]>
Co-authored-by: Svetlana Dolinina <[email protected]>
Co-authored-by: Victor Kuznetsov <[email protected]>
Co-authored-by: Dmitrii Khurtin <[email protected]>
Co-authored-by: Ivan Novoselov <[email protected]>
Co-authored-by: Aleksandr Pertovsky <[email protected]>
Co-authored-by: Nikolay Shchegolev <[email protected]>
Co-authored-by: Yegor Kruglov <[email protected]>
Co-authored-by: Polina Brzezinskaya <[email protected]>
Co-authored-by: iliya mironov <[email protected]>
Co-authored-by: mei, yang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: inference OpenVINO Runtime library - Inference
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants