Skip to content

{bio}[foss-2020b,fosscuda-2020b] OpenMM v7.5.1#13452

Merged
boegel merged 2 commits intoeasybuilders:developfrom
sassy-crick:20210717210532_new_pr_OpenMM751
Aug 4, 2021
Merged

{bio}[foss-2020b,fosscuda-2020b] OpenMM v7.5.1#13452
boegel merged 2 commits intoeasybuilders:developfrom
sassy-crick:20210717210532_new_pr_OpenMM751

Conversation

@sassy-crick
Copy link
Copy Markdown
Collaborator

(created using eb --new-pr)

@sassy-crick sassy-crick changed the title OpenMM update to 7.5.1 required for alphafold {bio}[Foss-2020b]/[Fosscuda-2020b]OpenMM update to 7.5.1 required for alphafold Jul 17, 2021
@sassy-crick
Copy link
Copy Markdown
Collaborator Author

Do we need a Python version as well? Please let me know and I will do that too.

Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

No need to duplicate the patch, just refer to the 7.5.0 version

@Micket Micket added the update label Jul 27, 2021
@Micket Micket added this to the next release (4.4.2?) milestone Jul 27, 2021
@Micket
Copy link
Copy Markdown
Contributor

Micket commented Jul 27, 2021

Do we need a Python version as well? Please let me know and I will do that too.

I don't understand what you mean

@sassy-crick
Copy link
Copy Markdown
Collaborator Author

No need to duplicate the patch, just refer to the 7.5.0 version
I agree. I was not sure if that is the accepted solution for the patch:
patches = ['%(name)s-7.5.0_fix_path_to_nvcc.patch']
as I basically hard-code the patch in there. If you are happy with this, I shall be too.

Python: I meant similar to this EC:
OpenMM-7.5.0-fosscuda-2020a-Python-3.8.2.eb
With all the recent discussions about how to handle CUDA and Python, I am a bit confused.

@Micket Micket changed the title {bio}[Foss-2020b]/[Fosscuda-2020b]OpenMM update to 7.5.1 required for alphafold {bio}[foss-2020b]/[fosscuda-2020b] OpenMM v7.5.1 required for alphafold Jul 30, 2021
@sassy-crick
Copy link
Copy Markdown
Collaborator Author

No need to duplicate the patch, just refer to the 7.5.0 version

I have set the version of the patch fix to 7.5.0 in the EC file. Let me know if there is a better way of doing that.

Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

LGTM

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Aug 3, 2021

Test report by @Micket
FAILED
Build succeeded for 2 out of 3 (2 easyconfigs in total)
alvis-c1 - Linux centos linux 7.9.2009, x86_64, Intel Xeon Processor (Skylake), Python 3.6.8
See https://gist.github.com/4b953a8a5b4d972f52da89bede8dfa60 for a full test report.

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Aug 3, 2021

I'm going to have a bit of a hard time with the test reports since they require a GPU (and my test-build machine doesn't have one)

@branfosj
Copy link
Copy Markdown
Member

branfosj commented Aug 3, 2021

I'm going to have a bit of a hard time with the test reports since they require a GPU (and my test-build machine doesn't have one)

I've set off test reports from our GPU systems.

@branfosj
Copy link
Copy Markdown
Member

branfosj commented Aug 3, 2021

Test report by @branfosj
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
bask-pg0309u05a - Linux RHEL 8.4, x86_64, Intel(R) Xeon(R) Platinum 8360Y CPU @ 2.40GHz (icelake), Python 3.6.8
See https://gist.github.com/a29e7cfcf5e162d9ef039cd920eff949 for a full test report.

@branfosj
Copy link
Copy Markdown
Member

branfosj commented Aug 3, 2021

Test report by @branfosj
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
bear-pg0305u15a.bear.cluster - Linux RHEL 8.4, POWER, 8335-GTX (power9le), Python 3.6.8
See https://gist.github.com/292d0fa2d0ae02ab360844d5de3d11a0 for a full test report.

@branfosj
Copy link
Copy Markdown
Member

branfosj commented Aug 3, 2021

Test report by @branfosj
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
bask-pg0309u04a - Linux RHEL 8.4, x86_64, Intel(R) Xeon(R) Platinum 8360Y CPU @ 2.40GHz (icelake), Python 3.6.8
See https://gist.github.com/e46ed2426a844f145c266e3000e2b6c3 for a full test report.

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 boegel changed the title {bio}[foss-2020b]/[fosscuda-2020b] OpenMM v7.5.1 required for alphafold {bio}[foss-2020b]/[fosscuda-2020b] OpenMM v7.5.1 Aug 4, 2021
@boegel boegel changed the title {bio}[foss-2020b]/[fosscuda-2020b] OpenMM v7.5.1 {bio}[foss-2020b,fosscuda-2020b] OpenMM v7.5.1 Aug 4, 2021
@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 4, 2021

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node3301.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/574837b5f5eddd43a22f74be46e30a23 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 4, 2021

Going in, thanks @sassy-crick!

@boegel boegel merged commit 2f5f44e into easybuilders:develop Aug 4, 2021
@sassy-crick sassy-crick deleted the 20210717210532_new_pr_OpenMM751 branch August 6, 2021 14:33
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.

4 participants