Merge pull request #5508 from pcm32/patch-7

Fix EdgeR: Deal with factor names on unified factor file
galaxyproject · Sep 27, 2023 · 95a36f5 · 95a36f5
2 parents 2dc6baa + 4b3d765
commit 95a36f5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/tools/edger/edger.R b/tools/edger/edger.R
@@ -283,7 +283,7 @@ if (!is.null(opt$filesPath)) {
     }
     # order samples as in counts matrix
     factordata <- factordata[match(colnames(counts), factordata[, 1]), ]
-    factors <- factordata[, -1, drop = FALSE]
+    factors <- data.frame(sapply(factordata[, -1, drop = FALSE], make.names))
   } else {
     factors <- unlist(strsplit(opt$factInput, "|", fixed = TRUE))
     factordata <- list()

diff --git a/tools/edger/edger.xml b/tools/edger/edger.xml
@@ -4,7 +4,7 @@
     </description>
     <macros>
         <token name="@TOOL_VERSION@">3.36.0</token>
-        <token name="@VERSION_SUFFIX@">1</token>
+        <token name="@VERSION_SUFFIX@">2</token>
     </macros>
     <edam_topics>
         <edam_topic>topic_3308</edam_topic>