Resolve review remarks: rename namespace to ov, export public classes #1
Conversation
This file contains 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
+ Support b_fs_yx_fsv32, bs_fs_yx_bsv32_fsv32 and bs_fs_yx_bsv32_fsv16 format.
…openvinotoolkit#7809) Otherwise Reshape / Output / etc nodes are missed in verbose log And blobs from Output nodes are node dumped
* Paddle_Op_Conversion_prior_box * use default_opset * apply review comments * const type variables modification
…#7547) * Paddle Op conversion for SSD-MobileNetV3: exp, stack * apply review comments * remove fp64 type check of exp * use default_opset * add axis check * apply review comments
* Added missed ifdef * Moved ifdef's to avoid unused variables and unclosed scopes * lower string number for ifdefs
…#7508) * # Conflicts: # docs/template_plugin/tests/functional/op_reference/convert_color_nv12.cpp # inference-engine/tests/functional/plugin/cpu/shared_tests_instances/single_layer_tests/convert_color_nv12.cpp # inference-engine/tests/functional/shared_test_classes/include/shared_test_classes/single_layer/convert_color_nv12.hpp # inference-engine/tests/functional/shared_test_classes/src/single_layer/convert_color_nv12.cpp # ngraph/core/include/openvino/core/preprocess/input_tensor_info.hpp # ngraph/core/include/openvino/core/preprocess/preprocess_steps.hpp # ngraph/core/include/openvino/op/nv12_to_bgr.hpp # ngraph/core/include/openvino/op/nv12_to_rgb.hpp # ngraph/core/src/op/nv12_to_bgr.cpp # ngraph/core/src/op/nv12_to_rgb.cpp # ngraph/core/src/preprocess/pre_post_process.cpp # ngraph/core/src/preprocess/preprocess_steps_impl.hpp # ngraph/test/CMakeLists.txt * Added more test to cover 100% of code Allow convert element type for 'multi-plane' color format * Inherit tensor names for 'convert_color' * Clang * Fix tests * Disable 'int8' preprocessing resize test * Fix review comments * Add more restrictions and tests for planes sub-names * 1) Added check for uniqueness of tensor names generated for nodes Raise error if user's plane sub-name conflicts with some node in a function 2) Added exception safety to preprocess build. Before, when input #2 fail, only one preprocess will be applied to function and it will be corrupted Exception guard will restore function to original state if exception occurs * Fix clang-format
…it#7688) (openvinotoolkit#7688) - Add functional tests
…penvinotoolkit#7866) * PrePostProcessor.output() - first implementation of post-processing Supported convert_layout, convert_element_type and custom operations * Fix review comments * Added test for pre and post processing together Fix clang-format * Move 'validate_and_infer_types' before post-processing
* Added serialization of old API map in ngraph. * Changed order type to int64_t. * Fixed uint64_t error, added comments. * Apply suggestions from code review Co-authored-by: Gleb Kazantaev <[email protected]> * Added tests with undefined type and empty order. * Added set, get and has methods. * Fix in tests. * Apply suggestions from code review Co-authored-by: Ilya Churaev <[email protected]> * Made inline methods, changed to shared_ptr. * Small fix. * Moved methods to header file. * Small fix. Co-authored-by: Gleb Kazantaev <[email protected]> Co-authored-by: Ilya Churaev <[email protected]>
* jit impl. optimized * reviewed * reviewed
…t#7833) * Fixed failed to inserting reshape around MatMul. * remove unnecessary * added comments
…notoolkit#7834) * Added ConcatAlignFilter support for Accuracy Aware mode * fails after reviewing and added tests * moved the test to gna/scale_factors_tests; added comments to test
* add template tests for scatter_update * rm CT from ref impl * fix namings * minor code refactor * fix style * change ref impl * add lambda func * new tests * remove CT from reference implementation, add template tests * fix cpu IE * refactor template tests * split template tests
Update FW with revert changes for 256MB
The ngraph issue has been fixed. WA is not needed anymore.
* Add If implementation with reference * fix test * fix comments * Fix validate_and_INFER_TYPES * rewrite tests for dynamic cases * Fix ci failed * add comentaries for validate_and_infer_types * fix api * Added ngraph checks and delete copied op from opset8 * code style fix * fix code style * add checkers to reference * add has_evaluate * fix eval * Fix code style * fix code style * Add template plugin tests * fix code style * delete boolean * fix IfParams * Fix comments * intermediate commit * delete eval test * add common header * Fix codestyle * fix set_invariant_input/set_body_output * fix code_style * fix codestyle * delete validate_and_infer_types from type prop tests * delete comments
…oolkit#7853) * Enabled support of IR v11 in IR frontend * Enabled serialization to v11 * Added indexes for Inputs/Outputs * Fixed typo * Fixed code style * Fixed deserialization tests * FIxed typo * Avoid changes in the original serialized function * Split IRv7 and IRv10 code * Added preprocessing steps to read network * Added preprocessing for new API for IR v10 * clang-format * Updates for compiler flags * Reverted some changes * Fixed TI tests * Removed version from the function and indexes from parameters/results * Restore function version * Clone function rt info * Removed v11 version * Added serialization of old API map in ngraph. * Changed order type to int64_t. * Fixed uint64_t error, added comments. * Apply suggestions from code review Co-authored-by: Gleb Kazantaev <[email protected]> * Read old_api_map * Added tests with undefined type and empty order. * Fixed reprocessing * Updated gitignore * Updated serialize / deserialize tests * Added set, get and has methods. * Fix in tests. * Applied old_api_map tests * Fixed empty convert_layout * Allow Layout from empty string * Fixed clang-format * Export FEM::load_by_model * Apply suggestions from code review Co-authored-by: Ilya Churaev <[email protected]> * Made inline methods, changed to shared_ptr. * Small fix. * Moved methods to header file. * Small fix. * Applied post-processing * Fixed review comments Co-authored-by: Ilya Churaev <[email protected]> Co-authored-by: Anastasia Popova <[email protected]> Co-authored-by: Gleb Kazantaev <[email protected]> Co-authored-by: Ilya Churaev <[email protected]>
Exclude python, contrib, and samples from the build. Clean up unnecessary dependencies installation. Publish artifacts.
…ry places (openvinotoolkit#7873) * gna * [IE TESTS] Remove functionRefs from some places
* Introduce new evaluate API * Removed redundant pointers from new API * Fixed eval tests * Fixed comments * Enable evaluate_lover/upper methods
* [GPU] Merge prepare_buffer_fusing updates + Disable choosing implementation using TBB if immad is supported * [GPU] Apply comments
) * update description of get_place_by_operation_name_and_input_port * create implementation for get_place_operation_name_and_output_port * fix typo * updatget_place_by_operation_name_and_input_port and get_place_by_operation_name_and_output_port * add tests for get_place_by_operation_name_and_input_port and get_place_by_operation_name_and_output_port * fix style
* Pre-process: - Implicit conversions for element type and layout - 'convert_element_type' with default argument to network - Convert_element_type - don't add ops if dst and src types are same - Convert_layout - don't add ops if dst and src layouts are same - Custom step - use Output<Node> instead of shared_ptr<Node> - Support of addressing input by tensor name Post-process: - Avoid duplication of tensor names after post-processing * Fixed IE tests
* Remove fp16 of Convert layer test from skip_tests.config.cpp as it works now * update repo * add convolution reference test and ngraph backend test * fix location of ngraph visitor convolution.cpp
…f tetsing to these (and see no delta first) (openvinotoolkit#7868)
…it#7580) * Split Transformations between MOC and Common pipelines * Test * Split passes * Fix int8 issue * Small refactoring
rkazants
pushed a commit
that referenced
this pull request
May 24, 2023
…f POT (openvinotoolkit#17398) * Update model_optimization_guide.md * Update model_optimization_guide.md * Update model_optimization_guide.md * Update model_optimization_guide.md * Update model_optimization_guide.md * Update model_optimization_guide.md * Update model_optimization_guide.md * Update home.rst * Update ptq_introduction.md * Update Introduction.md * Update Introduction.md * Update Introduction.md * Update ptq_introduction.md * Update ptq_introduction.md * Update basic_quantization_flow.md * Update basic_quantization_flow.md * Update basic_quantization_flow.md * Update quantization_w_accuracy_control.md * Update quantization_w_accuracy_control.md * Update quantization_w_accuracy_control.md * Update quantization_w_accuracy_control.md * Update quantization_w_accuracy_control.md * Update quantization_w_accuracy_control.md * Update quantization_w_accuracy_control.md * Update quantization_w_accuracy_control.md * Update quantization_w_accuracy_control.md * Update basic_quantization_flow.md * Update basic_quantization_flow.md * Update quantization_w_accuracy_control.md * Update basic_quantization_flow.md * Update basic_quantization_flow.md * Update model_optimization_guide.md * Update ptq_introduction.md * Update quantization_w_accuracy_control.md * Update model_optimization_guide.md * Update quantization_w_accuracy_control.md * Update model_optimization_guide.md * Update quantization_w_accuracy_control.md * Update model_optimization_guide.md * Update Introduction.md * Update basic_quantization_flow.md * Update basic_quantization_flow.md * Update quantization_w_accuracy_control.md * Update ptq_introduction.md * Update Introduction.md * Update model_optimization_guide.md * Update basic_quantization_flow.md * Update quantization_w_accuracy_control.md * Update quantization_w_accuracy_control.md * Update quantization_w_accuracy_control.md * Update Introduction.md * Update FrequentlyAskedQuestions.md * Update model_optimization_guide.md * Update Introduction.md * Update model_optimization_guide.md * Update model_optimization_guide.md * Update model_optimization_guide.md * Update model_optimization_guide.md * Update model_optimization_guide.md * Update ptq_introduction.md * Update ptq_introduction.md * added code snippet (#1) * Update basic_quantization_flow.md * Update basic_quantization_flow.md * Update quantization_w_accuracy_control.md * Update basic_quantization_flow.md * Update basic_quantization_flow.md * Update ptq_introduction.md * Update model_optimization_guide.md * Update basic_quantization_flow.md * Update ptq_introduction.md * Update quantization_w_accuracy_control.md * Update basic_quantization_flow.md * Update basic_quantization_flow.md * Update basic_quantization_flow.md * Update ptq_introduction.md * Update ptq_introduction.md * Delete ptq_introduction.md * Update FrequentlyAskedQuestions.md * Update Introduction.md * Update quantization_w_accuracy_control.md * Update introduction.md * Update basic_quantization_flow.md code blocks * Update quantization_w_accuracy_control.md code snippets * Update docs/optimization_guide/nncf/ptq/code/ptq_torch.py Co-authored-by: Alexander Suslov <[email protected]> * Update model_optimization_guide.md * Optimization docs proofreading (#2) * images updated * delete reminder * review * text review * change images to original ones * Update filter_pruning.md code blocks * Update basic_quantization_flow.md * Update quantization_w_accuracy_control.md * Update images (#3) * images updated * delete reminder * review * text review * change images to original ones * Update filter_pruning.md code blocks * update images * resolve conflicts * resolve conflicts * change images to original ones * resolve conflicts * update images * fix conflicts * Update model_optimization_guide.md * Update docs/optimization_guide/nncf/ptq/code/ptq_tensorflow.py Co-authored-by: Alexander Suslov <[email protected]> * Update docs/optimization_guide/nncf/ptq/code/ptq_torch.py Co-authored-by: Alexander Suslov <[email protected]> * Update docs/optimization_guide/nncf/ptq/code/ptq_onnx.py Co-authored-by: Alexander Suslov <[email protected]> * Update docs/optimization_guide/nncf/ptq/code/ptq_aa_openvino.py Co-authored-by: Alexander Suslov <[email protected]> * Update docs/optimization_guide/nncf/ptq/code/ptq_openvino.py Co-authored-by: Alexander Suslov <[email protected]> * table format fix * Update headers * Update qat.md code blocks --------- Co-authored-by: Alexander Suslov <[email protected]> Co-authored-by: Tatiana Savina <[email protected]>
rkazants
pushed a commit
that referenced
this pull request
Jan 4, 2024
* Remove `set_preprocess.cpp` * Remove `preprocessing.hpp` * Remove `locale.hpp` - ported to `CanCompileModelWithCustomLocale` * Port `version.cpp` and remove legacy * Revert shared `version.hpp`
rkazants
pushed a commit
that referenced
this pull request
Jan 5, 2024
* Delete `ngraph/visibility.hpp` * Delete `ngraph/log.hpp` * Delete `ngraph/file_util.hpp` * Delete `ngraph/type.hpp` * Delete `ngraph/dimension.hpp` * Delete `ngraph/coordinate.hpp` * ClangFormat * Fix build * Fix pyngraph * Remove comment * Fix build
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.