Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

{chem}[intel/2021a] CP2K v9.1 #15147

Closed

Conversation

branfosj
Copy link
Member

@branfosj branfosj commented Mar 19, 2022

(created using eb --new-pr)

Draft while we check the tests and dependencies.

@branfosj branfosj marked this pull request as draft March 19, 2022 16:56
@branfosj
Copy link
Member Author

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

@boegel boegel added this to the 4.x milestone Apr 26, 2022
@boegel
Copy link
Member

boegel commented Apr 26, 2022

@branfosj Any updates on this?

@branfosj
Copy link
Member Author

branfosj commented Apr 27, 2022

--------------------------------- Summary --------------------------------
Number of FAILED  tests 3621
Number of WRONG   tests 0
Number of CORRECT tests 7
Total number of   tests 3628
--------------------------------------------------------------------------
Number of memory  leaks 0
--------------------------------------------------------------------------
GREPME 3621 0 7 0 3628

Summary: correct: 7 / 3628; failed: 3621; 70min
Status: FAILED

The failures are an issue with OpenMP jobs and the FFTW we build with the Intel compiler. If we remove the EB built FFTW dependency then I see (and this changes toolchain from intel to iomkl):

--------------------------------- Summary --------------------------------
Number of FAILED  tests 1
Number of WRONG   tests 5
Number of CORRECT tests 3622
Total number of   tests 3628
--------------------------------------------------------------------------
Number of memory  leaks 0
--------------------------------------------------------------------------
GREPME 1 5 3622 0 3628

Summary: correct: 3622 / 3628; wrong: 5; failed: 1; 225min
Status: FAILED

The wrong results ones are similar to the foss version, with the tolerances being missed by about the same amounts.

@branfosj
Copy link
Member Author

branfosj commented Aug 9, 2022

I'm not working on this, so I've closed the PR.

@branfosj branfosj closed this Aug 9, 2022
@branfosj branfosj deleted the 20220319140013_new_pr_CP2K91 branch February 10, 2023 18:28
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.

2 participants