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

Remove trailing slash from params.igenomes_base #645

Closed
pansapiens opened this issue May 27, 2021 · 2 comments
Closed

Remove trailing slash from params.igenomes_base #645

pansapiens opened this issue May 27, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@pansapiens
Copy link

Check Documentation

I have checked the following places for your error:

Description of the bug

S3 URL to iGenomes reference and annotations listed in pipeline_info/pipeline_report.html and pipeline_info/pipeline_report.txt are invalid. They contain an additional slash which makes them incompatible with some tools used to download files from S3.

Steps to reproduce

Run the pipeline with a flag like: --genome GRCm38.
Observe the output in pipeline_info/pipeline_report.html.

Expected behaviour

URLs in pipeline_info/pipeline_report.html and pipeline_info/pipeline_report.txt should be like:

s3://ngi-igenomes/igenomes/Mus_musculus/Ensembl/GRCm38/Sequence/WholeGenomeFasta/genome.fa

but instead contain an additional slash:

s3://ngi-igenomes/igenomes//Mus_musculus/Ensembl/GRCm38/Sequence/WholeGenomeFasta/genome.fa (note the extra slash between igenomes//Mus_musculus).

These URLs fail when used directly with the aws s3 commandline tool (v1.18.121) unless the additional slash is removed.

Nextflow Installation

Nextflow version: 21.04.0
Nextflow build: 5552
Pipeline version (commit on master): 0fcbb0a

Additional context

This is likely a result of the default value of igenomes_base here having a trailing slash, so that when it is used here a double slash is introduced. I've not yet tested if removing the trailing slash from igenomes_base fixes the issue without breaking other parts of the pipeline.

@pansapiens pansapiens added the bug Something isn't working label May 27, 2021
@drpatelh
Copy link
Member

drpatelh commented Jun 1, 2021

Thanks @pansapiens ! Yes, was fixed here in the pipeline template and needs to be fixed in this pipeline too. It should have come through via the automated sync mechanism we have but probably something I missed.

@drpatelh drpatelh changed the title S3 URLs to iGenomes reference files in pipeline_info/pipeline_report.* are invalid in some contexts due to a double slash Remove trailing slash from params.igenomes_base Jun 1, 2021
drpatelh added a commit to drpatelh/nf-core-rnaseq that referenced this issue Jun 1, 2021
@drpatelh
Copy link
Member

drpatelh commented Jun 1, 2021

Fixed in #647

@drpatelh drpatelh closed this as completed Jun 1, 2021
drpatelh added a commit that referenced this issue Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants