[mob][photos] Update remote on remove feedback

mobile/lib/services/machine_learning/face_ml/feedback/cluster_feedback.dart

@@ -187,6 +187,10 @@ class ClusterFeedbackService {
       await FaceMLDataDB.instance
           .bulkCaptureNotPersonFeedback(notClusterIdToPersonId);
 
+      // Update remote so new sync does not undo this change
+      await PersonService.instance
+          .removeFilesFromPerson(person: p, faceIDs: faceIDs.toSet());
+
       Bus.instance.fire(PeopleChangedEvent());
       _logger.info('removeFilesFromPerson done');
       return;

mobile/lib/services/machine_learning/face_ml/person/person_service.dart

@@ -201,6 +201,38 @@ class PersonService {
     personData.logStats();
   }
 
+  Future<void> removeFilesFromPerson({
+    required PersonEntity person,
+    required Set<String> faceIDs,
+  }) async {
+    final personData = person.data;
+    final List<int> emptiedClusters = [];
+    for (final cluster in personData.assigned!) {
+      cluster.faces.removeWhere((faceID) => faceIDs.contains(faceID));
+      if (cluster.faces.isEmpty) {
+        emptiedClusters.add(cluster.id);
+      }
+    }
+
+    // Safety check to make sure we haven't created an empty cluster now, if so delete it
+    for (final emptyClusterID in emptiedClusters) {
+      personData.assigned!
+          .removeWhere((element) => element.id != emptyClusterID);
+      await faceMLDataDB.removeClusterToPerson(
+        personID: person.remoteID,
+        clusterID: emptyClusterID,
+      );
+    }
+
+    
+    await entityService.addOrUpdate(
+      EntityType.person,
+      json.encode(personData.toJson()),
+      id: person.remoteID,
+    );
+    personData.logStats();
+  }
+
   Future<void> deletePerson(String personID, {bool onlyMapping = false}) async {
     if (onlyMapping) {
       final PersonEntity? entity = await getPerson(personID);