Skip to content

MFiX/24.4.1-intel-2023b#22212

Merged
ocaisa merged 21 commits intoeasybuilders:developfrom
moravveji:mfix-2023b
Mar 6, 2025
Merged

MFiX/24.4.1-intel-2023b#22212
ocaisa merged 21 commits intoeasybuilders:developfrom
moravveji:mfix-2023b

Conversation

@moravveji
Copy link
Copy Markdown
Contributor

With this PR, I share a new easyconfig for MFiX (open source alternative to Fluent).

Remarks:

  • testing is done in the build folder, and the test sources/artefacts are preserved (i.e. not copied to the install folder)
  • even thought the software is compiled with Intel MPI, the test is done serially, because I did not know the correct configuration for running the test case and the correct number of MPI processes needed

@github-actions github-actions Bot added the new label Jan 24, 2025
Comment thread easybuild/easyconfigs/m/MFiX/MFiX-24.3-intel-2023b.eb Outdated
Comment on lines +33 to +36
'-DCMAKE_Fortran_COMPILER=mpiifort',
'-DCMAKE_C_COMPILER=mpiicc',
'-DCMAKE_CXX_COMPILER=mpiicpc',
'-DCMAKE_Fortran_FLAGS="-O2 -Wall -fcheck=all"',
Copy link
Copy Markdown
Member

@ocaisa ocaisa Jan 27, 2025

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure. I tried to adapt your requested changes in 0fef015.

Comment thread easybuild/easyconfigs/m/MFiX/MFiX-24.3-intel-2023b.eb Outdated
Comment thread easybuild/easyconfigs/m/MFiX/MFiX-24.3-intel-2023b.eb Outdated
@ocaisa
Copy link
Copy Markdown
Member

ocaisa commented Jan 27, 2025

@boegelbot please test @ jsc-zen3

Comment thread easybuild/easyconfigs/m/MFiX/MFiX-24.3-intel-2023b.eb Outdated
@boegelbot
Copy link
Copy Markdown
Collaborator

@ocaisa: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=22212 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_22212 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 5606

Test results coming soon (I hope)...

Details

- notification for comment with ID 2615618997 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

Test report by @boegelbot
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.5, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/749e3a5758557e08d12302cbbc3c2060 for a full test report.

Comment thread easybuild/easyconfigs/m/MFiX/MFiX-24.3-intel-2023b.eb Outdated
toolchainopts = {
'usempi': True,
'pic': True,
'extra_fflags': '-fallow-argument-mismatch',
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.

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'

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.

Did you mean -fcheck=all?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Shall I keep this option really?

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.

Not if you don't need it for the build

@moravveji
Copy link
Copy Markdown
Contributor Author

Test report by @boegelbot FAILED Build succeeded for 0 out of 1 (1 easyconfigs in total) jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.5, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21 See https://gist.github.com/boegelbot/749e3a5758557e08d12302cbbc3c2060 for a full test report.

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?

@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Jan 27, 2025

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

@moravveji
Copy link
Copy Markdown
Contributor Author

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

Comment thread easybuild/easyconfigs/m/MFiX/MFiX-24.3-intel-2023b.eb
Comment thread easybuild/easyconfigs/m/MFiX/MFiX-24.3-intel-2023b.eb Outdated
Comment thread easybuild/easyconfigs/m/MFiX/MFiX-24.3-intel-2023b.eb Outdated
@ocaisa
Copy link
Copy Markdown
Member

ocaisa commented Jan 29, 2025

I've requested an account, when I can get the sources I will trigger a build so we can get a test report.

@ocaisa
Copy link
Copy Markdown
Member

ocaisa commented Feb 3, 2025

Still waiting for access to the sources...

@moravveji
Copy link
Copy Markdown
Contributor Author

Still waiting for access to the sources...

This waiting time is odd. Shall we contact them directly and mention the request?

@boegelbot
Copy link
Copy Markdown
Collaborator

@ocaisa: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=22212 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_22212 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 5874

Test results coming soon (I hope)...

Details

- notification for comment with ID 2701511779 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.5, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/c1a4eec86cd4e1e700d2c7d6bc6f9c41 for a full test report.

Comment thread easybuild/easyconfigs/m/MFiX/MFiX-24.4.1-intel-2023b.eb Outdated
Comment thread easybuild/easyconfigs/m/MFiX/MFiX-24.4.1-intel-2023b.eb Outdated
Comment thread easybuild/easyconfigs/m/MFiX/MFiX-24.4.1-intel-2023b.eb Outdated
@ocaisa
Copy link
Copy Markdown
Member

ocaisa commented Mar 5, 2025

@boegelbot please test @ jsc-zen3

@easybuilders easybuilders deleted a comment from boegelbot Mar 5, 2025
@easybuilders easybuilders deleted a comment from boegelbot Mar 5, 2025
ocaisa
ocaisa previously approved these changes Mar 5, 2025
Copy link
Copy Markdown
Member

@ocaisa ocaisa left a comment

Choose a reason for hiding this comment

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

LGTM

@ocaisa
Copy link
Copy Markdown
Member

ocaisa commented Mar 5, 2025

@boegelbot please test @ jsc-zen3

@boegelbot
Copy link
Copy Markdown
Collaborator

@ocaisa: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=22212 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_22212 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 5876

Test results coming soon (I hope)...

Details

- notification for comment with ID 2701581631 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.5, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/5eb0329bd767a03e5ac5f50c66652776 for a full test report.

@ocaisa
Copy link
Copy Markdown
Member

ocaisa commented Mar 5, 2025

@ocaisa
Copy link
Copy Markdown
Member

ocaisa commented Mar 6, 2025

@boegelbot please test @ jsc-zen3

@boegelbot
Copy link
Copy Markdown
Collaborator

@ocaisa: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=22212 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_22212 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 5891

Test results coming soon (I hope)...

Details

- notification for comment with ID 2703197421 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.5, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/3dfb89b95f27529d8d6a2e05102045d4 for a full test report.

@ocaisa
Copy link
Copy Markdown
Member

ocaisa commented Mar 6, 2025

What a marathon, thanks for the endurance @moravveji !

@ocaisa ocaisa merged commit 4597baf into easybuilders:develop Mar 6, 2025
@moravveji
Copy link
Copy Markdown
Contributor Author

THANKS @ocaisa for all great inputs, and the pushes ;-)

@moravveji moravveji deleted the mfix-2023b branch March 6, 2025 12:46
@boegel boegel added this to the 5.0.0 milestone Mar 18, 2025
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.

5 participants