{chem}[foss/2023a] VeloxChem 1.0-rc4#24176
Conversation
|
@boegelbot please test @ jsc-zen3 |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3386032242 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@Crivella Does this depend on some other PR/fix? When I test this, I get a failure: |
|
Test report by @boegel |
|
Not sure if it is related to or something similar to but i think having a recent rebuild of FlexiBLAS might help. I would assume it is something already in develop since the bot succeeded? |
|
Worst case we can set it to https://gitlab.com/veloxchem/veloxchem/-/blob/v1.0-rc4/.github/workflows/ci.yml#L73 |
|
NOTE this is what i get EDIT: which on second inspection might need some extra sauce to avoid the same problem as in of mixing FlexiBLAS libs with the OpenBLAS headers |
|
@Crivella Why doesn't this happen consistently though? |
|
The CMake part for the lapack/blas seems to rely either on I am trying to fiddle with the first to force it to pick all stuff from FlexiBLAS. Concerning the failure in your build i am not sure why it is not finding |
|
Another note, using FlexiBLAS or any string that is not recognized is defering to the The only special values for
|
|
@boegelbot please test @ jsc-zen3 |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3397788101 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @Crivella NOTES |
|
Test report by @boegelbot |
|
Test report by @boegel |
|
Now it is picking up the libraries but failing somewhere else. Another difference i noticed in the logs, is for your build, the test for using the Ninja generator is failing and falling back to I would assume it is easier to solve this problem than whatever is causing the Makefiles build version to fail From my logs: |
|
The reason this is failing is the other way around (will fail with any new build of OpenBLAS after #2753) i thought i had a newer build of OpenBLAS/FlexiBLAS but not for this TC (and i guess the bot as well). The error in the frist failed report was due to failing the check (i assume since cblas_sgemm64_ is what is defined in cblas.h ) With the patch now it is picking up the include and lib folders from FlexiBLAS so it is passing that check, but for some reason during the compilation it still seems to try and use the file from OpenBLAS I will try to recompile a newer OpenBLAS/FlexiBLAS to reproduce the error and see how it can be fixed EDIT VeloxChem build system passes the include dir from the detected cblas.h as a system header with -isystem . |
|
@boegelbot please test @ jsc-zen3 |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3407215490 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @Crivella |
|
Test report by @boegelbot |
|
Test report by @boegel |
…in VeloxChem easyconfig
|
Test report by @boegel |
|
Going in, thanks @Crivella! |
Add EC file for VeloxChem