Skip to content

[DRAFT] Dev Conv - jit fix#16

Closed
yury-intel wants to merge 727 commits intonshchego:masterfrom
yury-intel:deform_conv/some_cpu_fixes_savechanges_jit1
Closed

[DRAFT] Dev Conv - jit fix#16
yury-intel wants to merge 727 commits intonshchego:masterfrom
yury-intel:deform_conv/some_cpu_fixes_savechanges_jit1

Conversation

@yury-intel
Copy link
Copy Markdown

Details:

  • item1
  • ...

Tickets:

  • ticket-id

dmitriikhurtin and others added 30 commits June 21, 2021 17:53
…NA library (openvinotoolkit#5953)

* additional checks are added to prevent errors from occurring on the GNA library side

* fixing after review

* move AreLayersSupported from Cnn2D namespace

* GNA_DEBUG is commented

* batch size <= 8 test was added
* [LPT] StridedSlice fix

* [LPT] separateInStandaloneBranch fix
* corrected output names in DeepSpeech conversion doc

* mo args correction

* changed instruction for DeepSpeech version 0.8.2

* added venv activate; removed redundant ending

* added picture and squashed MO graph input args into one

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <tatiana.savina@intel.com>

* applied review comments

Co-authored-by: Tatiana Savina <tatiana.savina@intel.com>
* Fix unknown command line flag errors if -sample-options is used

* Fix raw escape sequences output with dashes

* Fix raw escape sequences output with run_again message

* try change sampleoptions to array
…oolkit#5995)

* [NG Frontend] Introduce classes and macros for Frontend-specific exceptions

* [nGraph Frontend] Python bindings

Exposing Python API for nGraph FrontEndManager introduced in PR openvinotoolkit#5470

* [nGraph Frontend] Python bindings

Exposing Python API for nGraph FrontEndManager introduced in PR openvinotoolkit#5470

* Added docs + moved sources to 'pyngraph/frontend'

* Fix issues after rebase

* Fix clang

* Moving ngraph/python/tests_cpp to ngraph/python/tests/mock

* Fix cmakelists-related comments

* Apply review comments

* Added dots to docstrings

* Cmake - Set target properties for ARCHIVE/PDB/COMPILE_PDB as well as for LIBRARY_OUTPUT_DIRECTORY
* update spec

* add RTTI macro

* create visitor test

* clean backend test file

* add negative to type_prop tests of unary_ops

* add validate_and_infer_types method

* remove negative from type_propp unary_ops

* create negative type_prop tests

* remove boolean check from validate_and_infer_types

* update supported types in spec

* Remove check for signed numeric types

* Remove type_prop test cases with non-signed type

* Add zero sized negative test to interpreter manifest

* Revert "update supported types in spec"

This reverts commit a7ac3e4.

* Revert "update spec"

This reverts commit 4c431d7.

* Add minor changes to review op class

 * Add validation of inputs and outputs in evaluate
 * Use shape from host tensor to calculate element count

* Minor changes in variable names of backend tests

* Add SLT case with integer precision

* Add operation to list of trusted ops

* Address review comments:

 * Add bf16 precision to evaluate method
 * Use unary ops typed case tests for type_prop tests
 * Add end of line to interpreter manifest

* Refactored visitor unit test

* Fix typo in visitor test name

Co-authored-by: ggalieroc <gabriele.galiero.casay@intel.com>
…lkit#5938)

* change threshold to 20 instead of max limit of data type

* add invalid input type test case

* add invalid input data check

* add input type dynamic check

* add backend test case

* add more clarity on spec and align with real implementation

* Revert "[CPU] Fix for  CoreThreadingTestsWithIterations tests (openvinotoolkit#5892)"

This reverts commit 115aa14.

* Revert "Revert "[CPU] Fix for  CoreThreadingTestsWithIterations tests (openvinotoolkit#5892)""

This reverts commit 95afa50.

* Revert "change threshold to 20 instead of max limit of data type"

This reverts commit 91af825.

* Revert "add more clarity on spec and align with real implementation"

This reverts commit a3b232a.

* add visitor attribute test case

* Revert "add visitor attribute test case"

This reverts commit 610728f.

* add attribute test case

* revise the attribute visitor test per parametrized visitor API PR: openvinotoolkit#6181
* Revise OP Round

improve the input type error check
add attribute test case

Signed-off-by: Hu, Yuan2 <yuan2.hu@intel.com>

* fix clang code style issue

Signed-off-by: Hu, Yuan2 <yuan2.hu@intel.com>
Apply review feedback

Fix code style

Add visitors tests

Check visitor for type attribute
* ConvertPrecision for opset4::Range

* ConvertPrecision test for Range operation
* Move gtest to <root>/wqthirdparty

* Fixed inference engine tests

* Fixed unit-tests

* Fixed GPU

* Fixed clDNN_unit_tests compilation

* Reverted ngraph changes

* Fixed VPU tests

* Fixed deprecated tests compilation

* Removed OpenVINO clone

* Added public submodule

* Removed

* Fixed Windows

* More updates for INSTANTIATE_TEST_SUITE_P

* Use release-1.10.0

* Removed ngraph copy of gtest

* Revert "Removed ngraph copy of gtest"

This reverts commit ec9fe08.

* Fixes for TYPED_TEST_CASE

* Fixed GNA tests

* Updated submodule

* Updaed index

* TMP disabled tests

* Revert changes and fix compilation errors

* Fixed caching tests

* Revert back

* Revert back all

Co-authored-by: Michael Nosov <mikhail.nosov@intel.com>
* add single_layer_test for Interpolate-1 (openvinotoolkit#6133)

* Fixed InferenceEngineConfig.cmake usage in include() (openvinotoolkit#6136)

* Deprecated API updates (openvinotoolkit#6252)

* Fixed links to OMZ / DL Streamer (openvinotoolkit#6257)

* Added doxygen deprecation for LowLatency

Co-authored-by: Elena Gvozdeva <elena.gvozdeva@intel.com>
…vinotoolkit#6180)

* Add documentation for PyTorch RCAN model

* Update instruction

* Add new file to list

* Update instruction

* Update instruction

* cherry-pick changes from release branch

* Apply review feedback

* Fix merge conflict
…#6205)

* * Split and Merge kernel refactoring

* * SFINAE: replace condition compilation macro with std::enable_if
* added docs

* up to date with release branch

* delete print
* Docs: Model caching feature overview

* Update docs/IE_DG/Intro_to_Performance.md

Co-authored-by: Anastasiya Ageeva <anastasiya.ageeva@intel.com>

* Apply suggestions from code review

Co-authored-by: Anastasiya Ageeva <anastasiya.ageeva@intel.com>

* Review comments
- Moved code examples to snippets
- Added link to Model Caching overview from "Inference Engine Developer Guide"
- Few minor changes

* Update docs/IE_DG/Intro_to_Performance.md

Co-authored-by: Anastasiya Ageeva <anastasiya.ageeva@intel.com>

Co-authored-by: Anastasiya Ageeva <anastasiya.ageeva@intel.com>
* Update Swish OP description.

Signed-off-by: Luwei Zhou <luwei.zhou@intel.com>

* Use RTTI to declare/define NGraph Swish OP.
Add input element type check when constructing Swish OP.

Signed-off-by: Luwei Zhou <luwei.zhou@intel.com>

* Add Swish into activation serialization test list.

Signed-off-by: Luwei Zhou <luwei.zhou@intel.com>

* Add Swish into IE CPU plugin activation single layer test suit.

Signed-off-by: Luwei Zhou <luwei.zhou@intel.com>

* Add Swish NGraph backend and visitor API tests.

Signed-off-by: Luwei Zhou <luwei.zhou@intel.com>

* Add Swish unsupported parameter data type test cases.

Signed-off-by: Luwei Zhou <luwei.zhou@intel.com>

* Update the Swish OP visistor API to use typed test.

Signed-off-by: Luwei Zhou <luwei.zhou@intel.com>
issue:test_mode always use one type

Signed-off-by: Hu, Yuan2 <yuan2.hu@intel.com>
* Add yolov4 converting tutorial

* Update doc

* Update doc
slyubimt and others added 28 commits July 14, 2021 17:55
* Install guides improvements

* add bullet to conda  System Requirements

* fix formating

* - Add conda install command for Ubuntu20

- fix typo /tmp

* added conda prerequisites

* Update installing-openvino-apt.md

* Update installing-openvino-conda.md

* Update installing-openvino-conda.md

CentOS 7.6

* Update installing-openvino-apt.md

APT Repository

* Update installing-openvino-conda.md

Added Introduction & notice about runtime package

* Update installing-openvino-conda.md

Co-authored-by: Andrey Zaytsev <andrey.zaytsev@intel.com>
- Move RemoveEdge from graph optimizer to graph
- Remove extra removeEdge lambdas
* Move type prop tests to separate file, add cases with exceptions thrown

* Add visitor test, add step to pbc attributes

* SLT for CPU, adjust GPU and SSLT to serve the step attr

* Add backend test

* Fix variance issue, verify by SLT

* Formatting fix

* Add another backend testcase

* Add PriorBoxClustered to trusted ops list

* Add step attr description, remove obsolete comments from ref impl

* Test multiple step values in SLT
* update FakeQuantize spec

* remove `**Default value**: None` for required attribute
* Remove FusedOp inheritance

* Unify namespace usages

* Add type_prop test for dynamic

* Add check for attributes number in visitor test

* Add missing  ngraph op scope macro

* Update type_prop expect

* Remove ngraph deprecated start

* Remove node include
* Update opset in attributes test

* Update attributes test input shape

* Add to revised ops
)

* add SplitConvolution tranformation tests

* fix SplitConvolution transformation copying runtime info

* fix test names

* code review refactoring

* code review refactoring: add node general function

* code review refactoring: FakeQuantize node general function

* code review refactoring: graph output concatenation

* code review refactoring: coding style cleanup

* use clone function instead of creating reference function code duplicate

* use decorator for main tests

* rewrite all with decorators

* fix make_unique for Windows build

* code style fixes

* code review fixes: fix auto& -> &&

* code review fixes: fix template functions defenitions
* Add AdaptivePool to ngraph python API

* Disable tests until CPU implementation is ready

* Change ticket number

* Add create op tests

* Fix quotes

* Fix typo
* protobuf static + mixed editor and importer

* merged onnx common

* remove ngraph_test_util to onnx_importer and onnx

* styles applied

* Revert "remove ngraph_test_util to onnx_importer and onnx".

* test ngraph_test_util dependencies

* Fixed static protobuf

* Next

* Fixes

* Fixed cross-compilation

* Don't export / install onnx_proto

* Fixed Windows

* Fixed opencl headers

* Fix

* Added exclude-all for exe as well

* Fixed code style

* Try without LTO

* LTO off only for onnx importer

Co-authored-by: mbencer <mateusz.bencer@intel.com>
* use `floating-point` in docs

* Get back capital letter.
…notoolkit#6469)

* refactoring code using testing::Combine

* add unnamed namespace to prevent duplicate symbols

* add unnamed namespace to prevent duplicate symbols

* fix createActivation -> createActivationFactory

* code review fixes: align function arguments

* fix centos7 build
…openvinotoolkit#6524)

* ConvertMatmulToPointWiseConvolutionTest first test

* add ConvertMatmulToPointWiseConvolutionFqTest

* use general functions to create test subgraphs

* use general funstion to append node; add ConvertMatmulWithBiasToPointWiseConvolutionTest

* add ConvertMatmulWithBiasToPointWiseConvolutionFqTest

* use decorator instead of bool function arguments

* remove unused functions

* cleanup

* add ConvertMatmulWithFqToPointWiseConvolutionTest

* add ConvertMatmulWithFqToPointWiseConvolutionFqTest

* add ConvertMatmulWithFqToPointWiseConvolutionTestNoAddNode

* remove debug

* add ConvertMatmulToPointWiseConvolutionTestInputRank3

* use TEST_P for ConvertMatmulToPointWiseConvolution tests

* use testing::values fixture instead of multiple tests

* cleanup

* use combine tests for invalid inputs

* code style cleanup

* fix unique_ptr build under Windows

* code review fixes: function template params

* code review fixes: remove duplicated test entry

* fix function arguments alignments
@yury-intel yury-intel closed this Aug 2, 2021
nshchego pushed a commit that referenced this pull request Aug 12, 2021
…vinotoolkit#6653)

* init version, need revise: opset7

* add convert testcase

* multiclass_nms support spec

* init version

* matrixnms support spec

* init support for matrix_nms

* impl matirx_nms

* implemented multiclass_nms reference.

TODO: more test cases.

* support dynamic shape in test

* update to spec 0611

* update to spec 0611

* fixes.

* fix: now sort by class_id and score work.

* fix clang check error

* more test cases verified.

* fixes in ref impl.

* attribute nms_eta works

* test cross_batch and output_type i32.

* enable multiclass-nms cpu plugin fallback ngraph

* keep topk typo

* enable matrix-nms cpu plugin fallback ngraph

* support sort_result_across_batch

* Add matrix_nms unit test

* Add cross batch test cases

* fix typo

* move multiclass to opset8

* move matrixnms to opset8

* Reference implementations for MulticlassNms and MatrixNms ops

* fix name conflict

* remove unused var
sort_result_across_batch default set to false

* avoid float overflow

* fix clang check error

* info for mac fail

* change testcase due to unstable sort

* nms add 'normalized' attribute

* multiclass cpu test support 'normalized'

* nms add 'normalized' attribute

* fixes: 1. normalized support. 2. sort by score before keep_top_k inside a batch.

* fixes: 1. normalized support. 2. sort by score before keep_top_k inside a batch.

* fix sort order in matrix_nms

* fix review comments

* add matrix_nms MKLDNN extension layer

* parallel in matirx nms

* separate filtered_box

* separate class_nms result

* parallel in class

* parallel in batch

* partial new nms

* partial remove useless function

* debug & fix

* debug in indexing

* fix test cases

* remove logging

* fix code-style

* fix typo

* add matrix_nms extension

* nms python api

* remove unused testcases

* refactor transformation

* transform dynamic shape to static shape

* Update inference-engine/src/transformations/include/ngraph_ops/nms_static_shape_ie.hpp

Co-authored-by: Ilya Churaev <ilyachur@gmail.com>

* remove register_pass call

* [MKLDNN]migrate matrix_nms to MKLDNNNode

* bug fix in matrix_nms

* padding on matrix_nms

* remove logging

* test case refine

* merged transform_matrix_nms branch

* refine matrixnms testcase

* multiclass nms cpu plugin implement for static shape, rebased on Reference implementations PR

* rebase to new multi-classs transform provided by lc

* Name style algin with matrix-nms

* static shape padding style to batch inside,new unit test method, real classnum shape

* fix format

* fix ci error

* multi-class NMS modification based on PR reviewer opinion: code format, copyright, delete unused include and funciton way

* explicit template instantiation due to mac ci fail

* Yi3/fix review (#16)

* fix coding style

* use parallel_for2d

* fix ci fail

* unify 'copyright 2021'

* mkldnn_multiclass_nms node update based on PR review (#17)

* [MKLDNN] apply suggestion for matrix_nms (#18)

* fix bug

* apply review comments

* apply review comments

* apply review comments

* apply review comments

* skip only Nms test, not MatrixNms MulticlassNms test

Co-authored-by: Zhang Yi3 <yi3.zhang@intel.com>
Co-authored-by: jialipen <cecilia.peng@intel.com>
Co-authored-by: mangguo <mang.guo@intel.com>
Co-authored-by: Ilya Churaev <ilyachur@gmail.com>
Co-authored-by: liubo-intel <bo4.liu@intel.com>
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.