X-Git-Url: https://git.donarmstrong.com/?p=biopieces.git;a=blobdiff_plain;f=bp_bin%2Fwrite_fastq;h=d15c96f9dd31eb0ac6792b6b70d092ddcdd29bb6;hp=30bdd89cdaca441f1ab15d9d21a329e53abdd16f;hb=2f0fd91b461033529a4a72e161bd133252a22eb6;hpb=b1fadc0ce9224b6a13f8deae332bb902a54c65af

diff --git a/bp_bin/write_fastq b/bp_bin/write_fastq
index 30bdd89..d15c96f 100755
--- a/bp_bin/write_fastq
+++ b/bp_bin/write_fastq
@@ -34,38 +34,33 @@ require 'maasha/fastq'
 allowed_enc = 'base_33,base_64'
 
 casts = []
-casts << {:long=>'no_stream', :short=>'x', :type=>'flag',   :mandatory=>false, :default=>nil,       :allowed=>nil,         :disallowed=>nil}
-casts << {:long=>'data_out',  :short=>'o', :type=>'file',   :mandatory=>false, :default=>nil,       :allowed=>nil,         :disallowed=>nil}
-casts << {:long=>'encoding',  :short=>'e', :type=>'string', :mandatory=>false, :default=>'base_33', :allowed=>allowed_enc, :disallowed=>nil}
-casts << {:long=>'compress',  :short=>'Z', :type=>'flag',   :mandatory=>false, :default=>nil,       :allowed=>nil,         :disallowed=>nil}
+casts << {long: 'no_stream', short: 'x', type: 'flag',   mandatory: false, default: nil,       allowed: nil,               disallowed: nil}
+casts << {long: 'data_out',  short: 'o', type: 'file',   mandatory: false, default: nil,       allowed: nil,               disallowed: nil}
+casts << {long: 'encoding',  short: 'e', type: 'string', mandatory: false, default: 'base_33', allowed: allowed_enc,       disallowed: nil}
+casts << {long: 'compress',  short: 'Z', type: 'string', mandatory: false, default: nil,       allowed: "gzip,bzip,bzip2", disallowed: nil}
 
 options = Biopieces.options_parse(ARGV, casts)
 
 encoding = options[:encoding].to_sym
+compress = options[:compress] ? options[:compress].to_sym : nil
+
+raise "--data_out is mandatory for compressed output" if compress and not options[:data_out]
 
 Biopieces.open(options[:stream_in], options[:stream_out]) do |input, output|
-  if options[:data_out]
-    if options[:compress]
-      io_out = Zlib::GzipWriter.open(options[:data_out])
-    else
-      io_out = Fastq.open(options[:data_out], 'w')
-    end
-  else
-    io_out = $stdout
-  end
+  fastq_out = options[:data_out] ? Fastq.open(options[:data_out], 'w', compress: compress) : STDOUT
 
   input.each do |record|
     if record[:SEQ_NAME] and record[:SEQ] and record[:SCORES]
       entry = Seq.new_bp(record)
       entry.qual_convert!(:base_33, encoding)
 
-      io_out.puts entry.to_fastq
+      fastq_out.puts entry.to_fastq
     end
 
     output.puts record unless options[:no_stream]
   end
 
-  io_out.close
+  fastq_out.close
 end