This warning " stop("color limits should cover matrix") " have some issues,
while the code only used the comparison of greater than the min and smaller than the max, in order to determine should it be raise the warning and stop the program.
Would that be inclusive, such as if user provided a custom limit c(0, 1). Then, there might be chance that getting 0 or 1.
In the current design of the program will restrict the user to use this range, unless they specify with c(-0.001, 1.001).
The current code is
`
if (any(corr < cl.lim[1]) || any(corr > cl.lim[2])) {
stop("color limits should cover matrix")
}
This warning " stop("color limits should cover matrix") " have some issues, while the code only used the comparison of greater than the min and smaller than the max, in order to determine should it be raise the warning and stop the program.
Would that be inclusive, such as if user provided a custom limit c(0, 1). Then, there might be chance that getting 0 or 1. In the current design of the program will restrict the user to use this range, unless they specify with c(-0.001, 1.001).
The current code is
`
if (any(corr < cl.lim[1]) || any(corr > cl.lim[2])) { stop("color limits should cover matrix") }
`