Skip to content
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

run_indep_wkflow_with_gnu_par hangs after parallel build step before combine #12

Open
rmcolq opened this issue Feb 1, 2016 · 7 comments

Comments

@rmcolq
Copy link

rmcolq commented Feb 1, 2016

So I think the problem is that the list_all_raw_vcfs file ends up empty. Because it exists, no error is thrown by combine_vcfs.pl and instead it hangs.
I believe that the file list_all_raw_vcfs file is being created in the combine_vcfs.pl script as part of a subroutine 'check_list'.
So need to add something to make it work (why was it empty when there should have been _/vcfs/wk_raw created) or break properly if the file created is empty?

@iqbal-lab
Copy link
Owner

Good spot on possible reason

@iqbal-lab
Copy link
Owner

The whole thing is horribly non-robust. The right thing to do is for run_indep.. to check the results of the previous step before carrying on. And yes, it should stop if that list is empty. Usually that might be ecause of vcftools not being in the path or some nonsense. But you should be able to look into the per-sample VCFs and see if there are any variants in them here. Maybe we need a tracking/ directory, and each parallel process writes a file to that dir on completion, saying whether it has completed correctly in its opinion. Then the main script can look in that dir and check - do all the completion files exist, and do they all contain OK content.

@iqbal-lab
Copy link
Owner

If you are concerned about being able to do this, transfer ownership of this issue to me, and I can do it tomorrow.

@rmcolq
Copy link
Author

rmcolq commented Feb 1, 2016

Yes please

@hcdenbakker
Copy link
Contributor

Hi Guys,
Remember I have been able to solve this by altering the code around line 75/76 (where parallel is implemented I believe) , look at my fork. Works like a charm for Ubuntu Linux

-Henk

@iqbal-lab
Copy link
Owner

THANKS and sorry Henk. Will look,

@hcdenbakker
Copy link
Contributor

You're welcome! Happy to contribute. Have been making the change in the script every time a new version came out. Tell me if you need more info.

Sent from my iPhone

On Feb 1, 2016, at 10:08 AM, Zamin Iqbal [email protected] wrote:

THANKS and sorry Henk. Will look,


Reply to this email directly or view it on GitHub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants