Skip to content

Conversation

@szcf-weiya
Copy link

I noticed that the current package has introduced version argument, along with other new arguments in the addMatToArrow function,

ArchR/R/ArrowWrite.R

Lines 105 to 126 in 65968a4

.addMatToArrow <- function(
mat = NULL,
ArrowFile = NULL,
Group = NULL,
binarize = FALSE,
addRowSums = FALSE,
addColSums = FALSE,
addRowMeans = FALSE,
addRowVars = FALSE,
addColMeansLog2 = FALSE, #New
addRowMeansLog2 = FALSE, #New
addRowVarsLog2 = FALSE,
addRowMeansGeo = FALSE, #New
addRowVarsGeo = FALSE, #New
addRowSumsBinary = FALSE, #New
addColSumsBinary = FALSE, #New
addRowMeansLog2Norm = FALSE, #New
addRowVarsLog2Norm = FALSE, #New
scaleTo = 10000, #New
colSm = NULL, #New
logFile = NULL,
version = 2 #New

but the higher-level function addGeneExpressionMatrix does not support passing those new arguments.

This pull request aims to provide the option to use the hdf5 style of the old version (version = 1).

@immanuelazn immanuelazn force-pushed the dev branch 3 times, most recently from 322676f to f6fcc50 Compare January 22, 2025 05:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant