MFiX/24.4.1-intel-2023b#22212
Conversation
| '-DCMAKE_Fortran_COMPILER=mpiifort', | ||
| '-DCMAKE_C_COMPILER=mpiicc', | ||
| '-DCMAKE_CXX_COMPILER=mpiicpc', | ||
| '-DCMAKE_Fortran_FLAGS="-O2 -Wall -fcheck=all"', |
There was a problem hiding this comment.
Are you sure these are necessary? Setting the MPI compilers is done automatically via
toolchainopts = {'usempi': True}
For the Fortran flags, you may need additional options, but you can also use toolchainopts to set those, e.g.,
toolchainopts = {'extra_fflags': '-fcheck=all'}
You can always check what environment variables are being set, and what the configure command looks like using --trace
There was a problem hiding this comment.
Sure. I tried to adapt your requested changes in 0fef015.
|
@boegelbot please test @ jsc-zen3 |
|
@ocaisa: 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 2615618997 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
| toolchainopts = { | ||
| 'usempi': True, | ||
| 'pic': True, | ||
| 'extra_fflags': '-fallow-argument-mismatch', |
There was a problem hiding this comment.
In the logs I've seen, this is actually not recognised by the compiler:
ifort: command line warning #10006: ignoring unknown option '-fallow-argument-mismatch'
There was a problem hiding this comment.
Shall I keep this option really?
There was a problem hiding this comment.
Not if you don't need it for the build
The software is open-source and free of charge, but the access to the different versions of the source code are behind a login (more and less like MATLAB). How can we let different platforms test the build then? |
You can add download instructions to EasyConfigs. People with access to the test clusters can then download the software beforehand and trigger the bot to do the build. See https://github.com/easybuilders/easybuild-easyconfigs/blob/develop/easybuild/easyconfigs/p/PDT/PDT-3.25.2-GCCcore-13.3.0.eb for example |
Sure thing. I included that in 06b68e5 |
|
I've requested an account, when I can get the sources I will trigger a build so we can get a test report. |
|
Still waiting for access to the sources... |
This waiting time is odd. Shall we contact them directly and mention the request? |
|
@ocaisa: 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 2701511779 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@boegelbot please test @ jsc-zen3 |
|
@boegelbot please test @ jsc-zen3 |
|
@ocaisa: 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 2701581631 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Unfortunately this requires a framework PR to fix the style checks: https://github.com/easybuilders/easybuild-framework/blob/6e68abeaeb54789a6be46ebe398384d2203c82d4/easybuild/framework/easyconfig/style.py#L103 |
|
@boegelbot please test @ jsc-zen3 |
|
@ocaisa: 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 2703197421 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
What a marathon, thanks for the endurance @moravveji ! |
|
THANKS @ocaisa for all great inputs, and the pushes ;-) |
With this PR, I share a new easyconfig for MFiX (open source alternative to Fluent).
Remarks: