Skip to content

Commit 0bd3c4e

Browse files
committed
lint
Signed-off-by: xadupre <[email protected]>
1 parent c1cc37b commit 0bd3c4e

File tree

8 files changed

+61
-31
lines changed

8 files changed

+61
-31
lines changed

.github/actions/keras_application_test/action.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,10 @@ runs:
5050
pip install keras-self-attention
5151
pip install pytest pytest-cov pytest-runner
5252
53+
if [[ ${{ inputs.tf_version }} <= 2.15 ]]; then
54+
pip install "protobuf<3.20"
55+
fi
56+
5357
if [[ ${{ inputs.tf_version }} == 1.* ]]; then
5458
pip install keras==2.3.1
5559
pip install transformers==4.2.0
@@ -58,7 +62,9 @@ runs:
5862
pip install numpy==1.19.0
5963
else
6064
pip install transformers
61-
pip install tf_keras==${{ inputs.tf_version }}
65+
if [[ ${{ inputs.tf_version }} != 2.13 ]]; then
66+
pip install tf_keras==${{ inputs.tf_version }}
67+
fi
6268
fi
6369
6470
pip install -e .

.github/actions/keras_unit_test/action.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,9 @@ runs:
3636
if [[ ${{ inputs.tf_version }} == 1.* ]]; then
3737
pip install numpy==1.19.0
3838
else
39-
pip install tf_keras==${{ inputs.tf_version }}
39+
if [[ ${{ inputs.tf_version }} != 2.13 ]]; then
40+
pip install tf_keras==${{ inputs.tf_version }}
41+
fi
4042
fi
4143
4244
pip install -e .

.github/workflows/keras_application_test_ci.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ concurrency:
1515

1616
jobs:
1717

18-
Test_min_py38_with_min_tf: # Do not change this name because it is used in Ruleset of this repo.
18+
Test_min_py38_tf2_9: # Do not change this name because it is used in Ruleset of this repo.
1919
strategy:
2020
fail-fast: false
2121
runs-on: ubuntu-latest
@@ -39,7 +39,7 @@ jobs:
3939
name: Test Results (Py38-TF2.9-ubuntu)
4040
path: ./**/test-results-*.xml
4141

42-
Test_max_py310_with_latest_tf: # Do not change this name because it is used in Ruleset of this repo.
42+
Test_max_py310_tf2_18: # Do not change this name because it is used in Ruleset of this repo.
4343
strategy:
4444
fail-fast: false
4545
runs-on: ubuntu-latest
@@ -87,7 +87,7 @@ jobs:
8787
name: Test Results (Py37-TF1.15-ubuntu)
8888
path: ./**/test-results-*.xml
8989

90-
Extra_tests:
90+
Extra_tests1:
9191
strategy:
9292
fail-fast: false
9393
matrix:
@@ -146,7 +146,7 @@ jobs:
146146

147147
publish-test-results:
148148
name: "Publish Tests Results to Github"
149-
needs: [Test_min_py38_with_min_tf, Test_max_py310_with_latest_tf, Test_py37_with_tf1_15, Extra_tests]
149+
needs: [Test_min_py38_tf2_9, Test_max_py310_tf2_18, Test_py37_with_tf1_15, Extra_tests]
150150
runs-on: ubuntu-latest
151151
permissions:
152152
checks: write

.github/workflows/keras_unit_test_ci.yml

Lines changed: 32 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ concurrency:
1414
cancel-in-progress: true
1515

1616
jobs:
17-
Test_min_py38_with_min_tf: # Do not change this name because it is used in Ruleset of this repo.
17+
Test_min_py38_tf2_9: # Do not change this name because it is used in Ruleset of this repo.
1818
strategy:
1919
fail-fast: false
2020
runs-on: ubuntu-latest
@@ -38,7 +38,7 @@ jobs:
3838
name: Test Results (Py38-TF2.9-ubuntu)
3939
path: ./**/test-results-*.xml
4040

41-
Test_max_py310_with_latest_tf: # Do not change this name because it is used in Ruleset of this repo.
41+
Test_max_py310_tf2_18: # Do not change this name because it is used in Ruleset of this repo.
4242
strategy:
4343
fail-fast: false
4444
runs-on: ubuntu-latest
@@ -62,7 +62,7 @@ jobs:
6262
name: Test Results (Py310-TF2.18-ubuntu)
6363
path: ./**/test-results-*.xml
6464

65-
Test_max_py311_with_latest_tf: # Do not change this name because it is used in Ruleset of this repo.
65+
Test_max_py311_tf2_18: # Do not change this name because it is used in Ruleset of this repo.
6666
strategy:
6767
fail-fast: false
6868
runs-on: ubuntu-latest
@@ -86,7 +86,7 @@ jobs:
8686
name: Test Results (Py311-TF2.18-ubuntu)
8787
path: ./**/test-results-*.xml
8888

