Skip to content

switch to using python-parasail and tqdm dependencies in Bonito easyconfigs#11937

Merged
boegel merged 1 commit intoeasybuilders:developfrom
branfosj:20201228180741_new_pr_Bonito010
Dec 29, 2020
Merged

switch to using python-parasail and tqdm dependencies in Bonito easyconfigs#11937
boegel merged 1 commit intoeasybuilders:developfrom
branfosj:20201228180741_new_pr_Bonito010

Conversation

@branfosj
Copy link
Copy Markdown
Member

@branfosj branfosj commented Dec 28, 2020

(created using eb --new-pr)

While testing #11750 I hit an error connected with the installation of parasail (the python library) failing to install parasail - see #11750 (comment)

This:

  • Switches parasail-2.4.1-foss-2019b.eb to parasail-2.4.1-GCC-8.3.0.eb
  • Adds python-parasail-1.2-fosscuda-2019b-Python-3.7.4.eb
  • Switches the Bonito ecs to use the python-parasail and tqdm

To be able to use the tqdm dep, I had to add the 'preinstallopts': "sed -i 's/==/>=/g' requirements.txt && ", to the Bonito 0.1.0. This is already in the newer ecs.

For parasail, we already install the newer version at the iccifort level (parasail-2.4.2-iccifort-2020.1.217.eb) and I cannot see anything in the install or documentation that suggests that it requires to be at foss/fosscuda instead of dropping to GCC.

@branfosj branfosj added this to the next release (4.3.3?) milestone Dec 28, 2020
@branfosj branfosj changed the title Bonito - switch to using separate python-parasail and tqdm Bonito - switch to using python-parasail and tqdm dependencies Dec 28, 2020
@branfosj
Copy link
Copy Markdown
Member Author

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

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

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

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 751841110 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 7 out of 7 (7 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/33569ee3de613dee1f4f03895f95e062 for a full test report.

@branfosj
Copy link
Copy Markdown
Member Author

Test report by @branfosj
SUCCESS
Build succeeded for 7 out of 7 (7 easyconfigs in total)
bear-pg0212u15b.bear.cluster - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2640 v4 @ 2.40GHz (broadwell), Python 3.6.8
See https://gist.github.com/932141f81fedde6aae457cc0d2d4ef1a for a full test report.

@boegel boegel changed the title Bonito - switch to using python-parasail and tqdm dependencies switch to using python-parasail and tqdm dependencies in Bonito easyconfigs Dec 29, 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.

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 29, 2020

Test report by @boegel
SUCCESS
Build succeeded for 7 out of 7 (7 easyconfigs in total)
node3300.joltik.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6242 CPU @ 2.80GHz (cascadelake), Python 3.6.8
See https://gist.github.com/6a6b0392592871b05f7fd625527ac7fd for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 29, 2020

Going in, thanks @branfosj!

@boegel boegel merged commit 0aa943a into easybuilders:develop Dec 29, 2020
@branfosj branfosj deleted the 20201228180741_new_pr_Bonito010 branch December 29, 2020 10:15
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.

3 participants