From d19af300bdc9cd776453b10b7fbdffeb718a62ee Mon Sep 17 00:00:00 2001
From: mdurrani808 <mdurrani808@gmail.com>
Date: Sun, 25 Dec 2022 14:45:13 -0500
Subject: [PATCH 1/8] clean up front end ui

---
 .../src/views/PipelineViews/AprilTagTab.vue   | 74 -------------------
 .../apriltag/AprilTagDetectorParams.java      |  2 +-
 .../vision/pipeline/AprilTagPipeline.java     | 23 +-----
 .../pipeline/AprilTagPipelineSettings.java    |  8 +-
 4 files changed, 6 insertions(+), 101 deletions(-)

diff --git a/photon-client/src/views/PipelineViews/AprilTagTab.vue b/photon-client/src/views/PipelineViews/AprilTagTab.vue
index aa842868ba..c8af7c4b9c 100644
--- a/photon-client/src/views/PipelineViews/AprilTagTab.vue
+++ b/photon-client/src/views/PipelineViews/AprilTagTab.vue
@@ -1,25 +1,5 @@
 <template>
   <div>
-    <v-select
-      v-model="selectedFamily"
-      dark
-      color="accent"
-      item-color="secondary"
-      label="Select target family"
-      :items="familyList"
-      @input="handlePipelineUpdate('tagFamily', familyList.indexOf(selectedFamily))"
-    />
-    <v-select
-      v-model="selectedModel"
-      dark
-      color="accent"
-      item-color="secondary"
-      label="Select a target model"
-      :items="targetList"
-      item-text="name"
-      item-value="data"
-      @input="handlePipelineUpdate('targetModel', targetList.indexOf(selectedModel) + 6)"
-    />
     <CVslider
       v-model="decimate"
       class="pt-2"
@@ -61,28 +41,6 @@
       tooltip="Further refines the apriltag corner position initial estimate, suggested left on"
       @input="handlePipelineData('refineEdges')"
     />
-    <CVslider
-      v-model="hammingDist"
-      class="pt-2 pb-4"
-      slider-cols="8"
-      name="Max error bits"
-      min="0"
-      max="10"
-      step="1"
-      tooltip="Maximum number of error bits to correct; potential tags with more will be thrown out. For smaller tags (like 16h5), set this as low as possible."
-      @input="handlePipelineData('hammingDist')"
-    />
-    <CVslider
-      v-model="decisionMargin"
-      class="pt-2 pb-4"
-      slider-cols="8"
-      name="Decision Margin Cutoff"
-      min="0"
-      max="250"
-      step="1"
-      tooltip="Tags with a 'margin' (decoding quality score) less than this wil be rejected. Increase this to reduce the number of false positive detections"
-      @input="handlePipelineData('decisionMargin')"
-    />
     <CVslider
       v-model="numIterations"
       class="pt-2 pb-4"
@@ -107,23 +65,7 @@
           CVslider,
           CVswitch,
         },
