You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Weakness in current Makefile: as written will happily process the first R1 and R2 fastqs that it finds, ignoring any additional fastqs in dir that are found by the find command. This potentially results in a sample processed using only a subset of its reads, with no indication that anything is wrong.
If there are more than 2 valid fastqs, Makefile should instead crash. (Ideally, process them all; but that is not priority to implement as fab can take care of it.)
Weakness in current Makefile: as written will happily process the first R1 and R2 fastqs that it finds, ignoring any additional fastqs in dir that are found by the find command. This potentially results in a sample processed using only a subset of its reads, with no indication that anything is wrong.
If there are more than 2 valid fastqs, Makefile should instead crash. (Ideally, process them all; but that is not priority to implement as fab can take care of it.)
also minor cleanup:
https://github.com/UCSC-Treehouse/pipelines/blob/master/Makefile#L41
The text was updated successfully, but these errors were encountered: