OCA\FaceRecognition\BackgroundJob\Tasks\CreateClustersTask::createClusterIfNeeded()

Complexity

Conditions	10
Paths	34

Size

Total Lines	104
Code Lines	57

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	41
CRAP Score	11.9227

Importance

Changes	1
Bugs	0	Features	0

<?php
	private function createClusterIfNeeded(string $userId) {
		$modelId = $this->settingsService->getCurrentFaceModel();

		// Depending on whether we already have clusters, decide if we should create/recreate them.
		//
		$hasPersons = $this->personMapper->countPersons($userId, $modelId) > 0;
		if ($hasPersons) {
			$forceRecreate = $this->needRecreateBySettings($userId);
			$haveEnoughFaces = $this->hasNewFacesToRecreate($userId, $modelId);
			$haveStaled = $this->hasStalePersonsToRecreate($userId, $modelId);

			if ($forceRecreate) {
				$this->logInfo('Clusters already exist, but there was some change that requires recreating the clusters');
			}
			else if ($haveEnoughFaces || $haveStaled) {
				$this->logInfo('Face clustering will be recreated with new information or changes');
			}
			else {
				// If there is no invalid persons, and there is no recent new faces, no need to recreate cluster
				$this->logInfo('Clusters already exist, estimated there is no need to recreate them');
				return;
			}
		}
		else {
			// User should not be able to use this directly, used in tests
			$forceTestCreation = $this->settingsService->_getForceCreateClusters($userId);
			$needCreate = $this->needCreateFirstTime($userId, $modelId);

			if ($forceTestCreation) {
				$this->logInfo('Force the creation of clusters for testing');
			}
			else if ($needCreate) {
				$this->logInfo('Face clustering will be created for the first time.');
			}
			else {
				$this->logInfo(
					'Skipping cluster creation, not enough data (yet) collected. ' .
					'For cluster creation, you need either one of the following:');
				$this->logInfo('* have 1000 faces already processed');
				$this->logInfo('* or you need to have 95% of you images processed');
				$this->logInfo('Use stats command to track progress');
				return;
			}
		}

		// Ok. If we are here, the clusters must be recreated.
		//

		$min_face_size = $this->settingsService->getMinimumFaceSize();
		$min_confidence = $this->settingsService->getMinimumConfidence();

		$faces = array_merge(
			$this->faceMapper->getGroupableFaces($userId, $modelId, $min_face_size, $min_confidence),
			$this->faceMapper->getNonGroupableFaces($userId, $modelId, $min_face_size, $min_confidence)
		);

		$facesCount = count($faces);
		$this->logInfo('There are ' . $facesCount . ' faces for clustering');

		$noSlices = 1;
		$sliceSize = $facesCount;

		$defaultSlice = $this->settingsService->getClusterigBatchSize();
		if ($defaultSlice > 0)  {
			// The minimum batch size is 20000 faces
			$defaultSlice = max($defaultSlice, 2000);
			// The maximun batch size is the faces count.
			$defaultSlice = min($defaultSlice, $facesCount);
			$noSlices = intval($facesCount / $defaultSlice) + 1;
			$sliceSize = ceil($facesCount / $noSlices);
		}

		$this->logDebug('We will cluster with ' . $noSlices . ' batch(es) of ' . $sliceSize . ' faces');

		$newClusters = [];
		for ($i = 0; $i < $noSlices ; $i++) {
			$facesSliced = array_slice($faces, $i * $sliceSize, $sliceSize);

$i * $sliceSize of type double is incompatible with the type integer expected by parameter $offset of array_slice().

It seems like $sliceSize can also be of type double; however, parameter $length of array_slice() does only seem to accept integer|null, maybe add an additional type check?

			$newClusters = array_merge($newClusters, $this->getNewClusters($facesSliced));
		}

		// Cluster is associative array where key is person ID.
		// Value is array of face IDs. For old clusters, person IDs are some existing person IDs,
		// and for new clusters is whatever chinese whispers decides to identify them.
		//
		$currentClusters = $this->getCurrentClusters($faces);

		$this->logInfo(count($newClusters) . ' clusters found after clustering');

		// New merge
		$mergedClusters = $this->mergeClusters($currentClusters, $newClusters);

		$this->personMapper->mergeClusterToDatabase($userId, $currentClusters, $mergedClusters);

		// Remove all orphaned persons (those without any faces)
		// NOTE: we will do this for all models, not just for current one, but this is not problem.
		$orphansDeleted = $this->personMapper->deleteOrphaned($userId);
		if ($orphansDeleted > 0) {
			$this->logInfo('Deleted ' . $orphansDeleted . ' persons without faces');
		}

		// Prevents not create/recreate the clusters unnecessarily.

		$this->settingsService->setNeedRecreateClusters(false, $userId);
		$this->settingsService->_setForceCreateClusters(false, $userId);
	}