Fix precommit (#3879)

Author: Wovchena

.yamllint

@@ -10,6 +10,7 @@ yaml-files:
 #    which yamllint complains about.
 
 ignore: |
+  ci/bandit.yml
   demos/thirdparty/
   tools/accuracy_checker/dataset_definitions.yml
   tools/accuracy_checker/configs/

.bandit renamed to ci/bandit.yml

@@ -1,4 +1,3 @@
-
 ### This config may optionally select a subset of tests to run or skip by
 ### filling out the 'tests' and 'skips' lists given below. If no tests are
 ### specified for inclusion then it is assumed all tests are desired. The skips

ci/check-basics.py

@@ -21,7 +21,7 @@
 """
 
 import re
-import subprocess # nosec - disable B404:import-subprocess check
+import subprocess  # nosec B404  # disable import-subprocess check
 import sys
 
 from pathlib import Path
@@ -150,7 +150,7 @@ def complain(message):
         all_passed = False
 
     print('running bandit...', flush=True)
-    if subprocess.run([sys.executable, '-m', 'bandit', '.', '-r', '-c', '.bandit'], cwd=OMZ_ROOT).returncode != 0:
+    if subprocess.run([sys.executable, '-m', 'bandit', '-r', '-c', 'ci/bandit.yml', '.'], cwd=OMZ_ROOT).returncode != 0:
         all_passed = False
 
     print('running documentation checks...', flush=True)

ci/get-jobs-for-changes.py

@@ -28,7 +28,7 @@
 import argparse
 import json
 import re
-import subprocess # nosec - disable B404:import-subprocess check
+import subprocess  # nosec B404  # disable import-subprocess check
 import sys
 
 from pathlib import Path, PurePosixPath

ci/prepare-documentation.py

@@ -42,7 +42,7 @@
 import sys
 import urllib.parse
 import urllib.request
-import xml.etree.ElementTree as ET # nosec - disable B405:import-xml-etree check
+import xml.etree.ElementTree as ET  # nosec B405  # disable import-xml-etree check
 
 from pathlib import Path
 

ci/prepare-openvino-content.py

@@ -25,7 +25,7 @@
 """
 
 import argparse
-import subprocess # nosec - disable B404:import-subprocess check
+import subprocess  # nosec B404  # disable import-subprocess check
 
 from pathlib import Path, PurePath
 

demos/common/python/html_reader.py

@@ -28,7 +28,7 @@ def get_paragraphs(url_list):
     paragraphs_all = []
     for url in url_list:
         log.debug("Get paragraphs from {}".format(url))
-        with urllib.request.urlopen(url) as response: # nosec - disable B310:urllib-urlopen check
+        with urllib.request.urlopen(url) as response:  # nosec B310  # disable urllib-urlopen check
             parser = HTMLDataExtractor(['title', 'p'])
             charset='utf-8'
             if 'Content-type' in response.headers:

demos/common/python/visualizers/drawing_utils.py

@@ -25,7 +25,7 @@ def __init__(self, n, rng=None):
         if n == 0:
             raise ValueError('ColorPalette accepts only the positive number of colors')
         if rng is None:
-            rng = random.Random(0xACE)  # nosec - disable B311:random check
+            rng = random.Random(0xACE)  # nosec B311  # disable random check
 
         candidates_num = 100
         hsv_colors = [(1.0, 1.0, 1.0)]

demos/formula_recognition_demo/python/utils.py

@@ -19,7 +19,7 @@
 import logging as log
 import os
 import re
-import subprocess # nosec - disable B404:import-subprocess check
+import subprocess  # nosec B404  # disable import-subprocess check
 import tempfile
 from enum import Enum
 from multiprocessing.pool import ThreadPool

demos/multi_camera_multi_target_tracking_demo/python/mc_tracker/sct.py

@@ -43,7 +43,7 @@ def update(self, feature_vec):
             self.clusters.append(feature_vec)
             self.clusters_sizes.append(1)
         elif sum(self.clusters_sizes) < 2*self.feature_len:
-            idx = random.randint(0, self.feature_len - 1) # nosec - disable B311:random check
+            idx = random.randint(0, self.feature_len - 1)  # nosec B311  # disable random check
             self.clusters_sizes[idx] += 1
             self.clusters[idx] += (feature_vec - self.clusters[idx]) / \
                                             self.clusters_sizes[idx]