added depricated functions validation

Author: Vaghinak Basentsyan

superannotate/api.py

@@ -89,7 +89,6 @@ def init(self, config_location=None):
                 req_type='GET',
                 path=f'/team/{self.team_id}',
             )
-
             self.user_id = response.json().get('creator_id', None)
             self.team_name = response.json().get('name', None)
 

superannotate/common.py

@@ -59,6 +59,46 @@
     "Semantic Segmentation for Pixel Projects": "semantic_segmentation_pixel"
 }
 
+VIDEO_DEPRICATED_FUNCTIONS = [
+    "upload_images_from_folder_to_project",
+    "get_image_metadata",
+    "search_images",
+    "upload_images_to_project",
+    "upload_annotations_from_folder_to_project",
+    "upload_image_annotations",
+    "download_image",
+    "download_image_annotations",
+    "get_image_annotations",
+    "set_image_annotation_status",
+    "aggregate_annotations_as_df",
+    "attach_image_urls_to_project",
+    "clone_project",
+    "copy_image",
+    "export_annotation",
+    "upload_image_to_project",
+    "upload_video_to_project",
+    "add_annotation_bbox_to_image",
+    "assign_images",
+    "delete_images",
+    "get_project_image_count",
+    "set_project_workflow",
+    "upload_preannotations_from_folder_to_project",
+    "upload_videos_from_folder_to_project",
+    "add_annotation_comment_to_image",
+    "add_annotation_point_to_image",
+    "benchmark",
+    "class_distribution",
+    "consensus",
+    "convert_project_type",
+    "copy_images",
+    "get_project_workflow",
+    "move_image",
+    "move_images",
+    "set_images_annotation_statuses",
+    "set_project_default_image_quality_in_editor",
+    "upload_images_from_google_cloud_to_project",
+]
+
 
 def prediction_segmentation_status_from_str_to_int(status):
     return _PREDICTION_SEGMENTATION_STATUSES[status]
@@ -324,7 +364,7 @@ def write_to_json(output_path, json_data):
 
 
 def tqdm_converter(
-    total_num, images_converted, images_not_converted, finish_event
+        total_num, images_converted, images_not_converted, finish_event
 ):
     with tqdm(total=total_num) as pbar:
         while True:

superannotate/db/exports.py

@@ -127,20 +127,17 @@ def prepare_export(
     """
     if not isinstance(project, dict):
         project = get_project_metadata_bare(project)
-    if project["type"] == "Video":
-        if include_fuse or only_pinned:
-            raise SABaseException(
-                0,
-                "The function does not support include fuse and pin arguments "
-                "for projects containing videos attached with URLs"
-            )
 
     upload_state = upload_state_int_to_str(project.get("upload_state"))
-    if upload_state == "External" and include_fuse == True:
-        logger.info(
-            "Include fuse functionality is not supported for  projects containing images attached with URLs"
-        )
+
+    if upload_state == "External" and include_fuse:
+        logger.warning("Include fuse functionality is not supported for projects containing  items attached with URLs")
         include_fuse = False
+    if project["type"] == "Video":
+        if only_pinned:
+            logger.warning("Pin functionality is not supported for projects containing videos attached with URLs")
+        only_pinned, include_fuse = False, False
+
     team_id, project_id = project["team_id"], project["id"]
     if annotation_statuses is None:
         annotation_statuses = [2, 3, 4, 5]
@@ -153,13 +150,10 @@ def prepare_export(
     json_req = {
         "include": annotation_statuses,
         "coco": 0,
+        "is_pinned": only_pinned,
+        "fuse": include_fuse,
         "time": current_time
     }
-    if project["type"] != "Video":
-        json_req["fuse"] = int(include_fuse)
-        json_req["is_pinned"] = int(only_pinned)
-        json_req["coco"] = 0
-
     if folder_names is not None:
         json_req["folder_names"] = folder_names
     params = {'team_id': team_id, 'project_id': project_id}

superannotate/db/project_api.py

@@ -1,4 +1,5 @@
 import logging
+import inspect
 
 from .. import common
 from ..api import API
@@ -35,7 +36,7 @@ def get_project_metadata_bare(project_name, include_complete_image_count=False):
     if len(results) > 1:
         raise SAExistingProjectNameException(
             0, "Project name " + project_name +
-            " is not unique. To use SDK please make project names unique."
+               " is not unique. To use SDK please make project names unique."
         )
     elif len(results) == 1:
         res = results[0]
@@ -131,6 +132,13 @@ def get_project_and_folder_metadata(project):
             raise SAIncorrectProjectArgument(project)
     else:
         raise SAIncorrectProjectArgument(project)
+    current_frame = inspect.currentframe()
+    outer_function = inspect.getframeinfo(current_frame.f_back).function
+    outer_outer_function = inspect.getframeinfo(current_frame.f_back.f_back).function
+    if project["type"] == "Video" \
+            and (outer_function in common.VIDEO_DEPRICATED_FUNCTIONS
+                 or outer_outer_function in common.VIDEO_DEPRICATED_FUNCTIONS):
+        raise SABaseException(0, "The function does not support projects containing videos attached with URLs")
     return project, folder
 
 
@@ -202,9 +210,9 @@ def create_folder(project, folder_name):
     params = {"team_id": project["team_id"], "project_id": project["id"]}
     name_changed = False
     if len(
-        set(folder_name).intersection(
-            common.SPECIAL_CHARACTERS_IN_PROJECT_FOLDER_NAMES
-        )
+            set(folder_name).intersection(
+                common.SPECIAL_CHARACTERS_IN_PROJECT_FOLDER_NAMES
+            )
     ) > 0:
         logger.warning(
             "New folder name has special characters. Special characters will be replaced by underscores."