-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rnaseq_cgl_pipeline.py check that unzip is installed #203
Comments
This pipeline does not use unzip or mapsplice. |
Well, it crashes with that error. |
Can you post a stacktrace? |
Yeah of course, I'll rerun it. Had to terminate the cluster last night because it turns out you can't stop a spot cluster... |
Can't reproduce the error! Log file here |
Can you post the contents of the launch script? |
I do realize I should have probably removed the output_dir. |
Any chance if you know if the output showed up in the output dir? |
I did not create a /home/mesosbox/rnaseq_output and there is none on the master after the run. But also no error about that. |
It would've been on the worker. I'm running this pipeline (and the UNC one) as part of a test, so I'll see if i can replicate. |
okay I can have a look there |
Nope, not on either worker. |
Ok, thanks for checking. |
Could not reproduce. My run just finished and my file showed up in S3. s3://cgl-driver-projects/test/test_SRR1559177_Sample_NB2318tumor1_T_C3WFVACXX.tar.gz
|
I see unzip mentioned here. What's the relationship between this issue and BD2KGenomics/cgcloud#134? Is the latter invalid? What triggered Jeltje to believe that a missing unzip was the culprit? |
Unzip is needed for the UNC rna-seq pipeline, not this one.
An error that she wasn't able to reproduce. |
It appears that the code does not check for the unzip program, which leads to a fairly incomprehensible
No such file or directory
error from mapspliceThe text was updated successfully, but these errors were encountered: