Skip to content

{phys}[intel/2016b] BAGEL v1.1.1#6332

Merged
boegel merged 5 commits intoeasybuilders:developfrom
Micket:20180517163725_new_pr_BAGEL111
May 18, 2018
Merged

{phys}[intel/2016b] BAGEL v1.1.1#6332
boegel merged 5 commits intoeasybuilders:developfrom
Micket:20180517163725_new_pr_BAGEL111

Conversation

@Micket
Copy link
Copy Markdown
Contributor

@Micket Micket commented May 17, 2018

(created using eb --new-pr)

@vanzod vanzod added the new label May 17, 2018
@vanzod
Copy link
Copy Markdown
Member

vanzod commented May 17, 2018

Test report by @vanzod
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
cermis - Linux debian 9.4, Intel(R) Xeon(R) CPU E5620 @ 2.40GHz, Python 2.7.13
See https://gist.github.com/3b2392cdc98e1173b0134478081e5ad4 for a full test report.

is a parallel electronic-structure program."""

toolchain = {'name': 'intel', 'version': '2016b'}
# Note: A compiler bug in template deduction prevents newer versions of icpc to compile this software.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Any reference to more details on this issue (e.g. a URL where the bug is discussed)? If so, please include it as a comment as well, and maybe move this comment above the toolchain = line?

Copy link
Copy Markdown
Contributor Author

@Micket Micket May 18, 2018

Choose a reason for hiding this comment

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

I may have been to hasty to call it a compiler bug. I did a lot more (tedious) debugging and it all boils down to boost_serialization, and is probably related to https://github.com/nubakery/bagel/blob/master/src/util/serialization.h#L104

Only seems to affect the more recent icpc versions though. GCC compiles without errors.
So, I'm not sure if it's a compiler bug, boost-bug, and I have no link.
Poking around and trying different thing in serialization.h didn't help either, so I have no patch (GCC doesn't seem to care whether or not these shared_ptr specializations are present, compiles either way).

But, as bagel doesn't really support OpenBLAS either (qsimulate-open/bagel#116, qsimulate-open/bagel#117) i couldn't go for foss. Though, that issue should be resolved in the next release of OpenBLAS and bagel.
I currently are not building any g*mkl toolchains, but those would probably be the best fit for BAGEL.

preconfigopts = 'sed -i "s|-gcc-mt||g" configure.ac && '
preconfigopts += './autogen.sh && '
preconfigopts += 'CXXFLAGS="$CXXFLAGS -DNDEBUG" '
configopts = ' --with-boost=$BOOST_ROOT --with-mpi=intel --enable-mkl --with-libxc '
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Micket Please move this block up, above sanity_check_paths (just to follow the order we use in other easyconfigs)

]

sanity_check_paths = {
'files': ['bin/BAGEL', 'lib/libbagel.so'],
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Micket Please use SHLIB_EXT rather than hardcoding .so:

sanity_check_paths = {
+    'files': ['bin/BAGEL', 'lib/libbagel.%s' % SHLIB_EXT],

version = '1.1.1'

homepage = "http://www.nubakery.org"
description = """BAGEL (Brilliantly Advanced General Electronic-structure Lirary)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Typo: Lirary -> Library

builddependencies = [
('Autoconf', '2.69'),
('Automake', '1.15'),
('libtool', '2.4.6'),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Replace all those build dependencies with the equivalent ('Autotools', '20150215') bundle

@boegel
Copy link
Copy Markdown
Member

boegel commented May 18, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2435.golett.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/03e34759971a09b0e98174dcfaffd95b for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 18, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2065.delcatty.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/c163a6e05a2369fa6741154773a7700e for a full test report.

@boegel boegel added this to the 3.6.1 milestone May 18, 2018
@boegel
Copy link
Copy Markdown
Member

boegel commented May 18, 2018

Going in, thanks @Micket!

@boegel boegel merged commit 0db0269 into easybuilders:develop May 18, 2018
@Micket Micket deleted the 20180517163725_new_pr_BAGEL111 branch May 2, 2019 19:09
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