Skip to content

use 32-bit indexing for SUNDIALS 6.6.0 (with foss/2023a) to enable sunmatrix-cusparse#20075

Merged
Micket merged 1 commit intoeasybuilders:developfrom
appolloford:20240311084436_new_pr_SUNDIALS660
Mar 21, 2024
Merged

use 32-bit indexing for SUNDIALS 6.6.0 (with foss/2023a) to enable sunmatrix-cusparse#20075
Micket merged 1 commit intoeasybuilders:developfrom
appolloford:20240311084436_new_pr_SUNDIALS660

Conversation

@appolloford
Copy link
Copy Markdown
Contributor

@appolloford appolloford commented Mar 11, 2024

(created using eb --new-pr)

https://sundials.readthedocs.io/en/v6.6.0/sunmatrix/SUNMatrix_links.html#sunmatrix-cusparse-usage-notes

The SUNMATRIX_CUSPARSE module only supports 32-bit indexing, thus SUNDIALS must be built for 32-bit indexing to use this module.

@appolloford appolloford changed the title Use 32-bit indexing to enable sunmatrix-cusparse {math}[foss/2023a] SUNDIALS v6.6.0: Use 32-bit indexing to enable sunmatrix-cusparse Mar 11, 2024
@Micket Micket added the bug fix label Mar 11, 2024
Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

lgtm

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Mar 21, 2024

Test report by @Micket
FAILED
Failed during parsing of the easyconfigs, so no ecs were built (1 easyconfigs in total)
vera-gpu1 - Linux Rocky Linux 8.9, x86_64, Intel(R) Xeon(R) Gold 6130 CPU @ 2.10GHz, 1 x NVIDIA Tesla V100-SXM2-32GB, 550.54.14, Python 3.6.8
See https://gist.github.com/Micket/0ff8f88694b6a95278d23f6af8c77048 for a full test report.

@Micket Micket added this to the release after 4.9.0 milestone Mar 21, 2024
@appolloford
Copy link
Copy Markdown
Contributor Author

Test report by @appolloford
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
vera-gpu1 - Linux Rocky Linux 8.9, x86_64, Intel(R) Xeon(R) Gold 6130 CPU @ 2.10GHz, 1 x NVIDIA Tesla V100-SXM2-32GB, 550.54.14, Python 3.6.8
See https://gist.github.com/appolloford/314642ec96af5ada5b5f248f0c700221 for a full test report.

@appolloford
Copy link
Copy Markdown
Contributor Author

Test report by @Micket FAILED Failed during parsing of the easyconfigs, so no ecs were built (1 easyconfigs in total) vera-gpu1 - Linux Rocky Linux 8.9, x86_64, Intel(R) Xeon(R) Gold 6130 CPU @ 2.10GHz, 1 x NVIDIA Tesla V100-SXM2-32GB, 550.54.14, Python 3.6.8 See https://gist.github.com/Micket/0ff8f88694b6a95278d23f6af8c77048 for a full test report.

I don't know why it says the file cannot be found...

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Mar 21, 2024

@boegelbot please test @ jsc-zen3

@boegelbot
Copy link
Copy Markdown
Collaborator

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

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

Test results coming soon (I hope)...

Details

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

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Mar 21, 2024

Going in, thanks @appolloford!

@Micket Micket merged commit c7aea50 into easybuilders:develop Mar 21, 2024
@boegel boegel changed the title {math}[foss/2023a] SUNDIALS v6.6.0: Use 32-bit indexing to enable sunmatrix-cusparse use 32-bit indexing for SUNDIALS 6.6.0 (with foss/2023a) to enable sunmatrix-cusparse Mar 27, 2024
@appolloford appolloford deleted the 20240311084436_new_pr_SUNDIALS660 branch March 31, 2025 10:37
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.

3 participants