Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mpi on candide #661

Open
wants to merge 23 commits into
base: develop
Choose a base branch
from
Open

Conversation

martinkilbinger
Copy link
Contributor

@martinkilbinger martinkilbinger commented Oct 31, 2023

Summary

A number of fixes are proposed to get MPI running again (on candide):

  • Version upgrades (gcc -> 9.3.0, openmpi -> 5.0.0, numpy -> 1.22)
  • MPI job script update
  • Some small fixes (module_config_sec was not communicated to clients)
  • Module name fixes in example config file
  • Minor other fixes in modules

Need to wait for pysap and subsequent mccd update, see CEA-COSMIC/pysap#187

Reviewer Checklist

Reviewers should tick the following boxes before approving and merging the PR.

  • The PR targets the develop branch
  • The PR is assigned to the developer
  • The PR has appropriate labels
  • The PR is included in appropriate projects and/or milestones
  • The PR includes a clear description of the proposed changes
  • If the PR addresses an open issue the description includes "closes #"
  • The code and documentation style match the current standards
  • Documentation has been added/updated consistently with the code
  • All CI tests are passing
  • API docs have been built and checked at least once (if relevant)
  • All changed files have been checked and comments provided to the developer
  • All of the reviewer's comments have been satisfactorily addressed by the developer

@martinkilbinger martinkilbinger added the bug Something isn't working label Oct 31, 2023
@martinkilbinger martinkilbinger added this to the P3 re-run 2023 milestone Oct 31, 2023
@martinkilbinger martinkilbinger self-assigned this Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: To do
Development

Successfully merging this pull request may close these issues.

2 participants