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

Add option to remove ribosomal reads #80

Merged
merged 20 commits into from
May 19, 2020
Merged

Add option to remove ribosomal reads #80

merged 20 commits into from
May 19, 2020

Conversation

olgabot
Copy link
Collaborator

@olgabot olgabot commented May 14, 2020

Addresses #74 to remove ribosomal reads from the trimmed sequences, so that down the line the differential hash expression doesn't even have the option of looking at ribosomal abundance, which is usually more related to sequencing depth than biological function.

PR checklist

  • PR is to dev rather than master
  • 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 necessary, also make a PR on the nf-core/kmermaid branch on the nf-core/test-datasets repo
  • Ensure the test suite passes (nextflow run . -profile test,docker).
  • Make sure your code lints (nf-core lint .).
  • Documentation in docs is updated
  • CHANGELOG.md is updated
  • README.md is updated

@olgabot
Copy link
Collaborator Author

olgabot commented May 14, 2020

Docker is currently building so all the tests are failing because sortMeRNA is not installed.

Screen Shot 2020-05-14 at 2 37 52 PM

.travis.yml Outdated Show resolved Hide resolved
@olgabot olgabot merged commit ef478e7 into dev May 19, 2020
@ewels ewels deleted the olgabot/remove-ribosomal branch October 22, 2020 07:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants