From cadda0a3c1c2ee5bfdf573929d19285300080c9a Mon Sep 17 00:00:00 2001
From: BytingBulldogs3539 <bbprogrammers3539@gmail.com>
Date: Sat, 23 Dec 2023 22:50:07 -0500
Subject: [PATCH] Update VisionSourceManager.java

---
 .../vision/processes/VisionSourceManager.java     | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/photon-core/src/main/java/org/photonvision/vision/processes/VisionSourceManager.java b/photon-core/src/main/java/org/photonvision/vision/processes/VisionSourceManager.java
index 853d19c017..a84e6aba4e 100644
--- a/photon-core/src/main/java/org/photonvision/vision/processes/VisionSourceManager.java
+++ b/photon-core/src/main/java/org/photonvision/vision/processes/VisionSourceManager.java
@@ -47,6 +47,7 @@ public class VisionSourceManager {
 
     final List<CameraConfiguration> unmatchedLoadedConfigs = new CopyOnWriteArrayList<>();
     private boolean hasWarned;
+    private boolean hasWarnedNoCameras = false;
     private String ignoredCamerasRegex = "";
 
     private static class SingletonHolder {
@@ -138,16 +139,21 @@ protected List<VisionSource> tryMatchCamImpl(ArrayList<CameraInfo> cameraInfos)
             connectedCameras.addAll(new ArrayList<>(filterAllowedDevices(getConnectedCSICameras())));
         } else {
             connectedCameras = new ArrayList<>(filterAllowedDevices(cameraInfos));
-            createSources = false;
+            createSources =
+                    false; // Dont create sources if we are using supplied camerainfo for unit tests.
         }
 
         // Return no new sources because there are no new sources
         if (connectedCameras.isEmpty() && !cameraInfos.isEmpty()) {
-            logger.warn(
-                    "No cameras were detected! Check that all cameras are connected, and that the path is correct.");
+            if (hasWarnedNoCameras) {
+                logger.warn(
+                        "No cameras were detected! Check that all cameras are connected, and that the path is correct.");
+                hasWarnedNoCameras = true;
+            }
             return null;
-        }
+        } else hasWarnedNoCameras = false;
 
+        // Remove any known cameras.
         connectedCameras.removeIf(c -> knownCameras.contains(c));
 
         // All cameras are already loaded return no new sources.
@@ -458,6 +464,7 @@ private static List<VisionSource> loadVisionSourcesFromCamConfigs(
             boolean is_pi = Platform.isRaspberryPi();
 
             if (configuration.cameraType == CameraType.ZeroCopyPicam && is_pi) {
+                // If the camera was loaded from libcamera then create its source using libcamera.
                 var piCamSrc = new LibcameraGpuSource(configuration);
                 cameraSources.add(piCamSrc);
             } else {