-
Notifications
You must be signed in to change notification settings - Fork 773
{bio}[intel/2018a] CNVkit v0.9.3, Biopython v1.71, hmmlearn 0.2.0 #6445
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
{bio}[intel/2018a] CNVkit v0.9.3, Biopython v1.71, hmmlearn 0.2.0 #6445
Conversation
…on-1.71-intel-2018a-Python-3.6.4.eb, Pysam-0.14.1-intel-2018a-Python-3.6.4.eb, hmmlearn-0.2.0-intel-2018a-Python-3.6.4.eb
|
Test report by @boegel |
|
Test report by @boegel |
|
Test report by @boegel |
damianam
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, but as I asked in another PR, did you verify that no other dependencies are downloaded by pip?
|
@damianam That's verified automagically by setting |
|
Test report by @migueldiascosta |
|
missing |
|
Test report by @migueldiascosta |
|
@migueldiascosta Failure with Not sure about the issue with |
|
@boegel regarding the issue with |
|
@migueldiascosta Can you share a full debug log for the failing Also, was |
|
@migueldiascosta May be related to the fix in #5945, was |
|
added the |
|
@boegel Conflicts to be resolved... |
|
I get And so is the tests. |
|
@boegel: Tests failed in GitHub Actions, see https://github.com/easybuilders/easybuild-easyconfigs/actions/runs/367715320 bleep, bloop, I'm just a bot (boegelbot v20200716.01) |
|
Test report by @migueldiascosta |
|
@boegel I think my old failing tests can be ignored, I don't seem to be able to reproduce them even in that old CentOS 6 cluster |
…asyconfigs + enhance sanity check for Biopython
|
Test report by @boegel |
|
Test report by @boegel |
akesandgren
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Test report by @akesandgren |
|
Going in, thanks @boegel! |
(created using
eb --new-pr)