Merge pull request #446 from mapswipe/fix-duplicated-feature-ids

Fix duplicated feature ids

Author: Hagellach37

mapswipe_workers/mapswipe_workers/project_types/arbitrary_geometry/grouping_functions.py

@@ -55,6 +55,9 @@ def group_input_geometries(input_geometries_file, group_size):
     feature_count = 0
     for feature in layer:
         feature_count += 1
+        # feature count starts at 1
+        # assuming group size would be 10
+        # when feature_count = 11 then we enter the next group
         if feature_count % (group_size + 1) == 0:
             group_id += 1
             group_id_string = f"g{group_id}"
@@ -70,7 +73,9 @@ def group_input_geometries(input_geometries_file, group_size):
                 "screen": [],
             }
 
-        groups[group_id_string]["feature_ids"].append(feature.GetFID())
+        # we use a new id here based on the count
+        # since we are not sure that GetFID returns unique values
+        groups[group_id_string]["feature_ids"].append(feature_count)
         groups[group_id_string]["feature_geometries"].append(
             json.loads(feature.GetGeometryRef().ExportToJson())
         )