[ARVADOS] updated: f2bed1a9d44ccee1a5e1214e139d586b60b90e6a

git at public.curoverse.com git at public.curoverse.com
Thu Sep 11 11:19:53 EDT 2014


Summary of changes:
 crunch_scripts/split-fastq.py | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

       via  f2bed1a9d44ccee1a5e1214e139d586b60b90e6a (commit)
      from  741bf44b15569a3a4b1a4705b62e4f50c8365bc9 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit f2bed1a9d44ccee1a5e1214e139d586b60b90e6a
Author: Brett Smith <brett at curoverse.com>
Date:   Thu Sep 11 11:21:36 2014 -0400

    Clean up manifest whitespace in split-fastq.
    
    The previous code was generating two spaces between the stream name
    and block list, because it had the space from as_manifest() as well as
    its own join.  This yielded an invalid manifest.
    
    No issue #.

diff --git a/crunch_scripts/split-fastq.py b/crunch_scripts/split-fastq.py
index 253dd22..17aabf2 100755
--- a/crunch_scripts/split-fastq.py
+++ b/crunch_scripts/split-fastq.py
@@ -119,10 +119,11 @@ for s in inp.all_streams():
                     splitfastq(p)
                 else:
                     for i in xrange(0, len(p)):
-                        m = p[i]["reader"].as_manifest()[1:]
-                        manifest_list.append(["./_" + str(piece), m[:-1]])
+                        m = p[i]["reader"].as_manifest().split()
+                        m[0] = "./_" + str(piece)
+                        manifest_list.append(m)
                     piece += 1
 
-manifest_text = "\n".join(" ".join(m) for m in manifest_list)
+manifest_text = "\n".join(" ".join(m) for m in manifest_list) + "\n"
 
 arvados.current_task().set_output(manifest_text)

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list