-
Notifications
You must be signed in to change notification settings - Fork 8
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
CRAM2VCF.cpp needs a lot of memory #23
Comments
Hi @TorHou
Yes, you're referring to lines: https://github.com/NCBI-Hackathons/NovoGraph/blob/master/scripts/launch_CRAM2VCF_C%2B%2B.pl#L102-L116 There's certainly no reason why this couldn't be a parameter in the script.
This isn't too surprising, if I recall. I agree though---the question to my mind is how easily things could be improved in terms of , and if it's worth the investment. (Not the most helpful comment I know, but I'll have to take a closer look at the *cpp code). |
When running
launch_CRAM2VCF_C++.pl
I have noticed that running 10 processes in parallelwhile(scalar(keys %still_running) >= 10)
will use up a lot of mermory. Getting a job killed with 1024 Gbyte of RAM.So I had to change it to
while(scalar(keys %still_running) >= 2)
in code.The text was updated successfully, but these errors were encountered: