Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update RGs in Mammoth BAMs. update samplesheet BAM path. #1141

Merged
merged 1 commit into from
Mar 19, 2024
Merged

Update RGs in Mammoth BAMs. update samplesheet BAM path. #1141

merged 1 commit into from
Mar 19, 2024

Conversation

TCLamnidis
Copy link

@TCLamnidis TCLamnidis commented Mar 19, 2024 •
edited
Loading

Updated Mammoth BAMs to follow the latest RG setup see this PR

Updated test samplesheet for DSL2 to now use the BAM with the updated RGs

i.e.:

@RG	ID:ILLUMINA-JK2782_JK2782_TGGCCGATCAACGA_BAM	SM:JK2782	LB:JK2782_TGGCCGATCAACGA_BAM	PL:illumina	PU:ILLUMINA-JK2782_TGGCCGATCAACGA_BAM-double_stranded-SE

and

@RG	ID:ILLUMINA-JK2802_JK2802_AGAATAACCTACCA_BAM	SM:JK2802	LB:JK2802_AGAATAACCTACCA_BAM	PL:illumina	PU:ILLUMINA-JK2802_AGAATAACCTACCA_BAM-double_stranded-SE

@TCLamnidis TCLamnidis mentioned this pull request Mar 19, 2024
Merged
27 tasks
scarlhoff
scarlhoff approved these changes Mar 19, 2024
View reviewed changes
Copy link

@scarlhoff scarlhoff left a comment

Choose a reason for hiding this comment

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

👍

@scarlhoff scarlhoff merged commit 1d49272 into nf-core:eager Mar 19, 2024
@TCLamnidis TCLamnidis mentioned this pull request Mar 20, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@scarlhoff scarlhoff scarlhoff approved these changes

@jbv2 jbv2 Awaiting requested review from jbv2

@jfy133 jfy133 Awaiting requested review from jfy133

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@TCLamnidis @scarlhoff