-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update samtools #72
Comments
@niemasd Is there a particular reason it would be good to update? I am inclined to stick with the existing version until the regression tests are in place and accepted ... |
No particular reason, and I agree that all tool versions should generally remain as-is until regression tests and other relevant tests are in place and accepted. Once everything is 100% functional and ready for production, then I think it may be worth exploring updating samtools for future versions of the pipeline I just wanted to make this GitHub Issue while it was on my mind |
Updated comment:
A new version of samtools was just released, and I see that one of the features is that samtools depth was rewritten to be faster and more memory efficient. That's not a bottleneck of our pipeline, but it would still be good to update at some point (though not urgent):
https://github.com/samtools/samtools/releases/tag/1.13
Original comment:
The pipeline currently uses samtools 1.11, but samtools 1.12 was recently released:
https://github.com/samtools/samtools/releases/tag/1.12
Not urgent at all, but it would be good to update to samtools 1.12 at some point in the future
The text was updated successfully, but these errors were encountered: