refactor: ♻️ remove deprecated yolov5 functions and serialize methods (#1326)

Co-authored-by: Onuralp SEZER <thunderbirdtr@gmail.com>

Author: yogendrasinghx

sahi/utils/coco.py

@@ -5,7 +5,6 @@
 import os
 import random
 import threading
-import warnings
 from collections import Counter, defaultdict
 from dataclasses import dataclass
 from multiprocessing import Pool
@@ -285,10 +284,6 @@ def json(self):
             "area": self.area,
         }
 
-    def serialize(self):
-        warnings.warn("Use json property instead of serialize method", DeprecationWarning, stacklevel=2)
-        return self.json
-
     def __repr__(self):
         return f"""CocoAnnotation<
     image_id: {self.image_id},
@@ -433,9 +428,6 @@ def json(self):
             "area": self.area,
         }
 
-    def serialize(self):
-        warnings.warn("Use json property instead of serialize method", DeprecationWarning, stacklevel=2)
-
     def __repr__(self):
         return f"""CocoPrediction<
     image_id: {self.image_id},
@@ -1251,30 +1243,6 @@ def split_coco_as_train_val(self, train_split_rate=0.9, numpy_seed=0):
             "val_coco": val_coco,
         }
 
-    def export_as_yolov5(
-        self,
-        output_dir: str | Path,
-        train_split_rate: float = 1.0,
-        numpy_seed: int = 0,
-        mp: bool = False,
-        disable_symlink: bool = False,
-    ):
-        """Deprecated.
-
-        Please use export_as_yolo instead. Calls export_as_yolo with the same arguments.
-        """
-        warnings.warn(
-            "export_as_yolov5 is deprecated. Please use export_as_yolo instead.",
-            DeprecationWarning,
-        )
-        self.export_as_yolo(
-            output_dir=output_dir,
-            train_split_rate=train_split_rate,
-            numpy_seed=numpy_seed,
-            mp=mp,
-            disable_symlink=disable_symlink,
-        )
-
     def export_as_yolo(
         self,
         output_dir: str | Path,
@@ -2323,32 +2291,6 @@ def remove_invalid_coco_results(result_list_or_path: list | str, dataset_dict_or
     return fixed_result_list
 
 
-def export_coco_as_yolov5(
-    output_dir: str,
-    train_coco: Coco | None = None,
-    val_coco: Coco | None = None,
-    train_split_rate: float = 0.9,
-    numpy_seed=0,
-    disable_symlink=False,
-):
-    """Deprecated.
-
-    Please use export_coco_as_yolo instead. Calls export_coco_as_yolo with the same arguments.
-    """
-    warnings.warn(
-        "export_coco_as_yolov5 is deprecated. Please use export_coco_as_yolo instead.",
-        DeprecationWarning,
-    )
-    export_coco_as_yolo(
-        output_dir=output_dir,
-        train_coco=train_coco,
-        val_coco=val_coco,
-        train_split_rate=train_split_rate,
-        numpy_seed=numpy_seed,
-        disable_symlink=disable_symlink,
-    )
-
-
 def export_coco_as_yolo(
     output_dir: str,
     train_coco: Coco | None = None,
@@ -2441,26 +2383,6 @@ def export_coco_as_yolo(
     return yaml_path
 
 
-def export_coco_as_yolov5_via_yml(
-    yml_path: str, output_dir: str, train_split_rate: float = 0.9, numpy_seed=0, disable_symlink=False
-):
-    """Deprecated.
-
-    Please use export_coco_as_yolo_via_yml instead. Calls export_coco_as_yolo_via_yml with the same arguments.
-    """
-    warnings.warn(
-        "export_coco_as_yolov5_via_yml is deprecated. Please use export_coco_as_yolo_via_yml instead.",
-        DeprecationWarning,
-    )
-    export_coco_as_yolo_via_yml(
-        yml_path=yml_path,
-        output_dir=output_dir,
-        train_split_rate=train_split_rate,
-        numpy_seed=numpy_seed,
-        disable_symlink=disable_symlink,
-    )
-
-
 def export_coco_as_yolo_via_yml(
     yml_path: str, output_dir: str, train_split_rate: float = 0.9, numpy_seed=0, disable_symlink=False
 ):

tests/test_coco_utils.py

@@ -655,12 +655,12 @@ def test_get_area_filtered_coco(self):
         assert area_filtered_coco.stats["num_images"] == len(area_filtered_coco.images)
         assert area_filtered_coco.stats["num_annotations"] == len(area_filtered_coco.json["annotations"])
 
-    def test_export_coco_as_yolov5(self):
+    def test_export_coco_as_yolo(self):
         from sahi.utils.coco import Coco, export_coco_as_yolo
 
         coco_dict_path = "tests/data/coco_utils/combined_coco.json"
         image_dir = "tests/data/coco_utils/"
-        output_dir = "tests/data/export_coco_as_yolov5/"
+        output_dir = "tests/data/export_coco_as_yolo/"
         if os.path.isdir(output_dir):
             shutil.rmtree(output_dir, ignore_errors=True)
         coco = Coco.from_coco_dict_or_path(coco_dict_path, image_dir=image_dir)