Skip to content

{bio}[intel/2020a] Metagenome-Atlas v2.4.3 w/ Python 3.8.2#11620

Merged
boegel merged 7 commits intoeasybuilders:developfrom
Darkless012:20201105114545_new_pr_Metagenome-Atlas243
Dec 7, 2020
Merged

{bio}[intel/2020a] Metagenome-Atlas v2.4.3 w/ Python 3.8.2#11620
boegel merged 7 commits intoeasybuilders:developfrom
Darkless012:20201105114545_new_pr_Metagenome-Atlas243

Conversation

@Darkless012
Copy link
Copy Markdown
Contributor

@Darkless012 Darkless012 commented Nov 5, 2020

(created using eb --new-pr)

requires easyblock:

requires easyconfigs:

…b and patches: Metagenome-Atlas-2.4.3_remove-hardcoded-conda.patch
@boegel boegel added the new label Nov 11, 2020
@boegel boegel added this to the 4.x milestone Nov 11, 2020
@easybuilders easybuilders deleted a comment from boegelbot Nov 11, 2020
@easybuilders easybuilders deleted a comment from boegelbot Nov 16, 2020
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

@Darkless012 Please include sanity_pip_check in the easyconfig, which will raise a couple of missing extensions:

== 2020-11-24 09:20:17,121 easyblock.py:3335 WARNING build failed (first 300 chars): cmd "pip check" exited with exit code 1 and output:
ruamel-yaml 0.16.12 requires ruamel-yaml-clib, which is not installed.
plotly 4.11.0 requires retrying, which is not installed.
cookiecutter 1.7.2 has requirement poyo>=0.5.0, but you have poyo 0.2.0.

Also include a comment + mention author on top of the patch.

And close easybuilders/easybuild-easyblocks#2219, since it's no longer needed now PythonBundle is used?

@easybuilders easybuilders deleted a comment from boegelbot Nov 26, 2020
@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 7, 2020

@boegelbot please test @ generoso
EB_ARGS="--include-easyblocks-from-pr 2219"

@boegelbot
Copy link
Copy Markdown
Collaborator

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

PR test command 'EB_PR=11620 EB_ARGS="--include-easyblocks-from-pr 2219" /apps/slurm/default/bin/sbatch --job-name test_PR_11620 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 739776819 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
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2219
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
generoso-c1-s-1 - 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/dc295c6e9c98eff0905e297a1ecaa7b8 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 7, 2020

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2219
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3145.skitty.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 3.6.8
See https://gist.github.com/59094af491305322492fd6d519d8fb37 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 7, 2020

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2219
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node2728.swalot.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/0428e0b40f479740e6ac4724757bedca for a full test report.

@boegel boegel modified the milestones: 4.x, 4.3.2 (next release) Dec 7, 2020
@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 7, 2020

Going in, thanks @Darkless012!

@boegel boegel merged commit d2f834f into easybuilders:develop Dec 7, 2020
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