HenrikBengtsson / matrixStats

R package: Methods that Apply to Rows and Columns of Matrices (and to Vectors)
https://cran.r-project.org/package=matrixStats
203 stars 33 forks source link

Error message reports wrong number of rows/cols #165

Closed hpages closed 4 years ago

hpages commented 4 years ago

Using the current version of matrixStats (commit 286d6980):

library(matrixStats)

rowMedians(array(1:24, 4:2), dim.=c(-3L, 8L))
# Error in rowMedians(array(1:24, 4:2), dim. = c(-3L, 8L)) : 
#   Argument 'dim.' specifies a negative number of rows (dim.[1]): 13

rowMedians(array(1:24, 4:2), dim.=c(3L, -8L))
# Error in rowMedians(array(1:24, 4:2), dim. = c(3L, -8L)) : 
#   Argument 'dim.' specifies a negative number of columns (dim.[2]): 13

sessionInfo()

R version 3.6.0 Patched (2019-05-02 r76454)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 16.04.5 LTS

Matrix products: default
BLAS:   /home/hpages/R/R-3.6.r76454/lib/libRblas.so
LAPACK: /home/hpages/R/R-3.6.r76454/lib/libRlapack.so

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C              
 [3] LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8    
 [5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8   
 [7] LC_PAPER=en_US.UTF-8       LC_NAME=C                 
 [9] LC_ADDRESS=C               LC_TELEPHONE=C            
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C       

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base     

other attached packages:
[1] matrixStats_0.55.0-9000

loaded via a namespace (and not attached):
[1] compiler_3.6.0
HenrikBengtsson commented 4 years ago

Thxs. Fixed in commit 0bf5ebd.

Happy New Year to you too