Skip to content

Commit

Permalink
Freeze pyparsing version (openvinotoolkit#996)
Browse files Browse the repository at this point in the history
### Changes

The pyparsing version has been frozen at 2.x.

### Reason for changes

The recent major version update of pyparsing breaks the loading/matching of reference .dot files representing the NNCF graph structure which are used in tests.

### Related tickets

69520

### Tests

test_build_graph and related precommit failures should be passing now.
  • Loading branch information
vshampor committed Oct 28, 2021
1 parent 1bbf3b3 commit 1e9a6c0
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 1 deletion.
8 changes: 8 additions & 0 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,14 @@ def find_version(*file_paths):
"matplotlib~=3.3.4; python_version<'3.7'",
"matplotlib>=3.3.4; python_version>='3.7'",
"networkx>=2.5",

# The recent pyparsing major version update seems to break
# integration with networkx - the graphs parsed from current .dot
# reference files no longer match against the graphs produced in tests.
# Using 2.x versions of pyparsing seems to fix the issue.
# Ticket: 69520
"pyparsing<3.0",

"jsonschema==3.2.0",
"pydot>=1.4.1",
"jstyleson>=0.0.2",
Expand Down
3 changes: 3 additions & 0 deletions tests/tensorflow/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,6 @@ pytest-dependency
yattag>=1.14.0
prettytable>=2.0.0
pydot

# Ticket 69520
pyparsing<3.0
5 changes: 4 additions & 1 deletion tests/torch/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,7 @@ protobuf==3.14.0
onnx>=1.8.0
onnxruntime==1.6.0
pytest-mock>=3.3.1
pytest-dependency>=0.5.1
pytest-dependency>=0.5.1

# Ticket 69520
pyparsing<3.0

0 comments on commit 1e9a6c0

Please sign in to comment.