Skip to content
This repository has been archived by the owner on Apr 19, 2023. It is now read-only.

Cellranger updates #304

Merged
merged 7 commits into from
Feb 17, 2021
Merged

Cellranger updates #304

merged 7 commits into from
Feb 17, 2021

Conversation

KrisDavie
Copy link
Member

  • Enables use of --include-introns and --no-bam from cellranger 5.0
  • Checks for presence of cellranger container before attempting to run anything
  • Add small note to dependencies about using cellranger component
  • Adds VSC profile for use on the Flemish Super Computer (KUL-tier2)

@KrisDavie KrisDavie requested review from cflerin and dweemx February 10, 2021 11:12
dweemx
dweemx suggested changes Feb 11, 2021
View reviewed changes
Copy link
Contributor

@dweemx dweemx left a comment

Choose a reason for hiding this comment

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

Small changes requested then I'd be happy to approve the PR

src/cellranger/processes/count.nf Outdated Show resolved Hide resolved
src/cellranger/processes/count.nf Outdated Show resolved Hide resolved
src/cellranger/processes/count.nf Outdated Show resolved Hide resolved
src/cellranger/processes/count.nf Outdated Show resolved Hide resolved
@KrisDavie KrisDavie requested a review from dweemx February 17, 2021 14:24
@KrisDavie KrisDavie merged commit ec1fb30 into develop Feb 17, 2021
@cflerin cflerin deleted the cellranger_updates branch February 18, 2021 13:03
@cflerin cflerin mentioned this pull request Jun 29, 2021
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cflerin cflerin cflerin approved these changes

@dweemx dweemx dweemx approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@KrisDavie @cflerin @dweemx