Skip to content

update output structure#36

Merged
ch99l merged 4 commits into
mainfrom
devel
Jul 9, 2026
Merged

update output structure#36
ch99l merged 4 commits into
mainfrom
devel

Conversation

@ch99l

@ch99l ch99l commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the pipeline’s output layout to group quantification results into per-assay subdirectories and exports count matrices in 10x-compatible Matrix Market format, aligning on-disk outputs with downstream single-cell tooling expectations.

Changes:

  • Restructures output directories for unique/gene/transcript counts and cluster-level results.
  • Adds save_counts.R helper to export SummarizedExperiment assays to *.mtx.gz + barcodes.tsv.gz/features.tsv.gz alongside the .rds.
  • Updates the R container to include DropletUtils and refreshes README output documentation.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
README.md Documents the new output directory layout and matrix outputs.
modules/bambu/transcript_discovery.nf Writes unique/gene counts into dedicated directories and publishes them.
modules/bambu/EM_quant.nf Writes EM quant outputs into structured directories and publishes them.
containers/r/Dockerfile Adds bioconductor-dropletutils dependency required for matrix export.
bin/save_counts.R New helper for exporting 10x-style matrices and saving per-directory .rds outputs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread modules/bambu/transcript_discovery.nf
Comment thread modules/bambu/transcript_discovery.nf
Comment thread modules/bambu/EM_quant.nf
Comment thread README.md Outdated
@ch99l ch99l merged commit 6e11291 into main Jul 9, 2026
28 checks passed
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