Skip to content

Assorted fixes to combineCols and combineRows. - #98

Open
LTLA wants to merge 3 commits into
Bioconductor:develfrom
LTLA:combine-fixes
Open

Assorted fixes to combineCols and combineRows.#98
LTLA wants to merge 3 commits into
Bioconductor:develfrom
LTLA:combine-fixes

Conversation

@LTLA

@LTLA LTLA commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Partially motivated by libscran/scrapper#35.

@DarioS

DarioS commented Sep 7, 2026

Copy link
Copy Markdown

Also, even with SVT_SparseMatrix, I can't combine published G.E.O. data sets.

length(humanList)
    19
table(sapply(sapply(humanList, counts), class))
     SVT_SparseMatrix 
                   19
table(sapply(sapply(humanList, logcounts), class)) # Created by analyze.se
    DelayedMatrix 
               19
database <- Reduce(function(x, y) {combineCols(x, y, delayed = FALSE, fill = 0)}, unname(humanList))
            Error: C stack usage 7969312 is too close to the limit

I am aiming to do library(tiledbsoma) and write_soma(database) afterwards.

@LTLA

LTLA commented Sep 7, 2026

Copy link
Copy Markdown
Contributor Author

Seems like a separate issue, let's stay on topic here.

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.

2 participants