Skip to content

Commit

Permalink
Rolling back to slower but stable version of bam_to_fastq
Browse files Browse the repository at this point in the history
  • Loading branch information
Charles Dietz committed Aug 25, 2016
1 parent 46185a6 commit caad85e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pypiper/ngstk.py
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ def markDuplicates(self, aligned_file, out_file, metrics_file, remove_duplicates
cmd += " REMOVE_DUPLICATES=" + remove_duplicates
return cmd

def bam_to_fastq2(self, bam_file, out_fastq_pre, paired_end):
def bam_to_fastq(self, bam_file, out_fastq_pre, paired_end):
self.make_sure_path_exists(os.path.dirname(out_fastq_pre))
cmd = self.tools.java + " -Xmx" + self.pm.javamem
cmd += " -jar " + self.tools.picard + " SamToFastq"
Expand All @@ -111,7 +111,7 @@ def bam_to_fastq2(self, bam_file, out_fastq_pre, paired_end):
cmd += " VALIDATION_STRINGENCY=SILENT"
return cmd

def bam_to_fastq(self, bam_file, out_fastq_pre, paired_end):
def bam_to_fastq2(self, bam_file, out_fastq_pre, paired_end):
self.make_sure_path_exists(os.path.dirname(out_fastq_pre))

if paired_end:
Expand Down

1 comment on commit caad85e

@nsheff
Copy link
Member

@nsheff nsheff commented on caad85e Dec 14, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for future reference, this was just caused by a missing "+" sign, which was fixed in dev.

Please sign in to comment.