Merge pull request #121 from CCBR/iss-120

Hotfix: Bug in assess_signficance.R script
CCBR · Oct 3, 2024 · 2e60f68 · 2e60f68
2 parents 9403e67 + 886e6b2
commit 2e60f68
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,7 @@
 ## XAVIER development version
-
+- Bug fixes:
+  - Fix minor bug in assess_significance.R script associated with rule freec_exome_somatic (#120, @samarth8392)
+
 ## XAVIER 3.1.1
 
 - New contributing guide available on GitHub and the documentation website. (#114, @kelly-sovacool)

diff --git a/workflow/scripts/assess_significance.R b/workflow/scripts/assess_significance.R
@@ -48,7 +48,7 @@ for (i in c(1:length(cnvs[, 1]))) {
   }
   KS <- function(values, normals) {
     resultks <- try(ks.test(values, score(normals)), silent = TRUE)
-    if (class(resultks) == "try-error") {
+    if (paste(class(resultks), collapse="_") == "try-error") {
       return(list("statistic" = NA, "p.value" = NA, "alternative" = NA, "method" = NA, "data.name" = NA))
     } else {
       resultks