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.

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

Nanopore Fastq Host Removal Pipeline #14

Merged
merged 13 commits into from
Jan 5, 2022
Merged

Conversation

DarianHole
Copy link
Member

@DarianHole DarianHole commented Nov 22, 2021

Summary:

Adding a pipeline to do nanopore fastq host removal

Changes:

  • Wrote new host removal process based on minimap2
    • Fast5 host removal available
    • Folder structure of the output matches nanostripper
  • Variable name changes and additions
    • More to come in changelog and after adjustments to the processes here as names and configs are a bit messy
  • pysam added to nanopore environment
  • Added a help command
  • Adjusted resources for each process along with new tags and labels
  • Update CHANGELOG
  • Update README

Potential Changes:

  • Can maybe switch to 1 dehost script and have functions for nanopore and illumina inputs

Copy link
Collaborator

@ConnorChato ConnorChato left a comment

Choose a reason for hiding this comment

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

May not have too many suggestions for the nextflow code, but in general the obvious stuff checks out.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
bin/dehost_nanopore.py Outdated Show resolved Hide resolved
environments/nanopore.yml Outdated Show resolved Hide resolved
modules/help.nf Show resolved Hide resolved
@DarianHole DarianHole merged commit f1755a5 into master Jan 5, 2022
@DarianHole DarianHole deleted the add/nanopore_fastq_process branch January 5, 2022 21:51
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.

None yet

2 participants