Merge pull request #508 from superannotateai/1304_add_items_to_subset

fixing duplicate skip and success items issue

Author: TumanyanV

src/superannotate/__init__.py

@@ -1,7 +1,7 @@
 import os
 import sys
 
-__version__ = "4.4.5dev2"
+__version__ = "4.4.5dev3"
 
 sys.path.append(os.path.split(os.path.realpath(__file__))[0])
 

src/superannotate/lib/app/interface/sdk_interface.py

@@ -3058,7 +3058,7 @@ def add_items_to_subset(
         :param subset: a name of an existing/new subset to associate items with. New subsets will be automatically created.
         :type subset: str
 
-        :param items: list of items metadata. Required keys are 'name' and 'path'
+        :param items: – list of items metadata. Required keys are 'name' and 'path' if the 'id' key is not provided in the dict.
         :type items: list of dicts
 
         Request Example:

src/superannotate/lib/core/usecases/items.py

@@ -833,7 +833,7 @@ def __init__(
         self.project = project
         self.subset_name = subset_name
         self.items = items
-        self.results = {"failed": [], "skipped": [], "succeeded": []}
+        self.results = {"succeeded": [], "failed": [], "skipped": []}
         self.item_ids = []
         self.path_separated = defaultdict(dict)
         self._backend_client = backend_client
@@ -969,7 +969,7 @@ def __query(self, path, items):
         queried_items = queried_items.data
         # Adding the images missing from specified folder to 'skipped'
         tmp = {item["name"]: item for item in items["items"]}
-        tmp_q = (x.name for x in queried_items)
+        tmp_q = {x.name for x in queried_items}
 
         for i, val in tmp.items():
             if i not in tmp_q:
@@ -997,7 +997,7 @@ def validate_items(
         filtered_items = self.__filter_duplicates()
         if len(filtered_items) != len(self.items):
             self.reporter.log_info(
-                f"Dropping duplicates found {len(filtered_items)} / {len(self.items)} unique items"
+                f"Dropping duplicates. Found {len(filtered_items)} / {len(self.items)} unique items"
             )
         self.items = filtered_items
         self.items = self.__filter_invalid_items()