From e71a25733bceb28f8a683fe0d3ab9aa6ba508cea Mon Sep 17 00:00:00 2001
From: Julian Stirling <julian@julianstirling.co.uk>
Date: Tue, 4 Nov 2025 20:05:27 +0000
Subject: [PATCH 1/3] Fix CSM settings from erroring

CSM settings was erroring due to reading properties that don't exist and calculating
using nulls.
---
 .../CSMCalibrationSettings.vue                | 33 +++++++++++--------
 1 file changed, 19 insertions(+), 14 deletions(-)

diff --git a/webapp/src/components/tabContentComponents/settingsComponents/CSMSettingsComponents/CSMCalibrationSettings.vue b/webapp/src/components/tabContentComponents/settingsComponents/CSMSettingsComponents/CSMCalibrationSettings.vue
index fc3359c4..953473c4 100644
--- a/webapp/src/components/tabContentComponents/settingsComponents/CSMSettingsComponents/CSMCalibrationSettings.vue
+++ b/webapp/src/components/tabContentComponents/settingsComponents/CSMSettingsComponents/CSMCalibrationSettings.vue
@@ -104,20 +104,25 @@ export default {
         "camera_stage_mapping",
         "image_to_stage_displacement_matrix",
       );
-      this.csmResolution = await this.readThingProperty("camera_stage_mapping", "image_resolution");
-      let streamResolution = await this.readThingProperty("camera", "stream_resolution");
-      csmMatrix[0][0] = Number(csmMatrix[0][0].toFixed(3));
-      csmMatrix[0][1] = Number(csmMatrix[0][1].toFixed(3));
-      csmMatrix[1][0] = Number(csmMatrix[1][0].toFixed(3));
-      csmMatrix[1][1] = Number(csmMatrix[1][1].toFixed(3));
-      this.csmMatrix = csmMatrix;
-      this.csmRatio = Number((Math.abs(csmMatrix[1][0]) + Math.abs(csmMatrix[0][1])) / 2).toFixed(
-        3,
-      );
-      this.csmFOV = [
-        Number(((streamResolution[0] ** 2 * this.csmRatio) / this.csmResolution[1]).toFixed(0)),
-        Number(((streamResolution[1] ** 2 * this.csmRatio) / this.csmResolution[0]).toFixed(0)),
-      ];
+      if (csmMatrix) {
+        this.csmResolution = await this.readThingProperty(
+          "camera_stage_mapping",
+          "image_resolution",
+        );
+        csmMatrix[0][0] = Number(csmMatrix[0][0].toFixed(3));
+        csmMatrix[0][1] = Number(csmMatrix[0][1].toFixed(3));
+        csmMatrix[1][0] = Number(csmMatrix[1][0].toFixed(3));
+        csmMatrix[1][1] = Number(csmMatrix[1][1].toFixed(3));
+        this.csmMatrix = csmMatrix;
+        this.csmRatio = Number((Math.abs(csmMatrix[1][0]) + Math.abs(csmMatrix[0][1])) / 2).toFixed(
+          3,
+        );
+        // Note [1] then [0] as CSM reports resolution as (y, x)
+        this.csmFOV = [
+          Number((this.csmResolution[1] * this.csmRatio).toFixed(0)),
+          Number((this.csmResolution[0] * this.csmRatio).toFixed(0)),
+        ];
+      }
     },
     onRecalibrateResponse: function () {
       this.modalNotify("Finished stage-to-camera calibration.");

From e1172248b0370ba3191349eddd38f15e12f6701e Mon Sep 17 00:00:00 2001
From: Julian Stirling <julian@julianstirling.co.uk>
Date: Tue, 4 Nov 2025 20:13:46 +0000
Subject: [PATCH 2/3] Camera stage mapping step in wizard says skip until
 complete.

---
 .../serverSpecifiedActionButton.vue           | 19 ++++++++++++++++++-
 .../calibrationWizardTask.vue                 | 13 ++++++++++++-
 .../csmSteps/runCsmStep.vue                   | 16 +++++++++++++++-
 .../CSMCalibrationSettings.vue                |  3 ++-
 4 files changed, 47 insertions(+), 4 deletions(-)

diff --git a/webapp/src/components/labThingsComponents/serverSpecifiedActionButton.vue b/webapp/src/components/labThingsComponents/serverSpecifiedActionButton.vue
index d855092b..a49d3a0a 100644
--- a/webapp/src/components/labThingsComponents/serverSpecifiedActionButton.vue
+++ b/webapp/src/components/labThingsComponents/serverSpecifiedActionButton.vue
@@ -11,6 +11,7 @@
     :modal-progress="actionData.modal_progress"
     @response="actionResponse"
     @error="modalError"
+    @finished="actionFinished"
   />
 </template>
 
@@ -33,11 +34,27 @@ export default {
   },
 
   methods: {
-    actionResponse: function () {
+    /**
+     * Runs when the ActionButton's action completes successfully.
+     *
+     * It is used to send success notifications. It also forwards the response to the
+     * parent.
+     */
+    actionResponse: function (response) {
       if (this.actionData.notify_on_success) {
         this.modalNotify(this.actionData.success_message);
+        this.$emit("response", response);
       }
     },
+    /**
+     * Runs when the ActionButton's action finishes in any way (error, cancel,
+     * completion).
+     *
+     * This forwards the event to the parent
+     */
+    actionFinished: function () {
+      this.$emit("finished");
+    },
   },
 };
 </script>
diff --git a/webapp/src/components/modalComponents/calibrationWizardComponents/calibrationWizardTask.vue b/webapp/src/components/modalComponents/calibrationWizardComponents/calibrationWizardTask.vue
index 57151b7b..f2345223 100644
--- a/webapp/src/components/modalComponents/calibrationWizardComponents/calibrationWizardTask.vue
+++ b/webapp/src/components/modalComponents/calibrationWizardComponents/calibrationWizardTask.vue
@@ -27,6 +27,7 @@ gets very confusing.
       v-if="currentStep"
       :key="stepIndex"
       v-bind="currentStep.props"
+      @awaiting-user="handleAwaitingUser"
     />
     <p class="uk-text-right">
       <button
@@ -68,6 +69,7 @@ export default {
   data() {
     return {
       stepIndex: this.startOnLast ? this.steps.length - 1 : 0,
+      stepAwaitingUser: false,
     };
   },
 
@@ -79,7 +81,11 @@ export default {
       return !this.first || this.stepIndex > 0;
     },
     nextButtonText() {
-      return this.final && this.stepIndex === this.steps.length - 1 ? "Finish" : "Next";
+      const isLastStep = this.final && this.stepIndex === this.steps.length - 1;
+      if (this.stepAwaitingUser) {
+        return isLastStep ? "Skip and Finish" : "Skip";
+      }
+      return isLastStep ? "Finish" : "Next";
     },
   },
 
@@ -88,6 +94,7 @@ export default {
      * Move to the previous step in this task, or the previous task if first step.
      */
     previousStep: function () {
+      this.stepAwaitingUser = false;
       if (this.stepIndex > 0) {
         this.stepIndex = this.stepIndex - 1;
       } else {
@@ -99,6 +106,7 @@ export default {
      * Move to the next step in this task, or the next task if final step.
      */
     nextStep: function () {
+      this.stepAwaitingUser = false;
       if (this.stepIndex < this.steps.length - 1) {
         this.stepIndex = this.stepIndex + 1;
         return true;
@@ -106,6 +114,9 @@ export default {
         this.$emit("next");
       }
     },
+    handleAwaitingUser(isAwaiting) {
+      this.stepAwaitingUser = isAwaiting;
+    },
   },
 };
 </script>
diff --git a/webapp/src/components/modalComponents/calibrationWizardComponents/csmSteps/runCsmStep.vue b/webapp/src/components/modalComponents/calibrationWizardComponents/csmSteps/runCsmStep.vue
index 4f285d78..25dcdee0 100644
--- a/webapp/src/components/modalComponents/calibrationWizardComponents/csmSteps/runCsmStep.vue
+++ b/webapp/src/components/modalComponents/calibrationWizardComponents/csmSteps/runCsmStep.vue
@@ -6,7 +6,10 @@
     <p>If it is not in focus, click back and re-focus.</p>
     <template #below-stream>
       <div class="action-button-container">
-        <CSMCalibrationSettings :show-extra-settings="false" />
+        <CSMCalibrationSettings
+          :show-extra-settings="false"
+          @recalibrateResponse="checkCalibrationState"
+        />
       </div>
     </template>
   </stepTemplateWithStream>
@@ -23,6 +26,17 @@ export default {
     stepTemplateWithStream,
     CSMCalibrationSettings,
   },
+
+  mounted() {
+    this.checkCalibrationState();
+  },
+
+  methods: {
+    async checkCalibrationState() {
+      const needsCal = await this.readThingProperty("camera_stage_mapping", "calibration_required");
+      this.$emit("awaiting-user", needsCal);
+    },
+  },
 };
 </script>
 
diff --git a/webapp/src/components/tabContentComponents/settingsComponents/CSMSettingsComponents/CSMCalibrationSettings.vue b/webapp/src/components/tabContentComponents/settingsComponents/CSMSettingsComponents/CSMCalibrationSettings.vue
index 953473c4..0920d02f 100644
--- a/webapp/src/components/tabContentComponents/settingsComponents/CSMSettingsComponents/CSMCalibrationSettings.vue
+++ b/webapp/src/components/tabContentComponents/settingsComponents/CSMSettingsComponents/CSMCalibrationSettings.vue
@@ -124,9 +124,10 @@ export default {
         ];
       }
     },
-    onRecalibrateResponse: function () {
+    onRecalibrateResponse: function (response) {
       this.modalNotify("Finished stage-to-camera calibration.");
       this.updateDisplayedCSM();
+      this.$emit("recalibrateResponse", response);
     },
   },
 };

From 1f22346ce6da389b24194b8c4837478a8cd7a09f Mon Sep 17 00:00:00 2001
From: Julian Stirling <julian@julianstirling.co.uk>
Date: Tue, 4 Nov 2025 20:31:16 +0000
Subject: [PATCH 3/3] Camera calibration step in wizard skip until calibrated.

---
 .../cameraMainCalibrationStep.vue               | 17 ++++++++++++++++-
 .../cameraCalibrationSettings.vue               |  4 ++--
 2 files changed, 18 insertions(+), 3 deletions(-)

diff --git a/webapp/src/components/modalComponents/calibrationWizardComponents/cameraCalibrationSteps/cameraMainCalibrationStep.vue b/webapp/src/components/modalComponents/calibrationWizardComponents/cameraCalibrationSteps/cameraMainCalibrationStep.vue
index e393eead..0656597f 100644
--- a/webapp/src/components/modalComponents/calibrationWizardComponents/cameraCalibrationSteps/cameraMainCalibrationStep.vue
+++ b/webapp/src/components/modalComponents/calibrationWizardComponents/cameraCalibrationSteps/cameraMainCalibrationStep.vue
@@ -6,7 +6,11 @@
 
     <template #below-stream>
       <div class="action-button-container">
-        <cameraCalibrationSettings :show-extra-settings="false" :camera-uri="cameraUri" />
+        <cameraCalibrationSettings
+          :show-extra-settings="false"
+          :camera-uri="cameraUri"
+          @actionFinished="checkCalibrationState"
+        />
       </div>
     </template>
   </stepTemplateWithStream>
@@ -29,6 +33,17 @@ export default {
       return `${this.$store.getters.baseUri}/camera/`;
     },
   },
+
+  mounted() {
+    this.checkCalibrationState();
+  },
+
+  methods: {
+    async checkCalibrationState() {
+      const needsCal = await this.readThingProperty("camera", "calibration_required");
+      this.$emit("awaiting-user", needsCal);
+    },
+  },
 };
 </script>
 
diff --git a/webapp/src/components/tabContentComponents/settingsComponents/cameraSettingsComponents/cameraCalibrationSettings.vue b/webapp/src/components/tabContentComponents/settingsComponents/cameraSettingsComponents/cameraCalibrationSettings.vue
index 9bc847de..710897e8 100644
--- a/webapp/src/components/tabContentComponents/settingsComponents/cameraSettingsComponents/cameraCalibrationSettings.vue
+++ b/webapp/src/components/tabContentComponents/settingsComponents/cameraSettingsComponents/cameraCalibrationSettings.vue
@@ -6,7 +6,7 @@
       :key="'primary_cal' + index"
       class="uk-child-width-expand"
     >
-      <server-specified-action-button :action-data="action" />
+      <server-specified-action-button :action-data="action" @finished="$emit('actionFinished')" />
     </div>
     <div v-if="showExtraSettings">
       <div
@@ -14,7 +14,7 @@
         :key="'secondary_cal' + index"
         class="uk-child-width-expand"
       >
-        <server-specified-action-button :action-data="action" />
+        <server-specified-action-button :action-data="action" @finished="$emit('actionFinished')" />
       </div>
     </div>
   </div>