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

New Subworkflow: Bam_trim_primers_ivar #3130

Closed
wants to merge 10 commits into from
Closed

New Subworkflow: Bam_trim_primers_ivar #3130

wants to merge 10 commits into from

Conversation

Joon-Klaps
Copy link
Contributor

@Joon-Klaps Joon-Klaps commented Mar 27, 2023 •
edited
Loading

PR checklist

Addresses first bullet from nf-core/viralrecon#371

adds the subworkflow bam_trim_primers_ivar used in nf-core/viralrecon to nf-core/modules

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the module conventions in the contribution docs
  • Remove all TODO statements.
  • Emit the versions.yml file.
  • Follow the naming conventions.
  • Follow the parameters requirements.
  • Follow the input/output options guidelines.
  • Ensure that the test works with either Docker / Singularity. Conda CI tests can be quite flaky:
    • PROFILE=docker pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware
    • PROFILE=singularity pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware
    • PROFILE=conda pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware

@Joon-Klaps Joon-Klaps self-assigned this Mar 27, 2023
@Joon-Klaps Joon-Klaps marked this pull request as draft March 27, 2023 13:26
@Joon-Klaps Joon-Klaps added the hackathon2023 Topic for the hackathon of March 2023 label Mar 27, 2023
@Joon-Klaps Joon-Klaps marked this pull request as ready for review March 27, 2023 13:53
nvnieuwk
nvnieuwk previously requested changes Mar 27, 2023
View reviewed changes
subworkflows/nf-core/bam_trim_primers_ivar/main.nf Outdated Show resolved Hide resolved
subworkflows/nf-core/bam_trim_primers_ivar/main.nf Outdated Show resolved Hide resolved
subworkflows/nf-core/bam_trim_primers_ivar/meta.yml Show resolved Hide resolved
@Joon-Klaps Joon-Klaps changed the title Bam trim primers ivar New Subworkflow: Bam_trim_primers_ivar Mar 27, 2023
nvnieuwk
nvnieuwk reviewed Mar 28, 2023
View reviewed changes
Copy link
Contributor

@nvnieuwk nvnieuwk left a comment

Choose a reason for hiding this comment

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

One more small comment and then you are good to go!

subworkflows/nf-core/bam_trim_primers_ivar/main.nf Outdated Show resolved Hide resolved
subworkflows/nf-core/bam_trim_primers_ivar/meta.yml Show resolved Hide resolved
@Joon-Klaps
Copy link
Contributor Author

Not Sure why conda test is failing though..

@nvnieuwk
Copy link
Contributor

It seems to be something tool specific. Maybe a wrong conda package?

@Joon-Klaps
Copy link
Contributor Author

It seems to be something tool specific. Maybe a wrong conda package?

Yeah... doesn't seem something I'll be able to finish as part of this PR... andersen-lab/ivar#93

@edmundmiller edmundmiller requested a review from a team as a code owner October 19, 2023 16:26
@sateeshperi sateeshperi added the mismatched hashes Conda/singularity hashes aren't right label Dec 25, 2023
@Joon-Klaps Joon-Klaps closed this Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nvnieuwk nvnieuwk nvnieuwk left review comments

@sateeshperi sateeshperi Awaiting requested review from sateeshperi sateeshperi is a code owner automatically assigned from nf-core/nf-test

Assignees

@Joon-Klaps Joon-Klaps

Labels
hackathon2023 Topic for the hackathon of March 2023 mismatched hashes Conda/singularity hashes aren't right new subworkflow
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Joon-Klaps @nvnieuwk @SPPearce @sateeshperi @maxulysse