Skip to content

patch oxDNA/oxpy to use correct pip install prefix#18214

Merged
smoors merged 1 commit intoeasybuilders:developfrom
migueldiascosta:20230628184619_new_pr_oxDNA352
Jun 29, 2023
Merged

patch oxDNA/oxpy to use correct pip install prefix#18214
smoors merged 1 commit intoeasybuilders:developfrom
migueldiascosta:20230628184619_new_pr_oxDNA352

Conversation

@migueldiascosta
Copy link
Copy Markdown
Member

@migueldiascosta migueldiascosta commented Jun 28, 2023

(created using eb --new-pr)

fixes #18211

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Jun 28, 2023

Test report by @smoors
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node250.hydra.os - Linux CentOS Linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2650 v4 @ 2.20GHz (broadwell), 1 x NVIDIA Tesla P100-PCIE-16GB, 515.48.07, Python 3.6.8
See https://gist.github.com/smoors/640ab6dc128a7c92a96151cc83a73b33 for a full test report.

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Jun 29, 2023

@boegelbot: please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

@smoors: Request for testing this PR well received on login1

PR test command 'EB_PR=18214 EB_ARGS= EB_CONTAINER= /opt/software/slurm/bin/sbatch --job-name test_PR_18214 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1612484960 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)
cns1 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/boegelbot/60c73f222e25eb2f38901b0f12cda6c4 for a full test report.

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Jun 29, 2023

@boegelbot please test @ jsc-zen2

@boegelbot
Copy link
Copy Markdown
Collaborator

@smoors: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster

PR test command 'EB_PR=18214 EB_ARGS= /opt/software/slurm/bin/sbatch --mem-per-cpu=4000M --job-name test_PR_18214 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen2.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1612521525 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)
jsczen2c1.int.jsc-zen2.easybuild-test.cluster - Linux Rocky Linux 8.5, x86_64, AMD EPYC 7742 64-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/boegelbot/7189044e437e3176206392ce63f1225f for a full test report.

Copy link
Copy Markdown
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.

confirmed that this fixes the issue: no python packages are added to the Python installation or to the user account

lgtm

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Jun 29, 2023

Going in, thanks @migueldiascosta!

@smoors smoors merged commit 9b20eae into easybuilders:develop Jun 29, 2023
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.

oxDNA installs extra python packages in Python installdir

3 participants