lint(): pass ruff check

Author: AmitMY

.github/workflows/lint.yaml

@@ -20,7 +20,7 @@ jobs:
           python-version: '3.10'
 
       - name: Install Requirements
-        run: pip install .[dev]
+        run: pip install ".[dev,server]"
 
       - name: Lint Code
         run: pylint sign_language_segmentation

.github/workflows/test.yaml

@@ -20,7 +20,7 @@ jobs:
           python-version: '3.10'
 
       - name: Install Requirements
-        run: pip install .[dev]
+        run: pip install ".[dev,server]"
 
       - name: Test Code
         run: pytest sign_language_segmentation

sign_language_segmentation/src/evaluate_mediapi.py

@@ -3,7 +3,7 @@
 import io
 import zipfile
 from functools import lru_cache
-from typing import Any, Dict, Iterable, List, Optional, Tuple, TypedDict
+from typing import List, Optional, Tuple, TypedDict
 
 import numpy as np
 import torch
@@ -14,8 +14,7 @@
 from tqdm import tqdm
 
 from sign_language_segmentation.bin import load_model, process_pose, predict
-from .utils.metrics import frame_accuracy, frame_f1, frame_precision, frame_recall, frame_roc_auc, segment_percentage, \
-    segment_IoU
+from .utils.metrics import frame_f1, frame_precision, frame_recall, frame_roc_auc
 
 
 def read_pose_tsv_file(pose_tsv: bytes, num_keypoints: int = 33):
@@ -82,7 +81,8 @@ def read_mediapi_set(mediapi_path: str, pose_path: str = None, split='test'):
 
             print("Sample test data:", test_data[0])
 
-        name_number = lambda name: int(name.split('/')[-1].split('.')[0]) if not name.endswith('/') else -1
+        def name_number(name: str):
+            return int(name.split('/')[-1].split('.')[0]) if not name.endswith('/') else -1
 
         # Open the subtitles zip and extract the subtitles for every test datum
         subtitle_info = root_zip.getinfo(subtitle_zips)

sign_language_segmentation/src/model.py

@@ -232,7 +232,6 @@ def evaluate(self, level, fps, _gold, _probs, _segments_gold, _mask, _id, advanc
                 title = f"{level} probs curve #{idx}"
                 probs = np.exp(probs.numpy().squeeze()) * 100
                 x = range(probs.shape[0])
-                y_threshold = [50.0] * probs.shape[0]
                 y_B_probs = probs[:, 1].squeeze()
                 y_I_probs = probs[:, 2].squeeze()
                 y_O_probs = probs[:, 0].squeeze()

sign_language_segmentation/src/summary.py

@@ -1,11 +1,8 @@
 import datetime
 import json
 import os
-import re
 import subprocess
 from argparse import ArgumentParser
-from pprint import pprint
-from statistics import mean, stdev
 
 import pandas as pd
 
@@ -14,8 +11,8 @@ def find_value_from_line(lines, pattern):
     return [line.replace(pattern, '').replace(' ', '') for line in lines if (pattern in line)][0]
 
 
-def flatten(l):
-    return [item for sublist in l for item in sublist]
+def flatten(list_of_lists):
+    return [item for sublist in list_of_lists for item in sublist]
 
 
 parser = ArgumentParser()

sign_language_segmentation/src/summary_decoding.py

@@ -1,10 +1,7 @@
-import datetime
 import json
 import os
-import re
 from argparse import ArgumentParser
 from pathlib import Path
-from pprint import pprint
 from statistics import mean, stdev
 
 import pandas as pd
@@ -18,8 +15,8 @@ def find_value_from_line(lines, pattern, strict_start=False):
     ][0]
 
 
-def flatten(l):
-    return [item for sublist in l for item in sublist]
+def flatten(list_of_lists):
+    return [item for sublist in list_of_lists for item in sublist]
 
 
 parser = ArgumentParser()

sign_language_segmentation/src/summary_eval_external.py

@@ -1,10 +1,7 @@
-import datetime
 import json
 import os
-import re
 from argparse import ArgumentParser
 from pathlib import Path
-from pprint import pprint
 from statistics import mean, stdev
 
 import pandas as pd
@@ -18,8 +15,8 @@ def find_value_from_line(lines, pattern, strict_start=False):
     ][0]
 
 
-def flatten(l):
-    return [item for sublist in l for item in sublist]
+def flatten(list_of_lists):
+    return [item for sublist in list_of_lists for item in sublist]
 
 
 parser = ArgumentParser()

sign_language_segmentation/src/summary_mediapi.py

@@ -15,8 +15,8 @@ def find_value_from_line(lines, pattern, strict_start=False):
     ][0]
 
 
-def flatten(l):
-    return [item for sublist in l for item in sublist]
+def flatten(list_of_lists):
+    return [item for sublist in list_of_lists for item in sublist]
 
 
 parser = ArgumentParser()

sign_language_segmentation/src/summary_pro.py

@@ -1,10 +1,8 @@
 import datetime
 import json
 import os
-import re
 from argparse import ArgumentParser
 from pathlib import Path
-from pprint import pprint
 from statistics import mean, stdev
 
 import pandas as pd
@@ -18,8 +16,8 @@ def find_value_from_line(lines, pattern, strict_start=False):
     ][0]
 
 
-def flatten(l):
-    return [item for sublist in l for item in sublist]
+def flatten(list_of_lists):
+    return [item for sublist in list_of_lists for item in sublist]
 
 
 parser = ArgumentParser()

sign_language_segmentation/src/summary_pro_new.py

@@ -1,10 +1,7 @@
-import datetime
 import json
 import os
-import re
 from argparse import ArgumentParser
 from pathlib import Path
-from pprint import pprint
 from statistics import mean, stdev
 
 import pandas as pd
@@ -18,8 +15,8 @@ def find_value_from_line(lines, pattern, strict_start=False):
     ][0]
 
 
-def flatten(l):
-    return [item for sublist in l for item in sublist]
+def flatten(list_of_lists):
+    return [item for sublist in list_of_lists for item in sublist]
 
 
 parser = ArgumentParser()