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

Fix TREXIO installation in the Verificarlo workflow #55

Merged
merged 8 commits into from
Dec 14, 2021

Conversation

q-posev
Copy link
Member

@q-posev q-posev commented Dec 13, 2021

Install trexio was failing due to the missing C compiler (cause the workflow is running in a verificarlo container).

  • Fix this by configuring TREXIO with GNU compilers (gcc-7 and gfortran-7) that are available in the verificarlo container.
  • Also install BLAS and LAPACK due to the recent change in the configure.ac of QMCkl (now both libs are required)
  • Fix bug due to the use of byte-strings to get the EXTRA_DIST in tools/build_makefile.py

TODO : upgrade the TREXIO version in .yml files when QMCkl is upgraded to use the recent TREXIO basis set format from v.2.0.

@q-posev q-posev changed the title Fix TREXIO installation in the Verificarlo worflow Fix TREXIO installation in the Verificarlo workflow Dec 13, 2021
@q-posev q-posev marked this pull request as draft December 13, 2021 10:06
@q-posev q-posev marked this pull request as ready for review December 13, 2021 11:58
@q-posev q-posev requested a review from scemama December 13, 2021 12:53
@scemama scemama merged commit 12fc653 into master Dec 14, 2021
@scemama scemama deleted the fix-trexio-with-verificarlo branch December 14, 2021 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants