fix: don't require queried for HGNC IDs to be present (#418)

varfish-org · Mar 1, 2024 · 18894ba · 18894ba
1 parent a0ec18e
commit 18894ba
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 17 deletions.
diff --git a/src/server/actix_server/genes_clinvar.rs b/src/server/actix_server/genes_clinvar.rs
@@ -55,15 +55,17 @@ async fn handle(
     let mut genes = indexmap::IndexMap::new();
     if let Some(hgnc_id) = query.hgnc_id.as_ref() {
         for hgnc_id in hgnc_id {
-            let raw_buf = db_clinvar
-                .get_cf(&cf_genes, hgnc_id)
-                .map_err(|e| CustomError::new(anyhow::anyhow!("problem querying database: {}", e)))?
-                .ok_or_else(|| CustomError::new(anyhow::anyhow!("no such gene: {}", hgnc_id)))?;
-            let record = crate::pbs::clinvar::per_gene::ClinvarPerGeneRecord::decode(
-                std::io::Cursor::new(raw_buf),
-            )
-            .map_err(|e| CustomError::new(anyhow::anyhow!("problem decoding value: {}", e)))?;
-            genes.insert(hgnc_id.to_string(), record);
+            if let Some(raw_buf) = db_clinvar.get_cf(&cf_genes, hgnc_id).map_err(|e| {
+                CustomError::new(anyhow::anyhow!("problem querying database: {}", e))
+            })? {
+                let record = crate::pbs::clinvar::per_gene::ClinvarPerGeneRecord::decode(
+                    std::io::Cursor::new(raw_buf),
+                )
+                .map_err(|e| CustomError::new(anyhow::anyhow!("problem decoding value: {}", e)))?;
+                genes.insert(hgnc_id.to_string(), record);
+            } else {
+                tracing::debug!("no such gene: {}", hgnc_id);
+            }
         }
     }
 

diff --git a/src/server/actix_server/genes_info.rs b/src/server/actix_server/genes_info.rs
@@ -50,14 +50,17 @@ async fn handle(
     let mut genes = indexmap::IndexMap::new();
     if let Some(hgnc_id) = query.hgnc_id.as_ref() {
         for hgnc_id in hgnc_id {
-            let raw_buf = genes_db
-                .db
-                .get_cf(&cf_genes, hgnc_id)
-                .map_err(|e| CustomError::new(anyhow::anyhow!("problem querying database: {}", e)))?
-                .ok_or_else(|| CustomError::new(anyhow::anyhow!("no such gene: {}", hgnc_id)))?;
-            let record = genes::base::Record::decode(std::io::Cursor::new(raw_buf))
-                .map_err(|e| CustomError::new(anyhow::anyhow!("problem decoding value: {}", e)))?;
-            genes.insert(hgnc_id.to_string(), record);
+            if let Some(raw_buf) = genes_db.db.get_cf(&cf_genes, hgnc_id).map_err(|e| {
+                CustomError::new(anyhow::anyhow!("problem querying database: {}", e))
+            })? {
+                let record =
+                    genes::base::Record::decode(std::io::Cursor::new(raw_buf)).map_err(|e| {
+                        CustomError::new(anyhow::anyhow!("problem decoding value: {}", e))
+                    })?;
+                genes.insert(hgnc_id.to_string(), record);
+            } else {
+                tracing::debug!("no such gene: {}", hgnc_id);
+            }
         }
     }