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

Mapping with bowtie2 does not work #592

Closed
teepean opened this issue Oct 27, 2020 · 10 comments · Fixed by #594
Closed

Mapping with bowtie2 does not work #592

teepean opened this issue Oct 27, 2020 · 10 comments · Fixed by #594
Labels
bug Something isn't working pending Addressed on branch waiting for related PR
Milestone

Comments

@teepean
Copy link

teepean commented Oct 27, 2020

Hello!

I decided to test mapping with bowtie2 but ran into an error:

[-        ] process > makeFastaIndex -
[-        ] process > makeSeqDict    -
No such variable: bwa_index_bwamem

 -- Check script '.nextflow/assets/nf-core/eager/main.nf' at line: 1518 or see '.nextflow.log' file for more details

Running on Ubuntu 20.04 under WSL2. Both bwa aln and bwa mem work without issues.

Eager revision: f2a326d [dev]
Nextflow version 20.07.1 build 5412
Docker version 19.03.8, build afacb8b7f0

nextflow.log

@jfy133 jfy133 added the bug Something isn't working label Oct 27, 2020
@jfy133
Copy link
Member

jfy133 commented Oct 27, 2020

Thanks @teepean - I can replicate so will investigate!

@jfy133
Copy link
Member

jfy133 commented Oct 27, 2020

Note to self: this only occurs when the --bt2_index is supplied. Running bowtie2 without a pre-made index works without an issue.

@jfy133
Copy link
Member

jfy133 commented Oct 27, 2020

Issue is lines 834-891, where bwa_index_mem is only set to Channel.empty when BT2 indexing is requested.

@jfy133 jfy133 added the pending Addressed on branch waiting for related PR label Oct 27, 2020
@jfy133
Copy link
Member

jfy133 commented Oct 27, 2020

@teepean issue should be solved in an upcoming patch of 2.2! Will close this once that's out.

jfy133 added a commit that referenced this issue Oct 27, 2020
@teepean
Copy link
Author

teepean commented Oct 27, 2020

@teepean issue should be solved in an upcoming patch of 2.2! Will close this once that's out.

Thanks!

@jfy133 jfy133 added this to the 2.2.1 milestone Oct 27, 2020
@jfy133 jfy133 mentioned this issue Oct 27, 2020
8 tasks
@jfy133
Copy link
Member

jfy133 commented Oct 29, 2020

@teepean should be in -r 2.2.1!

@teepean
Copy link
Author

teepean commented Oct 30, 2020

@teepean should be in -r 2.2.1!

It is working. Thank you!

@martinacaruso98
Copy link

jordwil/nextflow-bowtie2#1 (comment)

Is there anyone that can help?

@jfy133
Copy link
Member

jfy133 commented Mar 16, 2023

Hi @martinacaruso98 I'm sorry but you appear to be referring to another pipeline (based on the link you posted), we can't help with that.

@martinacaruso98
Copy link

Hi @martinacaruso98 I'm sorry but you appear to be referring to another pipeline (based on the link you posted), we can't help with that.

Oh, sorry! Thank you anyway!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pending Addressed on branch waiting for related PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants