From 72c1367f4fa246c167ea986b71dc35740bdc43ad Mon Sep 17 00:00:00 2001
From: Kieron Taylor <kt19@sanger.ac.uk>
Date: Mon, 22 Jul 2024 16:31:15 +0000
Subject: [PATCH 1/8] Fix stdev function throwing when n=1 for a pool

---
 lang_qc/models/pacbio/qc_data.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/lang_qc/models/pacbio/qc_data.py b/lang_qc/models/pacbio/qc_data.py
index ca29192..a61c57f 100644
--- a/lang_qc/models/pacbio/qc_data.py
+++ b/lang_qc/models/pacbio/qc_data.py
@@ -227,7 +227,9 @@ def pre_root(cls, values: dict[str, Any]) -> dict[str, Any]:
                 cov = None
             else:
                 hifi_reads = [prod.hifi_num_reads for prod in product_metrics]
-                cov = round(stdev(hifi_reads) / mean(hifi_reads) * 100, 2)
+                if len(hifi_reads) > 1:
+                    # stdev throws on n=1
+                    cov = round(stdev(hifi_reads) / mean(hifi_reads) * 100, 2)
 
             for i, prod in enumerate(product_metrics):
                 sample_stats.append(

From 9203da1c7244bbf8db79f59fd758cab2d298c6ac Mon Sep 17 00:00:00 2001
From: Kieron Taylor <kt19@sanger.ac.uk>
Date: Mon, 22 Jul 2024 17:17:37 +0000
Subject: [PATCH 2/8] Give no data when pool size is zero, rather than sparse
 dict.

---
 lang_qc/endpoints/pacbio_well.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lang_qc/endpoints/pacbio_well.py b/lang_qc/endpoints/pacbio_well.py
index e72e7c9..93ebdab 100644
--- a/lang_qc/endpoints/pacbio_well.py
+++ b/lang_qc/endpoints/pacbio_well.py
@@ -236,7 +236,8 @@ def get_product_metrics(
         metrics = QCPoolMetrics(db_well=mlwh_well)
     except MissingLimsDataError:
         return
-
+    if len(metrics.products) == 0:
+        return
     return metrics
 
 

From 06eb2da6c747c4527e9d9dea0c937a91cbbe4e2f Mon Sep 17 00:00:00 2001
From: Kieron Taylor <kt19@sanger.ac.uk>
Date: Mon, 22 Jul 2024 17:31:55 +0000
Subject: [PATCH 3/8] Skip pool stats fetch if sample size is 0. Nothing useful
 happens

---
 frontend/src/components/QcView.vue | 34 ++++++++++++++++--------------
 1 file changed, 18 insertions(+), 16 deletions(-)

diff --git a/frontend/src/components/QcView.vue b/frontend/src/components/QcView.vue
index ce0a84c..2f70161 100644
--- a/frontend/src/components/QcView.vue
+++ b/frontend/src/components/QcView.vue
@@ -18,22 +18,6 @@
         well: Object,
     })
 
-    const poolStats = ref(null)
-    watch(() => props.well, () => {
-        poolStats.value = null // empty in case next well doesn't have a pool
-        dataClient.getPoolMetrics(props.well.id_product).then(
-            (response) => { poolStats.value = response }
-        ).catch((error) => {
-            if (error.message.match("Conflict")) {
-                // Nothing to do
-            } else {
-                console.log(error)
-                // make a banner show this error?
-            }
-        })
-    }, { immediate: true}
-    )
-
     const slURL = computed(() => {
         let hostname = props.well.metrics.smrt_link.hostname
         let url = ''
@@ -118,6 +102,24 @@
         }
         return ''
     })
+
+    const poolStats = ref(null)
+    watch(() => props.well, () => {
+        poolStats.value = null // empty in case next well doesn't have a pool
+        if (ssLimsNumSamples.value > 0) {
+            dataClient.getPoolMetrics(props.well.id_product).then(
+                (response) => { poolStats.value = response }
+            ).catch((error) => {
+                if (error.message.match("Conflict")) {
+                    // Nothing to do
+                } else {
+                    console.log(error)
+                    // make a banner show this error?
+                }
+            })
+        }
+    }, { immediate: true }
+    )
 </script>
 
 <template>

From 3e4a669d1531cf646a952e2114a6a6273c715195 Mon Sep 17 00:00:00 2001
From: Kieron Taylor <kt19@sanger.ac.uk>
Date: Wed, 24 Jul 2024 15:14:39 +0000
Subject: [PATCH 4/8] focusWell fetch errors were not getting their type due to
 mistake.

---
 frontend/src/stores/focusWell.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/stores/focusWell.js b/frontend/src/stores/focusWell.js
index 97bd65e..c08c164 100644
--- a/frontend/src/stores/focusWell.js
+++ b/frontend/src/stores/focusWell.js
@@ -75,7 +75,7 @@ export const useWellStore = defineStore('focusWell', {
                 (error) => {
                     ElMessage({
                         message: error.message,
-                        type: error,
+                        type: "error",
                         duration: 5000
                     })
                 }

From ca133918c47e753778f38abe5894ce6389450f88 Mon Sep 17 00:00:00 2001
From: Kieron Taylor <kt19@sanger.ac.uk>
Date: Wed, 24 Jul 2024 15:25:40 +0000
Subject: [PATCH 5/8] In some failed runs, hifi_read_bases can be null, which
 breaks the maths

---
 lang_qc/models/pacbio/qc_data.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/lang_qc/models/pacbio/qc_data.py b/lang_qc/models/pacbio/qc_data.py
index a61c57f..444038b 100644
--- a/lang_qc/models/pacbio/qc_data.py
+++ b/lang_qc/models/pacbio/qc_data.py
@@ -238,7 +238,11 @@ def pre_root(cls, values: dict[str, Any]) -> dict[str, Any]:
                         tag1_name=lib_lims_data[i].tag_identifier,
                         tag2_name=lib_lims_data[i].tag2_identifier,
                         deplexing_barcode=prod.barcode4deplexing,
-                        hifi_read_bases=convert_to_gigabase(prod, "hifi_read_bases"),
+                        hifi_read_bases=(
+                            convert_to_gigabase(prod, "hifi_read_bases")
+                            if (prod.hifi_read_bases)
+                            else None
+                        ),
                         hifi_num_reads=prod.hifi_num_reads,
                         hifi_read_length_mean=prod.hifi_read_length_mean,
                         hifi_bases_percent=prod.hifi_bases_percent,

From 0129d69d53497c062b6b960d8e609b36dba80204 Mon Sep 17 00:00:00 2001
From: Kieron Taylor <kt19@sanger.ac.uk>
Date: Wed, 24 Jul 2024 15:55:24 +0000
Subject: [PATCH 6/8] Fix a nav bug where searching for runs remove the well
 detail query, but did not reset the QC View

---
 frontend/src/views/WellsByRun.vue | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/frontend/src/views/WellsByRun.vue b/frontend/src/views/WellsByRun.vue
index 56d3bac..2086b12 100644
--- a/frontend/src/views/WellsByRun.vue
+++ b/frontend/src/views/WellsByRun.vue
@@ -36,7 +36,9 @@ function flatten_data(values) {
 getUserName((email) => { user.value = email }).then();
 
 watch(() => route.query, (after, before) => {
-    if (qcQueryChanged(before, after)) {
+    if (after.idProduct === undefined) {
+      focusWell.setFocusWell(null)
+    } else if (qcQueryChanged(before, after)) {
       focusWell.loadWellDetail(after.idProduct)
     }
   },

From c7043162f2e771f708e36c921c81ebb5663a5fdc Mon Sep 17 00:00:00 2001
From: Kieron Taylor <kt19@sanger.ac.uk>
Date: Thu, 25 Jul 2024 14:51:34 +0000
Subject: [PATCH 7/8] Pool stats no longer requested during the WellsByRun
 tests

---
 frontend/src/views/__tests__/WellsByRun.spec.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/frontend/src/views/__tests__/WellsByRun.spec.js b/frontend/src/views/__tests__/WellsByRun.spec.js
index 828ae60..2abf0b5 100644
--- a/frontend/src/views/__tests__/WellsByRun.spec.js
+++ b/frontend/src/views/__tests__/WellsByRun.spec.js
@@ -138,7 +138,6 @@ describe('Does it work?', async () => {
     // Not providing precisely the right data, but serves for the component
     fetch.mockResponses(
       [JSON.stringify(secondaryRun)], // well QC data loading
-      [JSON.stringify({})] // Pool stats loading
     )
 
     let buttons = wrapper.findAll('button')
@@ -165,8 +164,9 @@ describe('Does it work?', async () => {
           page_number: 1,
           total_number_of_items: 1,
           wells: [secondaryRun]
-        })
-      ]
+        }),
+        { status: 200 }
+      ],
     )
     await wrapper.setProps({runName: ['TRACTION-RUN-211', 'TRACTION-RUN-210']})
     await flushPromises()

From a7c025acaef50714bc5fc89646d9325a1179cd37 Mon Sep 17 00:00:00 2001
From: Kieron Taylor <kt19@sanger.ac.uk>
Date: Thu, 25 Jul 2024 14:52:12 +0000
Subject: [PATCH 8/8] Error logging added to the browser when the client sees
 an error in the WellsByRun view.

---
 frontend/src/views/WellsByRun.vue | 1 +
 1 file changed, 1 insertion(+)

diff --git a/frontend/src/views/WellsByRun.vue b/frontend/src/views/WellsByRun.vue
index 2086b12..e80bab0 100644
--- a/frontend/src/views/WellsByRun.vue
+++ b/frontend/src/views/WellsByRun.vue
@@ -53,6 +53,7 @@ watch(() => props.runName, () => {
   Promise.all(promises).then(
       (values) => (wellCollection.value = flatten_data(values))
   ).catch(error => {
+      console.log(error.message + " when resolving promises in props.runName watcher")
       ElMessage({
         message: error.message,
         type: "warning",