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

--force_sratools_download results in different fastq names compared to FTP download #119

Closed
ojziff opened this issue Oct 20, 2022 · 1 comment
Labels
bug Something isn't working
Milestone

Comments

@ojziff
Copy link

ojziff commented Oct 20, 2022

Description of the bug

Hey,

is it intended that --force_sratools_download results in different fastq names?

Without --force_sratools_download the fastqs have naming format SRX[0-9]+SRR[0-9][1-2].fastq.gz whereas with --force_sratools_download the naming is SRR[0-9]+_[1-2].fastq.gz.

This means that you cannot combine downloading fastqs both with and without --force_sratools_download in the event of ebi curl errors (without) e.g.

Command error:
  INFO:    Converting SIF file to temporary sandbox...
    % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                   Dload  Upload   Total   Spent    Left  Speed
  
    0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
    0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
    0     0    0     0    0     0      0      0 --:--:--  0:00:01 --:--:--     0
  curl: (7) Failed to connect to ftp.sra.ebi.ac.uk port 21 after 1614 ms: Connection refused

and fasterq-dump errors (with) e.g.

Command error:
  INFO:    Converting SIF file to temporary sandbox...
  .command.run: fork: Cannot allocate memory
  fasterq-dump was killed (signal 9 SIGKILL)
  .command.run: line 163: kill: (80666) - No such process
  INFO:    Cleaning up image...

Many thanks,
Oliver

@ojziff ojziff added the bug Something isn't working label Oct 20, 2022
@drpatelh drpatelh added this to the 1.8 milestone Nov 4, 2022
@drpatelh drpatelh changed the title --force_sratools_download results in different fastq names --force_sratools_download results in different fastq names compared to FTP download Nov 7, 2022
drpatelh added a commit to drpatelh/nf-core-fetchngs that referenced this issue Nov 7, 2022
@drpatelh drpatelh mentioned this issue Nov 7, 2022
@drpatelh
Copy link
Member

drpatelh commented Nov 7, 2022

Fixed in #125

@drpatelh drpatelh closed this as completed Nov 7, 2022
drpatelh added a commit that referenced this issue Nov 7, 2022
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