Skip to content

{perf}[foss/2025b] OTF-CPT v0.9.4#24041

Merged
Crivella merged 7 commits intoeasybuilders:developfrom
felilxtomski:OTF-CPT-0.9.2
Oct 30, 2025
Merged

{perf}[foss/2025b] OTF-CPT v0.9.4#24041
Crivella merged 7 commits intoeasybuilders:developfrom
felilxtomski:OTF-CPT-0.9.2

Conversation

@felilxtomski
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

Updated software OTF-CPT-0.9.2-gompi-2025b.eb

Diff against OTF-CPT-0.9.2-iimpi-2025b.eb

easybuild/easyconfigs/o/OTF-CPT/OTF-CPT-0.9.2-iimpi-2025b.eb

Updated software OTF-CPT-0.9.2-iimpi-2025b.eb

Diff against OTF-CPT-0.9.2-gompi-2025b.eb

easybuild/easyconfigs/o/OTF-CPT/OTF-CPT-0.9.2-gompi-2025b.eb

@Thyre Thyre added new 2025b issues & PRs related to 2025b common toolchains and removed update labels Sep 29, 2025
Comment on lines +32 to +33
dependencies = [
]
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We should be able to omit the dependencies entirely since there are none.
I think the CI won't complain about that.

@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Sep 29, 2025

Would it make sense include a modloadmsg with information on how to use the software (which environment variables may need to get set)?

Based on https://github.com/RWTH-HPC/OTF-CPT?tab=readme-ov-file#using-the-tool-with-an-application, it looks like users may need to do some extra work, and setting OMP_TOOL_LIBRARIES generally in the module might not be the best idea I think.

@felilxtomski felilxtomski marked this pull request as ready for review September 29, 2025 09:36
@felilxtomski felilxtomski changed the title {perf}[gompi/2025b,iimpi/2025b] OTF-CPT v0.9.2 {perf}[gompi/2025b] OTF-CPT v0.9.2 Sep 29, 2025
@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Sep 29, 2025

Test report by @Thyre
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
Linux - Linux Arch Linux UNKNOWN, x86_64, AMD Ryzen 7 7800X3D 8-Core Processor (zen4), 1 x AMD Navi 48 [Radeon RX 9070/9070 XT/9070 GRE] (device id: 0x7550, gfx: gfx1201, driver: 6.16.8-arch3-1), 1 x AMD Raphael (device id: 0x164e, gfx: gfx1036, driver: 6.16.8-arch3-1), Python 3.13.7
See https://gist.github.com/Thyre/7bc888255e07191138d6b5f0dbfe8d06 for a full test report.

@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Sep 29, 2025

Test report by @Thyre
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
ZAM054 - Linux Zorin OS 17, x86_64, 12th Gen Intel(R) Core(TM) i7-1260P, 1 x NVIDIA NVIDIA GeForce MX550, 580.65.06, Python 3.10.12
See https://gist.github.com/Thyre/0846913eb6d33662b0bae9af14adf071 for a full test report.

@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Sep 29, 2025

Test report by @Thyre
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
jpbl-s02-03 - Linux RHEL 9.5, AArch64, ARM UNKNOWN (neoverse_v2), 1 x NVIDIA NVIDIA GH200 480GB, 570.133.20, Python 3.9.21
See https://gist.github.com/Thyre/3a579c1dcf9839153b6c6c29e4eb2355 for a full test report.

@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Sep 29, 2025

Hm, the installation includes CPT-plot.py, which includes several Python libraries.

To ensure that this tool works correctly, we should probably move Python to be a dependency, and add dependencies to matplotlib, Pillow and SciPy-bundle. This would also move OTF-CPT from gompi to foss.

(the script should also have e.g. #!/usr/bin/env python3 or similar in a future release, as this is using the shell otherwise)

@felilxtomski
Copy link
Copy Markdown
Contributor Author

@Thyre For some reason, I cannot directly reply to your comment. Anyhow, we will prepare another release with a fixed version of the plotting script in the next days and I'll update the easyconfig accordingly with proper runtime dependencies.

@felilxtomski felilxtomski marked this pull request as draft September 29, 2025 13:40
@felilxtomski felilxtomski changed the title {perf}[gompi/2025b] OTF-CPT v0.9.2 {perf}[foss/2025b] OTF-CPT v0.9.3 Oct 10, 2025
@felilxtomski
Copy link
Copy Markdown
Contributor Author

felilxtomski commented Oct 10, 2025

@Thyre The release took a bit longer than expected, but now it's finished. I also updated the configuration to have the necessary Python packages as a runtime dependency. Should this really be on the foss/intel level or rather gfbf/iimkl level?

@felilxtomski felilxtomski marked this pull request as ready for review October 10, 2025 13:45
@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Oct 10, 2025

Should this really be on the foss/intel level or rather gfbf/iimkl level?

Since MPI is required based on the CMakeLists.txt, we at least need gompi / iimpi. Since matplotlib is in gfbf / iimkl, we're forced to use foss/intel here as the toolchain containing both (if I haven't missed anything).

If there were a version without MPI, we could put that in gfbf / iimkl. However, I don't think it makes much sense to create a separate version for that, or invest the work to even allow this in the CMake logic.

@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Oct 10, 2025

Test report by @Thyre
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
Linux - Linux Arch Linux UNKNOWN, x86_64, AMD Ryzen 7 7800X3D 8-Core Processor (zen4), 1 x AMD Navi 48 [Radeon RX 9070/9070 XT/9070 GRE] (device id: 0x7550, gfx: gfx1201, driver: 6.17.1-arch1-1), 1 x AMD Raphael (device id: 0x164e, gfx: gfx1036, driver: 6.17.1-arch1-1), Python 3.13.7
See https://gist.github.com/Thyre/160bde6fcec13dbe2038242d5e1cbaf9 for a full test report.

@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Oct 10, 2025

Test report by @Thyre
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
jrc0901.jureca - Linux Rocky Linux 9.6, AArch64, ARM UNKNOWN (neoverse_v2), 1 x NVIDIA NVIDIA GH200 480GB, 580.65.06, Python 3.9.21
See https://gist.github.com/Thyre/9cd1f5812e0fd43ae996dac4a34df931 for a full test report.

@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Oct 10, 2025

Test report by @Thyre
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
datenlager - Linux Ubuntu 24.04, x86_64, AMD Ryzen 7 3700X 8-Core Processor, Python 3.12.3
See https://gist.github.com/Thyre/81b90d3448186d4995151a310b68cfb5 for a full test report.

Comment thread easybuild/easyconfigs/o/OTF-CPT/OTF-CPT-0.9.4-foss-2025b.eb
@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Oct 21, 2025

Test report by @Thyre
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
ZAM054 - Linux Zorin OS 18, x86_64, 12th Gen Intel(R) Core(TM) i7-1260P (skylake), 1 x NVIDIA NVIDIA GeForce MX550, 580.95.05, Python 3.12.3
See https://gist.github.com/Thyre/ca1cab32bdccaff81a694cf8edc85d0d for a full test report.

@Thyre Thyre added this to the next release (5.2.0?) milestone Oct 21, 2025
@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Oct 21, 2025

@boegelbot please test @ jsc-zen3

@boegelbot
Copy link
Copy Markdown
Collaborator

@Thyre: 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=24041 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_24041 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 3425391567 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.6, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/3acf4b0d1738ef02e779adea89eb4d99 for a full test report.

@Crivella
Copy link
Copy Markdown
Contributor

Test report by @Crivella
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
crivella-desktop - Linux Ubuntu 22.04.5 LTS (Jammy Jellyfish), x86_64, 13th Gen Intel(R) Core(TM) i9-13900K (skylake), Python 3.11.13
See https://gist.github.com/Crivella/43b7b388d8e380f4ed53bc73a9d2cc94 for a full test report.

Comment thread easybuild/easyconfigs/o/OTF-CPT/OTF-CPT-0.9.4-foss-2025b.eb
Comment thread easybuild/easyconfigs/o/OTF-CPT/OTF-CPT-0.9.4-foss-2025b.eb
Comment thread easybuild/easyconfigs/o/OTF-CPT/OTF-CPT-0.9.4-foss-2025b.eb
@felilxtomski felilxtomski changed the title {perf}[foss/2025b] OTF-CPT v0.9.3 {perf}[foss/2025b] OTF-CPT v0.9.4 Oct 30, 2025
@felilxtomski
Copy link
Copy Markdown
Contributor Author

@Crivella Thank you for your suggetions, I applied them. I also fixed the segfault triggered by the test in a new release and adjusted the easyconfig.

@Crivella
Copy link
Copy Markdown
Contributor

Test report by @Crivella
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
crivella-desktop - Linux Ubuntu 22.04.5 LTS (Jammy Jellyfish), x86_64, 13th Gen Intel(R) Core(TM) i9-13900K (skylake), Python 3.11.13
See https://gist.github.com/Crivella/18cd0abc764fc9bbfd0357eff63d3b68 for a full test report.

@Crivella
Copy link
Copy Markdown
Contributor

@boegelbot please test @ jsc-zen3

@boegelbot
Copy link
Copy Markdown
Collaborator

@Crivella: 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=24041 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_24041 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 3467340603 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.6, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/7f6c73c26fd143cfdeb11499cb533c69 for a full test report.

Copy link
Copy Markdown
Contributor

@Crivella Crivella left a comment

Choose a reason for hiding this comment

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

LGTM

@Crivella
Copy link
Copy Markdown
Contributor

Going in, thanks @felilxtomski!

@Crivella Crivella merged commit 11c80c3 into easybuilders:develop Oct 30, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2025b issues & PRs related to 2025b common toolchains new

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants