ibest / ARC

Assembly by Reduced Complexity (ARC)
Apache License 2.0
41 stars 5 forks source link

ARC doesn't properly detect missing PE files #50

Closed samhunter closed 8 years ago

samhunter commented 9 years ago

In config.py, line 203 (if not (pe or se)) there is a small logic error which can cause ARC to fail to identify incorrectly formatted config files which contain a SE and one of two PE files. This also causes a weird edge case when the config file is formatted with comment characters in front of the column headers e.g.

Sample_ID FileName FileType

Sample1 ./reads/Sample1_R1.fasta PE1

instead of: Sample_ID FileName FileType Sample1 ./reads/Sample1_R1.fasta PE1

Fix this logic.

samhunter commented 8 years ago

This should be detected and raise an exception now.