Skip to content

{bio}[GCC/10.2.0] MetaEuk v4#12188

Merged
branfosj merged 1 commit intoeasybuilders:developfrom
boegel:20210217160421_new_pr_MetaEuk4
Mar 16, 2021
Merged

{bio}[GCC/10.2.0] MetaEuk v4#12188
branfosj merged 1 commit intoeasybuilders:developfrom
boegel:20210217160421_new_pr_MetaEuk4

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Feb 17, 2021

(created using eb --new-pr)

@boegel boegel added the new label Feb 17, 2021
@boegel boegel added this to the 4.x milestone Feb 17, 2021
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Feb 17, 2021

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

@boegel: Request for testing this PR well received on generoso

PR test command 'EB_PR=12188 EB_ARGS= /apps/slurm/default/bin/sbatch --job-name test_PR_12188 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 13858

Test results coming soon (I hope)...

Details

- notification for comment with ID 780617995 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
generoso-x-3 - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/19d6c5c3a5021a9ce488aeb0e25da629 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Feb 17, 2021

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3544.doduo.os - Linux RHEL 8.2, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/a44edcf846e6481ce0dc3f46cd340efa for a full test report.

@branfosj
Copy link
Copy Markdown
Member

The MetaEuk install is hiding away a MMseqs install - https://github.com/soedinglab/metaeuk/tree/master/lib

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 15, 2021

The MetaEuk install is hiding away a MMseqs install - https://github.com/soedinglab/metaeuk/tree/master/lib

@branfosj That looks like it difficult to avoid? Is it worth pulling that out? Maybe it's customized, or the code is required for building MetaEuk?

@branfosj
Copy link
Copy Markdown
Member

The MetaEuk install is hiding away a MMseqs install - https://github.com/soedinglab/metaeuk/tree/master/lib

@branfosj That looks like it difficult to avoid? Is it worth pulling that out? Maybe it's customized, or the code is required for building MetaEuk?

On reflection I am happy to accept it. We are getting a committed copy of that code and not having it just pull down the latest, and so potentially changing, version of the MMseqs code into the repository. So, while it annoy me that they are doing it like this, we can go forward here.

@branfosj
Copy link
Copy Markdown
Member

Test report by @branfosj
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
bear-pg0211u03a.bear.cluster - Linux centos linux 7.8.2003, x86_64, Intel(R) Xeon(R) Gold 6248 CPU @ 2.50GHz (cascadelake), Python 3.6.8
See https://gist.github.com/c2031c491e2941af698921ff9b9e7ae5 for a full test report.

Copy link
Copy Markdown
Member

@branfosj branfosj left a comment

Choose a reason for hiding this comment

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

lgtm

@branfosj branfosj modified the milestones: 4.x, next release (4.3.4?) Mar 16, 2021
@branfosj
Copy link
Copy Markdown
Member

Going in, thanks @boegel!

@branfosj branfosj merged commit cbef3ee into easybuilders:develop Mar 16, 2021
@boegel boegel deleted the 20210217160421_new_pr_MetaEuk4 branch March 16, 2021 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants