-
Notifications
You must be signed in to change notification settings - Fork 24
-
Notifications
You must be signed in to change notification settings - Fork 24
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
Strange error when QC some samples #91
Comments
Thanks for the excellent report, @AlessioMilanese |
I managed to reproduce it locally with v0.9.1, but it works correctly on master. Still looking into it, but might just be worthwhile to do a new release. |
Could this possibly be a permission error with an erroneous error message? Does your user have permissions to create the output files inside the output directory? |
I have writing permission. I can reproduce the error also with just one lane:
and also if I rename the samples to:
The error says
but when I check
It seems indeed a problem when moving the files to the final destination. |
If I change the name of the directory from
it seems to work. Can you reproduce it? |
Okay, I figured this out. It's a mix of (1) what I think is a bug in your code (2) NGLess does not correctly check if the output directory exists and (3) outputs a horrible error message, (4) partially because of another bug in the haskell posix library (haskell/unix#60) You were doing:
You load the data from
Just If this directory does not exist, you get the stupid behaviour you saw. I am re-classifying this bug as "better-error-message" |
See #91 for an example where this is relevant.
Many small fixes rather than any large new features. Full ChangeLog: * Fix to lock1's return value when used with paths (#68 - reopen) * Support _F/_R suffixes for forward/reverse in load_mocat_sample * samtools_sort() now accepts by={name} to sort by read name * Fixed bug where header was printed even when STDOUT was used * Fixed bug where writing interleaved FastQ to STDOUT did not work as expected * Indices created by bwa and minimap2 are now versioned * arg1 in external modules is no longer always treated as a path * Added expand_searchpath to external modules API (closes #56) * Fixed bug where detection of Fastq encoding was not performed on the second pair * Fix saving fastq sets with --subsample (issue #85) * Add __extra_megahit_args to assemble() (issue #86) * Better error message when user mis-specifies the ngless version string (issue #84) * Support NO_COLOR environment variable (issue #83) * Garbage collection for temporary files (issue #79) * Rename --search-dir to --search-path for consistency with other API * Fix corner case with select() producing incorrect CIGAR strings (#92) * Always check output file writability (#91) * Make paired() accept encoding argument
Hi,
I'm trying to filter reads based on quality and remove human reads, but I have the following error:
Some more information:
I installed ngless from bioconda (
ngless v0.9.1 (release date: July 17 2018)
)I'm running this script (
/scratch/milanese/TEST/QC/filter-human-merged.ngl
):submitting it to slurm with the script (
/scratch/milanese/TEST/QC/slurm_ex2.sh
):The file
/scratch/milanese/TEST/QC/list2
contains:The directory
/g/scb2/zeller/SHARED/DATA/metaG/REAL/CRC-META/FR-CRC_N156/CCIS12370844ST-4-0
contains:The output that I get using
--trace
is:The text was updated successfully, but these errors were encountered: