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

Fix for #219 - Bad python2 file lost in the bin folder #224

Open
wants to merge 10 commits into
base: dev
Choose a base branch
from

Conversation

cimendes
Copy link
Member

@cimendes cimendes commented Sep 16, 2019

This PR fixes the issues raised in #219. I've moved the problematic file from the bin folder and into templates, updating it to python3. I've also adjusted the components "remove_host" and "retrieve_mapped" to handle this template file.


This change is Reviewable

ODiogoSilva and others added 10 commits June 18, 2019 09:28
* Dag files (#209)

* move DAG JSON files to the resources directory

* added manifest information to the `nextflow.config` file to allow for remote execution (#204) - Partial solve to #194 issue
- Deprecation of the `manifest.config´ file

* Set phred encoding when it fails to be determined - trimmomatic (#211)

* fix bug publishdir (downsample_fastq component)

* add pphred33 when encoding fails to be determined, if still fails retry with phred64 encoding (trimmomatic component)

* Fix downsample (#222)

* edited file names for downsample fastqs
* stringified depth for file name
@codecov-io
Copy link

codecov-io commented Sep 16, 2019

Codecov Report

Merging #224 into dev will decrease coverage by 1%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #224      +/-   ##
==========================================
- Coverage   42.34%   41.34%   -1.01%     
==========================================
  Files          72       73       +1     
  Lines        6529     6584      +55     
==========================================
- Hits         2765     2722      -43     
- Misses       3764     3862      +98
Impacted Files Coverage Δ
flowcraft/generator/components/metagenomics.py 100% <ø> (ø) ⬆️
flowcraft/generator/components/mapping.py 100% <ø> (ø) ⬆️
flowcraft/templates/renamePE_samtoolsFASTQ.py 0% <0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a2074ca...441daf0. Read the comment docs.

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.

4 participants