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

add missing PyVCF dependency for BAMSurgeon #10564

Merged

Conversation

boegel
Copy link
Member

@boegel boegel commented May 4, 2020

(created using eb --new-pr)

(see also adamewing/bamsurgeon#158)

@boegel
Copy link
Member Author

boegel commented May 4, 2020

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node3401.kirlia.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) Gold 6240 CPU @ 2.60GHz (cascadelake), Python 2.7.5
See https://gist.github.com/511bd2ced5a9975cbb81a9c0873d09fb for a full test report.

@boegel
Copy link
Member Author

boegel commented May 4, 2020

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node3153.skitty.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/51288767d8c54461adaf4013b78725ea for a full test report.

@boegel
Copy link
Member Author

boegel commented May 4, 2020

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
generoso-1 - Linux centos linux 7.6.1810, x86_64, Intel(R) Xeon(R) CPU E5-2660 v4 @ 2.00GHz, Python 3.6.8
See https://gist.github.com/4b676541ffe37a456e6b6eb9bcdad5b5 for a full test report.

@boegel
Copy link
Member Author

boegel commented May 4, 2020

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2617.swalot.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 2.7.5
See https://gist.github.com/ae9e67987a031b8dbcc92c69cd7f5505 for a full test report.

@smoors
Copy link
Contributor

smoors commented May 19, 2020

Test report by @smoors
SUCCESS
Build succeeded for 5 out of 5 (2 easyconfigs in this PR)
node318.hydra.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz (skylake_avx512), Python 2.7.5
See https://gist.github.com/65ebf6bef0af20e6056536e6fbfb6722 for a full test report.

Copy link
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@smoors
Copy link
Contributor

smoors commented May 19, 2020

Going in, thanks @boegel!

@smoors smoors merged commit 19f916b into easybuilders:develop May 19, 2020
@boegel boegel deleted the 20200504085632_new_pr_BAMSurgeon12 branch May 19, 2020 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants