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

Different behavior for "-d" and "--dir" #141

Open
rzelle-lallemand opened this issue Oct 17, 2024 · 1 comment
Open

Different behavior for "-d" and "--dir" #141

rzelle-lallemand opened this issue Oct 17, 2024 · 1 comment

Comments

@rzelle-lallemand
Copy link

With FastQC v0.12.1 (bioconda hdfd78af_0) on RHEL 9.4, we're seeing a discrepancy when using the "-d" and "--dir" flags to specify a non-standard directory for temporary files

$ fastqc -d "/test/temp" file.fastq.gz 
Option d is ambiguous (delete, dir, dup_length)
null
Failed to process /test/temp
java.io.FileNotFoundException: /test/temp (Is a directory)
	at java.base/java.io.FileInputStream.open0(Native Method)
	at java.base/java.io.FileInputStream.open(FileInputStream.java:213)
	at java.base/java.io.FileInputStream.<init>(FileInputStream.java:152)
	at uk.ac.babraham.FastQC.Sequence.FastQFile.<init>(FastQFile.java:77)
	at uk.ac.babraham.FastQC.Sequence.SequenceFactory.getSequenceFile(SequenceFactory.java:106)
	at uk.ac.babraham.FastQC.Sequence.SequenceFactory.getSequenceFile(SequenceFactory.java:62)
	at uk.ac.babraham.FastQC.Analysis.OfflineRunner.processFile(OfflineRunner.java:163)
	at uk.ac.babraham.FastQC.Analysis.OfflineRunner.<init>(OfflineRunner.java:125)
	at uk.ac.babraham.FastQC.FastQCApplication.main(FastQCApplication.java:316)
application/gzip
Started analysis of file.fastq.gz
...

and

$ fastqc --dir "/test/temp" file.fastq.gz 
application/gzip
Started analysis of file.fastq.gz
...

$ fastqc -h meanwhile reports:

-d --dir Selects a directory to be used for temporary files written when
generating report images. Defaults to system temp directory if
not specified.

Also discussed here: sunbeam-labs/sunbeam#424 (comment) and bcbio/bcbio-nextgen#3702 (comment)

@s-andrews
Copy link
Owner

Yeah, that's just a flat out bug. I'll either need to explicitly assign -d to the directory option, or just remove it from the docs. The --dir will work and should be the option to use until this is fixed.

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

No branches or pull requests

2 participants