89-
Test_max_py312_with_latest_tf: # Do not change this name because it is used in Ruleset of this repo.
89+
Test_max_py312_tf2_18: # Do not change this name because it is used in Ruleset of this repo.
9090
strategy:
9191
fail-fast: false
9292
runs-on: ubuntu-latest
@@ -134,23 +134,43 @@ jobs:
134134
name: Test Results (Py37-TF1.15-ubuntu)
135135
path: ./**/test-results-*.xml
136136

137-
Extra_tests:
137+
Extra_tests2:
138138
strategy:
139139
fail-fast: false
140140
matrix:
141141
name:
142-
- 'py39-tf2.10'
142+
- 'py38-tf2.13'
143143
- 'py39-tf2.15'
144+
- 'py310-tf2.18'
145+
- 'py311-tf2.18'
146+
- 'py312-tf2.18'
144147
os: ['ubuntu-latest', 'windows-2022']
145-
ort_version: ['1.20.1']
146-
onnx_version: ['1.17.0']
147148
include:
148-
- name: 'py39-tf2.10'
149-
tf_version: '2.10.0'
150-
python_version: '3.9'
149+
- name: 'py38-tf2.13'
150+
tf_version: '2.13.0'
151+
python_version: '3.8'
152+
ort_version: '1.16.3'
153+
onnx_version: '1.16.1'
151154
- name: 'py39-tf2.15'
152155
tf_version: '2.15.0'
153156
python_version: '3.9'
157+
ort_version: '1.16.3'
158+
onnx_version: '1.16.1'
159+
- name: 'py310-tf2.18'
160+
tf_version: '2.18.0'
161+
python_version: '3.10'
162+
ort_version: '1.20.1'
163+
onnx_version: '1.17.0'
164+
- name: 'py311-tf2.18'
165+
tf_version: '2.18.0'
166+
python_version: '3.11'
167+
ort_version: '1.20.1'
168+
onnx_version: '1.17.0'
169+
- name: 'py312-tf2.18'
170+
tf_version: '2.18.0'
171+
python_version: '3.12'
172+
ort_version: '1.20.1'
173+
onnx_version: '1.17.0'
154174
runs-on: ${{ matrix.os }}
155175

156176
steps:
@@ -173,7 +193,7 @@ jobs:
173193

174194
publish-test-results:
175195
name: "Publish Tests Results to Github"
176-
needs: [Test_min_py38_with_min_tf, Test_max_py310_with_latest_tf, Test_max_py311_with_latest_tf, Test_max_py312_with_latest_tf, Test_py37_with_tf1_15, Extra_tests]
196+
needs: [Test_min_py38_tf2_9, Test_max_py310_tf2_18, Test_max_py311_tf2_18, Test_max_py312_tf2_18, Test_py37_with_tf1_15, Extra_tests]
177197
runs-on: ubuntu-latest
178198
permissions:
179199
checks: write

.github/workflows/pretrained_model_test_ci.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ concurrency:
1515

1616
jobs:
1717

18-
Test_min_py38_with_min_tf: # Do not change this name because it is used in 'publish-test-results' section below.
18+
Test_min_py38_tf2_9: # Do not change this name because it is used in 'publish-test-results' section below.
1919
strategy:
2020
fail-fast: false
2121
runs-on: ubuntu-latest
@@ -42,7 +42,7 @@ jobs:
4242
name: Test Results (Py38-TF2.9-18-ubuntu)
4343
path: ./**/test-results-*.xml
4444

45-
Test_max_py310_with_latest_tf: # Do not change this name because it is used in 'publish-test-results' section below.
45+
Test_max_py310_tf2_18: # Do not change this name because it is used in 'publish-test-results' section below.
4646
strategy:
4747
fail-fast: false
4848
runs-on: ubuntu-latest
@@ -69,7 +69,7 @@ jobs:
6969
name: Test Results (Py310-TF2.18-18-ubuntu)
7070
path: ./**/test-results-*.xml
7171

72-
Test_max_py312_with_latest_tf: # Do not change this name because it is used in 'publish-test-results' section below.
72+
Test_max_py312_tf2_18: # Do not change this name because it is used in 'publish-test-results' section below.
7373
strategy:
7474
fail-fast: false
7575
runs-on: ubuntu-latest
@@ -122,7 +122,7 @@ jobs:
122122
name: Test Results (Py37-TF1.15-15-ubuntu)
123123
path: ./**/test-results-*.xml
124124

125-
Extra_tests: # Do not change this name because it is used in 'publish-test-results' section below.
125+
Extra_tests3: # Do not change this name because it is used in 'publish-test-results' section below.
126126
strategy:
127127
fail-fast: false
128128
matrix:
@@ -186,7 +186,7 @@ jobs:
186186

187187
publish-test-results:
188188
name: "Publish Tests Results to Github"
189-
needs: [Test_min_py38_with_min_tf, Test_max_py310_with_latest_tf, Test_max_py312_with_latest_tf, Test_py37_with_tf1_15, Extra_tests]
189+
needs: [Test_min_py38_tf2_9, Test_max_py310_tf2_18, Test_max_py312_tf2_18, Test_py37_with_tf1_15, Extra_tests]
190190
runs-on: ubuntu-latest
191191
permissions:
192192
checks: write

.github/workflows/unit_test_ci.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ concurrency:
1515

1616
jobs:
1717

18-
Test_min_py38_with_min_tf: # Do not change this name because it is used in Ruleset of this repo.
18+
Test_min_py38_tf2_9: # Do not change this name because it is used in Ruleset of this repo.
1919
strategy:
2020
fail-fast: false
2121
runs-on: ubuntu-latest
@@ -42,7 +42,7 @@ jobs:
4242
name: Test Results (Py38-TF2.9-18-ubuntu)
4343
path: ./**/test-results-*.xml
4444

45-
Test_max_py310_with_latest_tf: # Do not change this name because it is used in Ruleset of this repo.
45+
Test_max_py310_tf2_18: # Do not change this name because it is used in Ruleset of this repo.
4646
strategy:
4747
fail-fast: false
4848
runs-on: ubuntu-latest
@@ -69,7 +69,7 @@ jobs:
6969
name: Test Results (Py310-TF2.18-18-ubuntu)
7070
path: ./**/test-results-*.xml
7171

72-
Test_max_py311_with_latest_tf: # Do not change this name because it is used in Ruleset of this repo.
72+
Test_max_py311_tf2_18: # Do not change this name because it is used in Ruleset of this repo.
7373
strategy:
7474
fail-fast: false
7575
runs-on: ubuntu-latest
@@ -96,7 +96,7 @@ jobs:
9696
name: Test Results (Py311-TF2.18-18-ubuntu)
9797
path: ./**/test-results-*.xml
9898

99-
Test_max_py312_with_latest_tf: # Do not change this name because it is used in Ruleset of this repo.
99+
Test_max_py312_tf2_18: # Do not change this name because it is used in Ruleset of this repo.
100100
strategy:
101101
fail-fast: false
102102
runs-on: ubuntu-latest
@@ -123,7 +123,7 @@ jobs:
123123
name: Test Results (Py312-TF2.18-18-ubuntu)
124124
path: ./**/test-results-*.xml
125125

126-
Extra_tests:
126+
Extra_tests4:
127127
strategy:
128128
fail-fast: false
129129
matrix:
@@ -196,7 +196,7 @@ jobs:
196196

197197
publish-test-results:
198198
name: "Publish Tests Results to Github"
199-
needs: [Test_min_py38_with_min_tf, Test_max_py310_with_latest_tf, Test_max_py311_with_latest_tf, Test_max_py312_with_latest_tf, Extra_tests]
199+
needs: [Test_min_py38_tf2_9, Test_max_py310_tf2_18, Test_max_py311_tf2_18, Test_max_py312_tf2_18, Extra_tests]
200200
runs-on: ubuntu-latest
201201
permissions:
202202
checks: write

tests/utils/setup_test_env.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,9 @@ pip install onnx==$ONNX_VERSION onnxruntime==$ORT_VERSION onnxruntime-extensions
2222
if [[ $TF_VERSION == 1.* ]]; then
2323
pip install numpy==1.19.0
2424
else
25-
pip install tf_keras==$TF_VERSION
25+
if [[ ${{ inputs.tf_version }} != 2.13 ]]; then
26+
pip install tf_keras==$TF_VERSION
27+
fi
2628
fi
2729

2830
python setup.py install

tools/pylintrc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ enable=indexing-exception,old-raise-syntax
3939
# --enable=similarities". If you want to run only the classes checker, but have
4040
# no Warning level messages displayed, use"--disable=all --enable=classes
4141
# --disable=W"
42-
disable=design,similarities,no-self-use,attribute-defined-outside-init,locally-disabled,star-args,pointless-except,bad-option-value,global-statement,fixme,suppressed-message,useless-suppression,locally-enabled,no-member,no-name-in-module,import-error,unsubscriptable-object,unbalanced-tuple-unpacking,undefined-variable,not-context-manager,useless-object-inheritance
42+
disable=design,similarities,no-self-use,attribute-defined-outside-init,locally-disabled,star-args,pointless-except,bad-option-value,global-statement,fixme,suppressed-message,useless-suppression,locally-enabled,no-member,no-name-in-module,import-error,unsubscriptable-object,unbalanced-tuple-unpacking,undefined-variable,not-context-manager,useless-object-inheritance,consider-using-f-string,unspecified-encoding,unnecessary-lambda-assignment,use-dict-literal
4343

4444

4545
# Set the cache size for astng objects.

0 commit comments

Comments
 (0)