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

Bump tox from 3.20.1 to 3.21.1 in /ngraph/python #6

Closed
wants to merge 435 commits into from

Conversation

dependabot[bot]
Copy link

@dependabot dependabot bot commented on behalf of github Jan 14, 2021

Bumps tox from 3.20.1 to 3.21.1.

Changelog

Sourced from tox's changelog.

v3.21.1 (2021-01-13)

Bugfixes ^^^^^^^^

  • Fix regression that broke using install_command in config replacements - by :user:jayvdb [#1777](https://github.com/tox-dev/tox/issues/1777) <https://github.com/tox-dev/tox/issues/1777>_
  • Fix regression parsing posargs default containing colon. - by :user:jayvdb [#1785](https://github.com/tox-dev/tox/issues/1785) <https://github.com/tox-dev/tox/issues/1785>_

Features ^^^^^^^^

  • Prevent .tox in envlist - by :user:jayvdb [#1684](https://github.com/tox-dev/tox/issues/1684) <https://github.com/tox-dev/tox/issues/1684>_

Miscellaneous ^^^^^^^^^^^^^

  • Enable building tox with setuptools_scm 4 and 5 by :user:hroncok [#1799](https://github.com/tox-dev/tox/issues/1799) <https://github.com/tox-dev/tox/issues/1799>_

v3.21.0 (2021-01-08)

Bugfixes ^^^^^^^^

  • Fix the false congratulations message that appears when a KeyboardInterrupt occurs during package installation. - by :user:gnikonorov [#1453](https://github.com/tox-dev/tox/issues/1453) <https://github.com/tox-dev/tox/issues/1453>_
  • Fix platform support for install_command. - by :user:jayvdb [#1464](https://github.com/tox-dev/tox/issues/1464) <https://github.com/tox-dev/tox/issues/1464>_
  • Fixed regression in v3.20.0 that caused escaped curly braces in setenv to break usage of the variable elsewhere in tox.ini. - by :user:jayvdb [#1690](https://github.com/tox-dev/tox/issues/1690) <https://github.com/tox-dev/tox/issues/1690>_
  • Prevent {} and require {: is only followed by }. - by :user:jayvdb [#1711](https://github.com/tox-dev/tox/issues/1711) <https://github.com/tox-dev/tox/issues/1711>_
  • Raise MissingSubstitution on access of broken ini setting. - by :user:jayvdb [#1716](https://github.com/tox-dev/tox/issues/1716) <https://github.com/tox-dev/tox/issues/1716>_

Features ^^^^^^^^

  • Allow { and } in default of {env:key:default}. - by :user:jayvdb [#1502](https://github.com/tox-dev/tox/issues/1502) <https://github.com/tox-dev/tox/issues/1502>_

... (truncated)

Commits
  • ffac605 release 3.21.1
  • 4b44584 config: Prevent .tox in envlist
  • ff4bd47 config: Revert removal of {packages}/{opts} hack
  • c1c1c5b test_config: Add test for indirect install_command
  • 3914667 Allow building with setuptools_scm 4 and 5
  • 7f13cf6 config: Fix parsing posargs default with colon
  • ad6ca10 Merge pull request #1775 from tox-dev/release-3.21.0
  • 2cc64c1 release 3.21.0
  • 49edaed Add instructions for choosing pip version (#1771)
  • 91e180d Fix CI (#1774)
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

SDxKeeper and others added 30 commits January 14, 2021 12:25
* resolve clash of tests compile pdb file names

* Solving problem using different way

* resolve duplicate flags for gtest
* Added ie_docs dependency on ngraph_cpp_api because of tag files

* Fixed tags files for Plugin API
* ROIPooling: Specification and op class alignment

* ROIPooling: Add check to input tensor type to be aligned with spec

* ROIPooling: Corrected spec description for input tensor shape and box coordinates

* ROIPooling: Changed attributes pooled_h and pooled_w from Shape to plain int

* Revert "ROIPooling: Changed attributes pooled_h and pooled_w from Shape to plain int"

This reverts commit d49cfa8.

* ROIPooling: Further specification changes

* ROIPooling: Rename enum class ROIPoolingMethod methods

* Fix style

* ROIPooling: Draft reference implementation

* ROIPooling: Adjust feature map element type to float for attribute unit test

* ROIPooling: Add single layer test class

* ROIPooling: Corrected output index to iterate through output tensor elements

* ROIPooling: Added validation checks for input types in op constructor

* ROIPooling: Add unit tests

* ROIPooling: Attributes unit test changed to align with spec

* ROIPooling: Add check for batch id in reference implementation and unit test

* ROIPooling: Refactor single layer test class

* ROIPooling: Add test for invalid pooling method

* ROIPooling: Clean up unnecessary function declaration

* ROIPooling: Remove duplicated default ROIPooling method in op constructors

* ROIPooling: Add Infer method to generate suitable ROI data

* ROIPooling: CPU single layer test instantiation for max method

* ROIPooling: Remove enum class ROIPoolingMethod

* Revert "ROIPooling: Clean up unnecessary function declaration"

This reverts commit 074b540.

* ROIPooling: Refactor single layer tests after removing enum class ROIPoolingMethod

* ROIPooling: Add attribute checks in op constructor to align with spec and unit tests

* Resolve CI failure: clang could not resolve static conversion from uint64_t to size_t

* ROIPooling: Fix for output index calculation to loop through all ROIs

* ROIPooling: Add unit test for bilinear interpolation method

* ROIPooling: Add CPU single layer test instantiation for bilinear method

* ROIPooling: Clean up unnecessary enum class for pooling method

* ROIPooling: Add myriad single layer test instantiation

* ROIPooling: Add F16 precision single layer tests for CPU plugin

* ROIPooling: Add node validation check for string method attribute in constructor and unit tests

* ROIPooling: Spec changes to improve understanding of the operation

* ROIPooling: Fix for bilinear method when pooled size is 1x1

* ROIPooling: Add unit test for bilinear method and pooled size 1x1

* ROIPooling: Fix to broken format of specifications

* ROIPooling: Disable Myriad single layer tests

* ROIPooling: Handle dynamic dims and ranks for input tensors and unit tests

* ROIPooling: Code clean up

* ROIPooling: Address review comments

* ROIPooling: Changed location for makeROIPooling helper method

Co-authored-by: Kirill Molchanov <[email protected]>
Co-authored-by: apankratovantonp <[email protected]>
Co-authored-by: Alexander Zhogov <[email protected]>
* [MO] Add CMake install for Model Optimizer

* [MO] Update test for version.py

* [MO] Add CMake install for Model Optimizer

* [MO] Update test for version.py

* [MO] Add CMake install for Model Optimizer

* [MO] Update test for version.py

* Add CMake install rules for OpenVINO 3rd party dependencies

Remove unwanted targets from final distribution (test, static libs, etc)

* remove duplicated targets from install

align GNA Windows install path with Linux

* Add COMPONENT install name for OMP

remove empty lines

* Add OMP to CPack components

* remove gtest from deliverables

add explicitly clDNN_unit_tests target to build

* set clDNN_unit_tests properties only by condition

* remove install commands for clDNN

Co-authored-by: Alexander Zhogov <[email protected]>
* Updated transformations docs

* Added RTTI to examples

* Fixed grammar mistakes

* Mention Sinks in NG docs
* Azure: Add contrib build

* Fix checkout
…ing) (openvinotoolkit#3286)

* Optimized Infer Request Scheduling

* Fixed misprint

* Brushing the code and comments a bit

* further brushing of the ScheduleToWorkerRequest: moving the task execution directly into the loop over devices (avoids pointers and 'else' clause)

* 1) zero-copy (assuming determenistic app-level scheduling) for the multi-device, via "borrowing" the corresponding device-specific blobs and letting the app to implicitly use these

2) Initial MULTI section in the opt guide (primarily to document a tip on helping the MULTI to keep the zero-copy path)

Co-authored-by: apankratovantonp <[email protected]>
Introduce vpu ngraph transformation that decomposes Mish layer with the following formula: mish(x) = x * tanh(ln(exp(x) + 1)). It is needed to fix the accuracy drop of Yolo-v4 public model that happened due to Mish implementation in the myriad plugin is not accurate enough. It is just a workaround and will be removed when Mish layer will be improved.
map: 70.97% (ref: 70.96%)
coco_precision: 73.50% (ref 73.49%)
…peration. The output exists always but it may be not connected to anywhere. (openvinotoolkit#3423)
…ove unused code) (openvinotoolkit#3410)

* Remove unnecessary code from the reshape operator shape deduction

* Add new tests to cover cornercases after documentation update

* Fix typo in test name

* Fix codestyle issues

* Fix tests naming
* Fixed documentation build when paths contain spaces

* ops math formula fix

* Fixed typo in

* Added more dependencies for openvino_docs

* Improvements

Co-authored-by: Nikolay Tyukaev <[email protected]>
* Restore compilaltion with tbb 2017u7

Signed-off-by: Alexander Peskov <[email protected]>

* Fix unsupported arg for tbb deterministic_reduce

Signed-off-by: Alexander Peskov <[email protected]>

Co-authored-by: Alexander Zhogov <[email protected]>
* Comment layer creator and node converter for Elu and Clamp ops.

* Add deserialization for GRN, HardSigmoid, GatherTree.

* Add implementation for Interp, GRN, GatherTree.

* Remove layer creation from ie_ir_parser.cpp

* Remove header files from cnn builder.

* Change op's type in elu functional test, remove name transformtion for this op from ir_parser.

* Remove empty lines.
* initial commit

* add cast

* data type fix

* added tests

* added test without axes and steps

* remove redundant imports

* discussions resolving

* Add cast to TFSliceToSlice

* layer tests fix

* update unittest

* rework transformation

* added clamp

* move broadcast

* update unittests

* failed e2e fix

* added comment

* little fixes

* comments update
…ions (openvinotoolkit#2897)

* faster implementation of gather_nd operator for reference implementations

* remove old impl and time measurements

* exclude test for gather_nd for IE_CPU (output mismatch)

* apply review comments and rename variables for clarify

* rename variables according to PR comments

* try to apply all PR suggestions

* fix indices calcualtions

Co-authored-by: Patryk Elszkowski <[email protected]>
* [MO] Aligned MatMul-1 with spec

* removed link to the doc
* [GNA] Flatten trivial concatenations

* [GNA] move test to subgraph tests

* Style

* style

* review

* fix

* fixes
* Add support for onnx FasterRCNN model

* Add documentation
* [MO] Add CMake install for Model Optimizer

* [MO] Update test for version.py

* [MO] Add CMake install for Model Optimizer

* [MO] Update test for version.py

* [MO] Add CMake install for Model Optimizer

* [MO] Update test for version.py

* export TBB_DIR  in setupvars
mateusztabaka and others added 22 commits January 14, 2021 12:50
We have models with inputs with no initializer and not connected to any other node.
* [MO] Add CMake install for Model Optimizer

* [MO] Update test for version.py

* [MO] Add CMake install for Model Optimizer

* [MO] Update test for version.py

* [MO] Add CMake install for Model Optimizer

* [MO] Update test for version.py

* [MO] fix file permissions for install location

* enable make install for OMZ

* Add option description

* remove OMZ fetching & install
This change fixes the error

Input blob size is not equal network input size (1!=0)

seen when passing a scalar input to a model in the case of VPU plugins.
…t#3670)

* Added addition parameters for StridedSlice
* Firmware updated 1570 -> 1574
…t#2879)

* Fixed multiple linker warnings on Win32

* Fixed linkage on Windows

* Fixed compilation for Win32

* Last fix
Removed Mish decomposition because new Mish kernel implemented
…kit#3715)

* [IE][nGraph]: Enables begin/end iterators for PartialShape

It's convenient to be able to use STL algorithms on
PartialShape since semantically PartialShape is a
sequence of Dimensions.

* [IE][VPU][nGraph]: Introduces tree utilities

Introduces Depth-First-Search and Breadth-First-Search
utilities for tree traversal. Templated arguments
makes them extensible for different use-case scenarios.

BFS is designed in way to make it possible to guarantee
node will be visited only after all its predecessors
have been visited:

       a
      / \
     b   c
     |   |
     d   |
     \  /
       e

There with accordingly provided functors (NumEntries) it's
guaranteed node "e" will be visited after "d" and "c".
Such a property is important for nodes depth evaluation.

* [IE][VPU][nGraph]: Fixes printTo for nGraph type

For some reason if printTo for nGraph type is
usual function it's not picked up by VPU_THROW_UNLESS
triggered inside DynamicToStaticShape transformations.

Making it template specialization does the job.

* [IE][VPU]: Introduces SliceConfiguration class

SliceConfiguration is a class that's intended
to express the result of operation slicing by
batch. The result of slicing is configuration
that specifies what to do with each data object
associated with operation. There are two options
defined: Slice and Unchanged. Typical slice
scenario is Slice, when operation has the same
batch for all inputs and outputs, so all
corresponding data object will be "sliced"
(replaced with copy where batch equal to 1).

At some cases, data object should not sliced
(ex. if operation has constant input which
is the same for all input data batches and
so, does not have batch - Add of 2 tensors
with shapes [10, 1000] and [1000]). To
represent such cases there is option
"Unchanged".

At cases when operation should not be sliced
at all (ex. does not have batch, have different
batch for inputs and outputs, has static
batch and so on) SliceConfiguration object will
return false for "hasSlice" method call. In
these cases inputs and outputs methods calls
will throw an exception.

* [IE][VPU][nGraph]: Enables MatMul operation slice

In case of static batch, operation is not going to be sliced,
since for handling such cases other transformation is used.
Such approach allows both passes to co-exist while one is
being replaced with another.

If data input has other dynamic dimension than batch error
will be thrown since Myriad-X plugin does not support
convolutions (HW accelerated operations) with dynamism in
spatial dimensions.

* [IE][VPU][nGraph]: Enables Convolution operations slice

In case of static batch, operation is not going to be sliced,
since for handling such cases other transformation is used.
Such approach allows both passes to co-exist while one is
being replaced with another.

If data input has other dynamic dimension than batch error
will be thrown since Myriad-X plugin does not support
convolutions (HW accelerated operations) with dynamism in
spatial dimensions.

* [IE][VPU][nGraph]: Enables unary eltwise slice

Since extract dynamic batch transformation will handle
dynamism only by batch (so requires body loop to be static)
operations with dynamism in dimension other than batch should
not be covered by loop.

In case of dynamism in dimension other than batch eltwise
will be considered unsupported for sub-graph extraction.

* [IE][VPU][nGraph]: Enables binary eltwise slice

Since extract dynamic batch transformation will handle
dynamism only by batch (so requires body loop to be static)
operations with dynamism in dimension other than batch should
not be covered by loop.

In case of dynamism in dimension other than batch eltwise
will be considered unsupported for sub-graph extraction.

It's template function since different binary eltwise
operations have the same broadcasting rules.

* [IE][VPU][nGraph]: Enables extract dynamic batch transformation

General approach is following:

1. Extracted sub-graphs should have exactly one input and output
   operation. Otherwise, it's possible that memory consumption of
   model will be increased since loops implementation on Myriad-X
   requires to keep all inputs and outputs of loop to be alive
   along with memory used by loop body. In layout consolidation
   scenario it reflects intention to use minimized amount of
   permutations.

2. Extracted sub-graph should not have external connections (
   the only nodes that allowed to have predecessor or successor
   outside of sub-graph are input and output). Otherwise, it's
   possible that memory consumption of model will be increased
   for the same reason as in previous point.

   To make sure this restriction is met transformation looks
   for leaves in both directions, finds corresponding LCA
   (Lowest Common Ancestor) and checks if such sub-graph has
   external connections. If so, it repeats leaves search
   procedure stopping if it approaches leaves from previous
   iteration and finds LCA again. It is repeated until
   sub-graph without external connections is found (it exists,
   at least source itself forms it).

   Leaf in current context is a node which satisfies one of
   the following conditions (depending on direction):
     Top:
       1. It has no predecessors which are neither Parameter,
          nor Constant
       2. It's unknown how to slice this operation
       3. It could not be sliced (different batch for inputs and
          outputs)
     Bottom:
       1. It has no successors which are not Result
       2. It's unknown how to slice this operation
       3. It could not be sliced (different batch for inputs and
          outputs)

Signed-off-by: Gladilov, Gleb <[email protected]>
…hannel (openvinotoolkit#3481)

Fix of concat scale factors calculation causing infinite loop during memory layer requantization

Fix of copy layer unsupported batch size
It will allow to turn on IE logging from CMD line
without extra configuration file.
Might be usefull for debugging.
* Supported opsets check improvement

* Optimization of V10 nodes creation in nGraph

* Redundant type checking in opset removed

* opset2 existance check added
* Update the spec

* add unit-tests

* add avgPool unit-tests to CMakelist

* Remove second constructor and change the first one to take default values for rounding_type and pad_type

* add type_prop test for default values

* add 5d input single layer test instances

* add type_prop tests

* Require input to be 4D or 5D

* add validation check for pads size

* Update few tests to take 5D input instead of 6D

* Update validate_and_infer_types method

* Update infer_batched_pooling_forward and try_apply_auto_padding methods

* Update auto_padding_spatial_dims_dynamic type_prop test for binary_conv, conv, deformable_conv, group_conv and max_pool

* style-apply

* add validation check for kernel size

* add xfail for avgpool python backend test

* style-apply

* remove avgpool backend test from xfail list

* Update spec

* Allow the 3D input

* Update type_prop test with 3D input

* style-apply

* Remove xfail_issue_38709

* fix typo

* Update spec

* Update outputs section in spec

* Update spec

* fix typo

* clean file

* Update detailed description and fix xml examples

* fix exclude-type typo

* fix typo in outputs section
@dependabot @github
Copy link
Author

dependabot bot commented on behalf of github Jan 14, 2021

Dependabot tried to add @postrational as a reviewer to this PR, but received the following error from GitHub:

POST https://api.github.com/repos/mryzhov/openvino/pulls/6/requested_reviewers: 422 - Reviews may only be requested from collaborators. One or more of the users or teams you specified is not a collaborator of the mryzhov/openvino repository. // See: https://docs.github.com/rest/reference/pulls#request-reviewers-for-a-pull-request

@dependabot @github
Copy link
Author

dependabot bot commented on behalf of github Jan 14, 2021

The following labels could not be found: category: dependencies.

@dependabot @github
Copy link
Author

dependabot bot commented on behalf of github Jan 25, 2021

Superseded by #7.

@dependabot dependabot bot closed this Jan 25, 2021
@dependabot dependabot bot deleted the dependabot/pip/ngraph/python/tox-3.21.1 branch January 25, 2021 13:02
mryzhov pushed a commit that referenced this pull request Feb 16, 2021
* DPC++ link error workaround.

OpenVINO C++ program failed to link when DPC++ compiler is used.
'make_shared_blob' causes 'unresolved external symbol' error on linking.
Commented out some __clang__ specific directives to workaround the issue in "ie_blob.h".

* DPC++ compatibility issue fix #2

1. Removed type-by-type template class definition for __clang__.
2. Modified TBlob() destructor. The 'unresolved reference' error occur again if I left 'virtual ~TBlob();' only. It seems it needs to be 'virtual ~TBlob() {};'.

* DPC++ compatibility fix #3 - Add DPC++ conditional code

Uses '__SYCL_COMPILER_VERSION' predefined macro to check if the compiler is a DPC++ or not.
Added conditional directive to switch code based of the detected compiler.
NOTE: User program must include <CL/sycl.hpp>, or the '__SYCL_COMPILER_VERSION' macro won't be defined and this fix won't take effect.

* DPC++ compatibility issue fix #4

Changed from #ifdef to #if + logical formulas.

* DPC++ compatibility issue fix #5

Added compiler check logic in src/ie_rtti.cpp

* DPC++ Compatibility issue #6 - ie_parameter.cpp

Added compiler check macro for DPC++ to ie_parameter.cpp as well.

Co-authored-by: Yasunori Shimura <[email protected]>
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.