-        data() {
-            return {
-              familyList: ["tag36h11", "tag25h9", "tag16h5"],
-              // Selected model is offset (ew) by 6 from the photon ordinal, as we only wanna show the 36h11 and 16h5 options
-              targetList: ['6.5in (36h11) AprilTag', '6in (16h5) AprilTag'], //Keep in sync with TargetModel.java
-            }
-        },
         computed: {
-          selectedModel: {
-              get() {
-                  let ret = this.$store.getters.currentPipelineSettings.targetModel - 6
-                  return this.targetList[ret];
-              },
-              set(val) {
-                  this.$store.commit("mutatePipeline", {"targetModel": this.targetList.indexOf(val) + 6})
-              }
-          },
           selectedFamily: {
             get() {
                 let ret = this.$store.getters.currentPipelineSettings.tagFamily
@@ -141,22 +83,6 @@
                 this.$store.commit("mutatePipeline", {"decimate": val});
             }
           },
-          hammingDist: {
-            get() {
-                return this.$store.getters.currentPipelineSettings.hammingDist
-            },
-            set(val) {
-                this.$store.commit("mutatePipeline", {"hammingDist": val});
-            }
-          },
-          decisionMargin: {
-            get() {
-                return this.$store.getters.currentPipelineSettings.decisionMargin
-            },
-            set(val) {
-                this.$store.commit("mutatePipeline", {"decisionMargin": val});
-            }
-          },
           numIterations: {
             get() {
                 return this.$store.getters.currentPipelineSettings.numIterations
diff --git a/photon-core/src/main/java/org/photonvision/vision/apriltag/AprilTagDetectorParams.java b/photon-core/src/main/java/org/photonvision/vision/apriltag/AprilTagDetectorParams.java
index 8942b50f6c..25fc6184ec 100644
--- a/photon-core/src/main/java/org/photonvision/vision/apriltag/AprilTagDetectorParams.java
+++ b/photon-core/src/main/java/org/photonvision/vision/apriltag/AprilTagDetectorParams.java
@@ -30,7 +30,7 @@
 
 public class AprilTagDetectorParams {
     public static AprilTagDetectorParams DEFAULT_36H11 =
-            new AprilTagDetectorParams(AprilTagFamily.kTag36h11, 1.0, 0.0, 4, false, false);
+            new AprilTagDetectorParams(AprilTagFamily.kTag16h5, 1.0, 0.0, 4, false, false);
 
     public final AprilTagFamily tagFamily;
     public final double decimate;
diff --git a/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipeline.java b/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipeline.java
index a1096c62ce..3fba6fdb56 100644
--- a/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipeline.java
+++ b/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipeline.java
@@ -79,28 +79,7 @@ protected void setPipeParamsImpl() {
         // target model for the draw 3d targets pipeline to work...
 
         // for now, hard code tag width based on enum value
-        double tagWidth;
-
-        // This needs
-        switch (settings.targetModel) {
-            case k200mmAprilTag:
-                {
-                    tagWidth = Units.inchesToMeters(3.25 * 2);
-                    break;
-                }
-            case k6in_16h5:
-                {
-                    tagWidth = Units.inchesToMeters(3 * 2);
-                    break;
-                }
-            default:
-                {
-                    // guess at 200mm?? If it's zero everything breaks, but it should _never_ be zero. Unless
-                    // users select the wrong model...
-                    tagWidth = 0.16;
-                    break;
-                }
-        }
+        double tagWidth = Units.inchesToMeters(3 * 2); // for 6in 16h5 tag.
 
         aprilTagDetectionPipe.setParams(
                 new AprilTagDetectionPipeParams(
diff --git a/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipelineSettings.java b/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipelineSettings.java
index 12d53d28d0..4315377165 100644
--- a/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipelineSettings.java
+++ b/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipelineSettings.java
@@ -24,7 +24,7 @@
 
 @JsonTypeName("AprilTagPipelineSettings")
 public class AprilTagPipelineSettings extends AdvancedPipelineSettings {
-    public AprilTagFamily tagFamily = AprilTagFamily.kTag36h11;
+    public AprilTagFamily tagFamily = AprilTagFamily.kTag16h5;
     public double decimate = 1.0;
     public double blur = 0;
     public int threads = 1;
@@ -33,8 +33,8 @@ public class AprilTagPipelineSettings extends AdvancedPipelineSettings {
     public int numIterations = 200;
 
     // TODO is this a legit, reasonable default?
-    public int hammingDist = 1;
-    public int decisionMargin = 30;
+    public int hammingDist = 0;
+    public int decisionMargin = 35;
 
     // 3d settings
 
@@ -42,7 +42,7 @@ public AprilTagPipelineSettings() {
         super();
         pipelineType = PipelineType.AprilTag;
         outputShowMultipleTargets = true;
-        targetModel = TargetModel.k200mmAprilTag;
+        targetModel = TargetModel.k6in_16h5;
         cameraExposure = -1;
         cameraAutoExposure = true;
         ledMode = false;

From d564bcb883a4b25a82a58ed3b0d80d92fda456f6 Mon Sep 17 00:00:00 2001
From: mdurrani808 <mdurrani808@gmail.com>
Date: Fri, 30 Dec 2022 19:20:44 -0500
Subject: [PATCH 2/8] address changes

---
 .../src/views/PipelineViews/AprilTagTab.vue   | 252 ++++++++++--------
 .../vision/apriltag/AprilTagDetector.java     |   2 +-
 .../apriltag/AprilTagDetectorParams.java      |   2 +-
 3 files changed, 150 insertions(+), 106 deletions(-)

diff --git a/photon-client/src/views/PipelineViews/AprilTagTab.vue b/photon-client/src/views/PipelineViews/AprilTagTab.vue
index c8af7c4b9c..59fb129858 100644
--- a/photon-client/src/views/PipelineViews/AprilTagTab.vue
+++ b/photon-client/src/views/PipelineViews/AprilTagTab.vue
@@ -1,122 +1,166 @@
 <template>
   <div>
+    <v-select
+        v-model="selectedFamily"
+        dark
+        color="accent"
+        item-color="secondary"
+        label="Select target family"
+        :items="familyList"
+        @input="handlePipelineUpdate('tagFamily', familyList.indexOf(selectedFamily))"
+    />
     <CVslider
-      v-model="decimate"
-      class="pt-2"
-      slider-cols="8"
-      name="Decimate"
-      min="1"
-      max="8"
-      step="1.0"
-      tooltip="Increases FPS at the expense of range by reducing image resolution initially"
-      @input="handlePipelineData('decimate')"
+        v-model="decimate"
+        class="pt-2"
+        slider-cols="8"
+        name="Decimate"
+        min="1"
+        max="8"
+        step="1.0"
+        tooltip="Increases FPS at the expense of range by reducing image resolution initially"
+        @input="handlePipelineData('decimate')"
     />
     <CVslider
-      v-model="blur"
-      class="pt-2"
-      slider-cols="8"
-      name="Blur"
-      min="0"
-      max="5"
-      step=".01"
-      tooltip="Gaussian blur added to the image, high FPS cost for slightly decreased noise"
-      @input="handlePipelineData('blur')"
+        v-model="blur"
+        class="pt-2"
+        slider-cols="8"
+        name="Blur"
+        min="0"
+        max="5"
+        step=".01"
+        tooltip="Gaussian blur added to the image, high FPS cost for slightly decreased noise"
+        @input="handlePipelineData('blur')"
     />
     <CVslider
-      v-model="threads"
-      class="pt-2"
-      slider-cols="8"
-      name="Threads"
-      min="1"
-      max="8"
-      step="1"
-      tooltip="Number of threads spawned by the AprilTag detector"
-      @input="handlePipelineData('threads')"
+        v-model="threads"
+        class="pt-2"
+        slider-cols="8"
+        name="Threads"
+        min="1"
+        max="8"
+        step="1"
+        tooltip="Number of threads spawned by the AprilTag detector"
+        @input="handlePipelineData('threads')"
     />
     <CVswitch
-      v-model="refineEdges"
-      class="pt-2"
-      slider-cols="8"
-      name="Refine Edges"
-      tooltip="Further refines the apriltag corner position initial estimate, suggested left on"
-      @input="handlePipelineData('refineEdges')"
+        v-model="refineEdges"
+        class="pt-2"
+        slider-cols="8"
+        name="Refine Edges"
+        tooltip="Further refines the apriltag corner position initial estimate, suggested left on"
+        @input="handlePipelineData('refineEdges')"
+    />
+    <CVslider
+        v-model="decisionMargin"
+        class="pt-2 pb-4"
+        slider-cols="8"
+        name="Decision Margin Cutoff"
+        min="0"
+        max="250"
+        step="1"
+        tooltip="Tags with a 'margin' (decoding quality score) less than this wil be rejected. Increase this to reduce the number of false positive detections"
+        @input="handlePipelineData('decisionMargin')"
     />
     <CVslider
-      v-model="numIterations"
-      class="pt-2 pb-4"
-      slider-cols="8"
-      name="Pose Estimation Iterations"
-      min="0"
-      max="500"
-      step="1"
-      tooltip="Number of iterations the pose estimation algorithm will run, 50-100 is a good starting point"
-      @input="handlePipelineData('numIterations')"
+        v-model="numIterations"
+        class="pt-2 pb-4"
+        slider-cols="8"
+        name="Pose Estimation Iterations"
+        min="0"
+        max="500"
+        step="1"
+        tooltip="Number of iterations the pose estimation algorithm will run, 50-100 is a good starting point"
+        @input="handlePipelineData('numIterations')"
     />
   </div>
 </template>
 
 <script>
-    import CVslider from '../../components/common/cv-slider'
-    import CVswitch from '../../components/common/cv-switch'
+import CVslider from '../../components/common/cv-slider'
+import CVswitch from '../../components/common/cv-switch'
 
-    export default {
-        name: "AprilTag",
-        components: {
-          CVslider,
-          CVswitch,
-        },
-        computed: {
-          selectedFamily: {
-            get() {
-                let ret = this.$store.getters.currentPipelineSettings.tagFamily
-                return this.familyList[ret];
-            },
-            set(val) {
-                this.$store.commit("mutatePipeline", {"tagFamily": this.familyList.indexOf(val)})
-            }
-          },
-          decimate: {
-            get() {
-                return this.$store.getters.currentPipelineSettings.decimate
-            },
-            set(val) {
-                this.$store.commit("mutatePipeline", {"decimate": val});
-            }
-          },
-          numIterations: {
-            get() {
-                return this.$store.getters.currentPipelineSettings.numIterations
-            },
-            set(val) {
-                this.$store.commit("mutatePipeline", {"numIterations": val});
-            }
-          },
-          blur: {
-            get() {
-                return this.$store.getters.currentPipelineSettings.blur
-            },
-            set(val) {
-                this.$store.commit("mutatePipeline", {"blur": val});
-            }
-          },
-          threads: {
-            get() {
-                return this.$store.getters.currentPipelineSettings.threads
-            },
-            set(val) {
-                this.$store.commit("mutatePipeline", {"threads": val});
-            }
-          },
-          refineEdges: {
-            get() {
-                return this.$store.getters.currentPipelineSettings.refineEdges
-            },
-            set(val) {
-                this.$store.commit("mutatePipeline", {"refineEdges": val});
-            }
-          },
-        },
-        methods: {
-        }
+export default {
+  name: "AprilTag",
+  components: {
+    CVslider,
+    CVswitch,
+  },
+  data() {
+    return {
+      familyList: ["tag36h11", "tag25h9", "tag16h5"],
+      // Selected model is offset (ew) by 6 from the photon ordinal, as we only wanna show the 36h11 and 16h5 options
+      targetList: ['6in (16h5) AprilTag'], //Keep in sync with TargetModel.java
     }
-</script>
+  },
+  computed: {
+    selectedModel: {
+      get() {
+        let ret = this.$store.getters.currentPipelineSettings.targetModel - 6
+        return this.targetList[ret];
+      },
+      set(val) {
+        this.$store.commit("mutatePipeline", {"targetModel": this.targetList.indexOf(val) + 6})
+      }
+    },
+    selectedFamily: {
+      get() {
+        let ret = this.$store.getters.currentPipelineSettings.tagFamily
+        return this.familyList[ret];
+      },
+      set(val) {
+        this.$store.commit("mutatePipeline", {"tagFamily": this.familyList.indexOf(val)})
+      }
+    },
+    decimate: {
+      get() {
+        return this.$store.getters.currentPipelineSettings.decimate
+      },
+      set(val) {
+        this.$store.commit("mutatePipeline", {"decimate": val});
+      }
+    },
+    decisionMargin: {
+      get() {
+        return this.$store.getters.currentPipelineSettings.decisionMargin
+      },
+      set(val) {
+        this.$store.commit("mutatePipeline", {"decisionMargin": val});
+      }
+    },
+    numIterations: {
+      get() {
+        return this.$store.getters.currentPipelineSettings.numIterations
+      },
+      set(val) {
+        this.$store.commit("mutatePipeline", {"numIterations": val});
+      }
+    },
+    blur: {
+      get() {
+        return this.$store.getters.currentPipelineSettings.blur
+      },
+      set(val) {
+        this.$store.commit("mutatePipeline", {"blur": val});
+      }
+    },
+    threads: {
+      get() {
+        return this.$store.getters.currentPipelineSettings.threads
+      },
+      set(val) {
+        this.$store.commit("mutatePipeline", {"threads": val});
+      }
+    },
+    refineEdges: {
+      get() {
+        return this.$store.getters.currentPipelineSettings.refineEdges
+      },
+      set(val) {
+        this.$store.commit("mutatePipeline", {"refineEdges": val});
+      }
+    },
+  },
+  methods: {
+  }
+}
+</script>
\ No newline at end of file
diff --git a/photon-core/src/main/java/org/photonvision/vision/apriltag/AprilTagDetector.java b/photon-core/src/main/java/org/photonvision/vision/apriltag/AprilTagDetector.java
index 0ff18937dc..71a894f667 100644
--- a/photon-core/src/main/java/org/photonvision/vision/apriltag/AprilTagDetector.java
+++ b/photon-core/src/main/java/org/photonvision/vision/apriltag/AprilTagDetector.java
@@ -36,7 +36,7 @@
 public class AprilTagDetector {
     private static final Logger logger = new Logger(AprilTagDetector.class, LogGroup.VisionModule);
     private long m_detectorPtr = 0;
-    private AprilTagDetectorParams m_detectorParams = AprilTagDetectorParams.DEFAULT_36H11;
+    private AprilTagDetectorParams m_detectorParams = AprilTagDetectorParams.DEFAULT_16H5;
 
     public AprilTagDetector() {
         updateDetector();
diff --git a/photon-core/src/main/java/org/photonvision/vision/apriltag/AprilTagDetectorParams.java b/photon-core/src/main/java/org/photonvision/vision/apriltag/AprilTagDetectorParams.java
index 25fc6184ec..5455099696 100644
--- a/photon-core/src/main/java/org/photonvision/vision/apriltag/AprilTagDetectorParams.java
+++ b/photon-core/src/main/java/org/photonvision/vision/apriltag/AprilTagDetectorParams.java
@@ -29,7 +29,7 @@
 import java.util.Objects;
 
 public class AprilTagDetectorParams {
-    public static AprilTagDetectorParams DEFAULT_36H11 =
+    public static AprilTagDetectorParams DEFAULT_16H5 =
             new AprilTagDetectorParams(AprilTagFamily.kTag16h5, 1.0, 0.0, 4, false, false);
 
     public final AprilTagFamily tagFamily;

From eb0a3d3b3cf790fbe5accbeb5ce6526861e9ef16 Mon Sep 17 00:00:00 2001
From: Chris <chrisgerth010592@gmail.com>
Date: Sat, 31 Dec 2022 11:47:28 -0600
Subject: [PATCH 3/8] Further tweaks to camera default gains to help make sure
 users get a good first impression

---
 .../org/photonvision/vision/pipeline/CVPipelineSettings.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/photon-core/src/main/java/org/photonvision/vision/pipeline/CVPipelineSettings.java b/photon-core/src/main/java/org/photonvision/vision/pipeline/CVPipelineSettings.java
index ac6838d5fa..f2e3543a6f 100644
--- a/photon-core/src/main/java/org/photonvision/vision/pipeline/CVPipelineSettings.java
+++ b/photon-core/src/main/java/org/photonvision/vision/pipeline/CVPipelineSettings.java
@@ -46,8 +46,8 @@ public class CVPipelineSettings implements Cloneable {
     // quirk
     public int cameraGain = 50;
     // Currently only used by the zero-copy Pi Camera driver
-    public int cameraRedGain = 18;
-    public int cameraBlueGain = 24;
+    public int cameraRedGain = 11;
+    public int cameraBlueGain = 20;
     public int cameraVideoModeIndex = 0;
     public FrameDivisor streamingFrameDivisor = FrameDivisor.NONE;
     public boolean ledMode = false;

From ac59465ab9a53764cd46bb58511225a7f25edbe2 Mon Sep 17 00:00:00 2001
From: Chris <chrisgerth010592@gmail.com>
Date: Sat, 31 Dec 2022 11:53:16 -0600
Subject: [PATCH 4/8] even more saner defaults

---
 .../vision/pipeline/AprilTagPipelineSettings.java    |  4 +---
 .../vision/pipeline/CVPipelineSettings.java          |  2 +-
 .../photonvision/vision/processes/VisionModule.java  | 12 ++++++------
 3 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipelineSettings.java b/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipelineSettings.java
index 9c35c8309f..f85399fd2d 100644
--- a/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipelineSettings.java
+++ b/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipelineSettings.java
@@ -27,12 +27,10 @@ public class AprilTagPipelineSettings extends AdvancedPipelineSettings {
     public AprilTagFamily tagFamily = AprilTagFamily.kTag16h5;
     public int decimate = 1;
     public double blur = 0;
-    public int threads = 1;
+    public int threads = 4; //Multiple threads seems to be better performance on most platforms
     public boolean debug = false;
     public boolean refineEdges = true;
     public int numIterations = 200;
-
-    // TODO is this a legit, reasonable default?
     public int hammingDist = 0;
     public int decisionMargin = 35;
 
diff --git a/photon-core/src/main/java/org/photonvision/vision/pipeline/CVPipelineSettings.java b/photon-core/src/main/java/org/photonvision/vision/pipeline/CVPipelineSettings.java
index f2e3543a6f..8d6469d2b7 100644
--- a/photon-core/src/main/java/org/photonvision/vision/pipeline/CVPipelineSettings.java
+++ b/photon-core/src/main/java/org/photonvision/vision/pipeline/CVPipelineSettings.java
@@ -44,7 +44,7 @@ public class CVPipelineSettings implements Cloneable {
     public int cameraBrightness = 50;
     // Currently only used by a few cameras (notably the zero-copy Pi Camera driver) with the Gain
     // quirk
-    public int cameraGain = 50;
+    public int cameraGain = 75;
     // Currently only used by the zero-copy Pi Camera driver
     public int cameraRedGain = 11;
     public int cameraBlueGain = 20;
diff --git a/photon-core/src/main/java/org/photonvision/vision/processes/VisionModule.java b/photon-core/src/main/java/org/photonvision/vision/processes/VisionModule.java
index 98fa1cba98..3a9f2df47c 100644
--- a/photon-core/src/main/java/org/photonvision/vision/processes/VisionModule.java
+++ b/photon-core/src/main/java/org/photonvision/vision/processes/VisionModule.java
@@ -104,14 +104,14 @@ public VisionModule(PipelineManager pipelineManager, VisionSource visionSource,
         if (cameraQuirks.hasQuirk(CameraQuirk.Gain)) {
             pipelineManager.userPipelineSettings.forEach(
                     it -> {
-                        if (it.cameraGain == -1) it.cameraGain = 20; // Sane default
+                        if (it.cameraGain == -1) it.cameraGain = 75; // Sane default
                     });
         }
         if (cameraQuirks.hasQuirk(CameraQuirk.AWBGain)) {
             pipelineManager.userPipelineSettings.forEach(
                     it -> {
-                        if (it.cameraRedGain == -1) it.cameraRedGain = 16; // Sane defaults
-                        if (it.cameraBlueGain == -1) it.cameraBlueGain = 16;
+                        if (it.cameraRedGain == -1) it.cameraRedGain = 11; // Sane defaults
+                        if (it.cameraBlueGain == -1) it.cameraBlueGain = 20;
                     });
         }
 
@@ -401,7 +401,7 @@ void setPipeline(int index) {
         }
         if (cameraQuirks.hasQuirk(CameraQuirk.Gain)) {
             // If the gain is disabled for some reason, re-enable it
-            if (pipelineSettings.cameraGain == -1) pipelineSettings.cameraGain = 20;
+            if (pipelineSettings.cameraGain == -1) pipelineSettings.cameraGain = 75;
             visionSource.getSettables().setGain(Math.max(0, pipelineSettings.cameraGain));
         } else {
             pipelineSettings.cameraGain = -1;
@@ -409,8 +409,8 @@ void setPipeline(int index) {
 
         if (cameraQuirks.hasQuirk(CameraQuirk.AWBGain)) {
             // If the AWB gains are disabled for some reason, re-enable it
-            if (pipelineSettings.cameraRedGain == -1) pipelineSettings.cameraRedGain = 16;
-            if (pipelineSettings.cameraBlueGain == -1) pipelineSettings.cameraBlueGain = 16;
+            if (pipelineSettings.cameraRedGain == -1) pipelineSettings.cameraRedGain = 11;
+            if (pipelineSettings.cameraBlueGain == -1) pipelineSettings.cameraBlueGain = 20;
             visionSource.getSettables().setRedGain(Math.max(0, pipelineSettings.cameraRedGain));
             visionSource.getSettables().setBlueGain(Math.max(0, pipelineSettings.cameraBlueGain));
         } else {

From c08b03f81f16e18a5cbd8488cbce19a85a42bbac Mon Sep 17 00:00:00 2001
From: Chris <chrisgerth010592@gmail.com>
Date: Sat, 31 Dec 2022 12:09:39 -0600
Subject: [PATCH 5/8] Even even more camera sane defaults

---
 .../org/photonvision/vision/pipeline/CVPipelineSettings.java    | 2 +-
 .../vision/pipeline/Calibration3dPipelineSettings.java          | 2 +-
 .../vision/pipeline/ColoredShapePipelineSettings.java           | 1 +
 .../vision/pipeline/DriverModePipelineSettings.java             | 1 +
 .../vision/pipeline/ReflectivePipelineSettings.java             | 2 ++
 5 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/photon-core/src/main/java/org/photonvision/vision/pipeline/CVPipelineSettings.java b/photon-core/src/main/java/org/photonvision/vision/pipeline/CVPipelineSettings.java
index 8d6469d2b7..f8653a03ac 100644
--- a/photon-core/src/main/java/org/photonvision/vision/pipeline/CVPipelineSettings.java
+++ b/photon-core/src/main/java/org/photonvision/vision/pipeline/CVPipelineSettings.java
@@ -40,7 +40,7 @@ public class CVPipelineSettings implements Cloneable {
     public String pipelineNickname = "New Pipeline";
     public boolean cameraAutoExposure = false;
     // manual exposure only used if cameraAutoExposure if false
-    public double cameraExposure = 100;
+    public double cameraExposure = 20;
     public int cameraBrightness = 50;
     // Currently only used by a few cameras (notably the zero-copy Pi Camera driver) with the Gain
     // quirk
diff --git a/photon-core/src/main/java/org/photonvision/vision/pipeline/Calibration3dPipelineSettings.java b/photon-core/src/main/java/org/photonvision/vision/pipeline/Calibration3dPipelineSettings.java
index 529aa0c6d7..196c9e7d65 100644
--- a/photon-core/src/main/java/org/photonvision/vision/pipeline/Calibration3dPipelineSettings.java
+++ b/photon-core/src/main/java/org/photonvision/vision/pipeline/Calibration3dPipelineSettings.java
@@ -30,7 +30,7 @@ public class Calibration3dPipelineSettings extends AdvancedPipelineSettings {
 
     public Calibration3dPipelineSettings() {
         super();
-
+        this.cameraAutoExposure = true;
         this.inputShouldShow = true;
         this.outputShouldShow = true;
     }
diff --git a/photon-core/src/main/java/org/photonvision/vision/pipeline/ColoredShapePipelineSettings.java b/photon-core/src/main/java/org/photonvision/vision/pipeline/ColoredShapePipelineSettings.java
index e615b39801..7f00089ed5 100644
--- a/photon-core/src/main/java/org/photonvision/vision/pipeline/ColoredShapePipelineSettings.java
+++ b/photon-core/src/main/java/org/photonvision/vision/pipeline/ColoredShapePipelineSettings.java
@@ -45,6 +45,7 @@ public class ColoredShapePipelineSettings extends AdvancedPipelineSettings {
     public ColoredShapePipelineSettings() {
         super();
         pipelineType = PipelineType.ColoredShape;
+        cameraExposure = 20;
     }
 
     @Override
diff --git a/photon-core/src/main/java/org/photonvision/vision/pipeline/DriverModePipelineSettings.java b/photon-core/src/main/java/org/photonvision/vision/pipeline/DriverModePipelineSettings.java
index 25911458e8..70beebf7a5 100644
--- a/photon-core/src/main/java/org/photonvision/vision/pipeline/DriverModePipelineSettings.java
+++ b/photon-core/src/main/java/org/photonvision/vision/pipeline/DriverModePipelineSettings.java
@@ -31,5 +31,6 @@ public DriverModePipelineSettings() {
         pipelineIndex = PipelineManager.DRIVERMODE_INDEX;
         pipelineType = PipelineType.DriverMode;
         inputShouldShow = true;
+        cameraAutoExposure = true;
     }
 }
diff --git a/photon-core/src/main/java/org/photonvision/vision/pipeline/ReflectivePipelineSettings.java b/photon-core/src/main/java/org/photonvision/vision/pipeline/ReflectivePipelineSettings.java
index 7376f3962a..b06dcaddef 100644
--- a/photon-core/src/main/java/org/photonvision/vision/pipeline/ReflectivePipelineSettings.java
+++ b/photon-core/src/main/java/org/photonvision/vision/pipeline/ReflectivePipelineSettings.java
@@ -27,5 +27,7 @@ public class ReflectivePipelineSettings extends AdvancedPipelineSettings {
     public ReflectivePipelineSettings() {
         super();
         pipelineType = PipelineType.Reflective;
+        cameraExposure = 6;
+        cameraGain = 20;
     }
 }

From 2be04e731bbe6518a4506b01c7f8719ce04bc9c3 Mon Sep 17 00:00:00 2001
From: Chris <chrisgerth010592@gmail.com>
Date: Sat, 31 Dec 2022 12:16:28 -0600
Subject: [PATCH 6/8] lint

---
 .../photonvision/vision/pipeline/AprilTagPipelineSettings.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipelineSettings.java b/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipelineSettings.java
index f85399fd2d..f476b640f9 100644
--- a/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipelineSettings.java
+++ b/photon-core/src/main/java/org/photonvision/vision/pipeline/AprilTagPipelineSettings.java
@@ -27,7 +27,7 @@ public class AprilTagPipelineSettings extends AdvancedPipelineSettings {
     public AprilTagFamily tagFamily = AprilTagFamily.kTag16h5;
     public int decimate = 1;
     public double blur = 0;
-    public int threads = 4; //Multiple threads seems to be better performance on most platforms
+    public int threads = 4; // Multiple threads seems to be better performance on most platforms
     public boolean debug = false;
     public boolean refineEdges = true;
     public int numIterations = 200;

From cbec96db59afd6da5cdeb5ac46d5c02073e91441 Mon Sep 17 00:00:00 2001
From: Chris <chrisgerth010592@gmail.com>
Date: Sat, 31 Dec 2022 12:21:36 -0600
Subject: [PATCH 7/8] lint pt 2

---
 photon-client/src/views/PipelineViews/AprilTagTab.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/photon-client/src/views/PipelineViews/AprilTagTab.vue b/photon-client/src/views/PipelineViews/AprilTagTab.vue
index 59fb129858..2fd47b7363 100644
--- a/photon-client/src/views/PipelineViews/AprilTagTab.vue
+++ b/photon-client/src/views/PipelineViews/AprilTagTab.vue
@@ -163,4 +163,4 @@ export default {
   methods: {
   }
 }
-</script>
\ No newline at end of file
+</script>

From 732467d5f0618bc5dbb716d90ab934c8194120aa Mon Sep 17 00:00:00 2001
From: Chris <chrisgerth010592@gmail.com>
Date: Sat, 31 Dec 2022 12:26:47 -0600
Subject: [PATCH 8/8] unit test fixup

---
 .../java/org/photonvision/vision/pipeline/AprilTagTest.java     | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/photon-core/src/test/java/org/photonvision/vision/pipeline/AprilTagTest.java b/photon-core/src/test/java/org/photonvision/vision/pipeline/AprilTagTest.java
index 9fede51cda..792ca95fbf 100644
--- a/photon-core/src/test/java/org/photonvision/vision/pipeline/AprilTagTest.java
+++ b/photon-core/src/test/java/org/photonvision/vision/pipeline/AprilTagTest.java
@@ -24,6 +24,7 @@
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.photonvision.common.util.TestUtils;
+import org.photonvision.vision.apriltag.AprilTagFamily;
 import org.photonvision.vision.camera.QuirkyCamera;
 import org.photonvision.vision.frame.provider.FileFrameProvider;
 import org.photonvision.vision.pipeline.result.CVPipelineResult;
@@ -46,6 +47,7 @@ public void testApriltagFacingCamera() {
         pipeline.getSettings().cornerDetectionAccuracyPercentage = 4;
         pipeline.getSettings().cornerDetectionUseConvexHulls = true;
         pipeline.getSettings().targetModel = TargetModel.k200mmAprilTag;
+        pipeline.getSettings().tagFamily = AprilTagFamily.kTag36h11;
 
         var frameProvider =
                 new FileFrameProvider(