Skip to content

Commit aabad67

Browse files
committed
fix error that was breaking build
1 parent 36d9a81 commit aabad67

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

R/AllChecks.R

+4-3
Original file line numberDiff line numberDiff line change
@@ -115,9 +115,10 @@
115115
if(object@merge_index==object@merge_dendrocluster_index) return("merge_index should not be same as merge_dendrocluster_index")
116116
if(!length(object@merge_method)==1) return("merge_method must be of length 1")
117117
if(!object@merge_method %in% .availMergeMethods) return(paste("merge_method must be one of available merge methods:", paste(.availMergeMethods,collapse=",")))
118-
# if(ncol(object@merge_nodeMerge)!=4 || any(sort(colnames(object@merge_nodeMerge)) != c('Contrast','isMerged','mergeClusterId','Node') )) {
119-
# return("merge_nodeMerge must have 4 columns and column names equal to: 'Node','Contrast','isMerged','mergeClusterId'")
120-
# }
118+
allowMergeColumns<-c('Contrast','isMerged','mergeClusterId','Node')
119+
if(ncol(object@merge_nodeMerge)!=length(allowMergeColumns) || any(sort(colnames(object@merge_nodeMerge)) != sort(allowMergeColumns))) {
120+
return(paste("merge_nodeMerge must have 4 columns and column names equal to:",paste(allowMergeColumns,collapse=",")))
121+
}
121122
if(!is.character(object@merge_nodeMerge[,"Node"])) return("'Node' column of merge_nodeMerge must be character")
122123
if(!is.character(object@merge_nodeMerge[,"Contrast"])) return("'Contrast' column of merge_nodeMerge must be character")
123124
if(!is.logical(object@merge_nodeMerge[,"isMerged"])) return("'isMerged' column of merge_nodeMerge must be character")

0 commit comments

Comments
 (0)