Skip to content

ci: remove container, install mpi from python wheel#72

Merged
romanc merged 2 commits into
NOAA-GFDL:developfrom
romanc:romanc/wheels-mpi
Jul 22, 2025
Merged

ci: remove container, install mpi from python wheel#72
romanc merged 2 commits into
NOAA-GFDL:developfrom
romanc:romanc/wheels-mpi

Conversation

@romanc
Copy link
Copy Markdown
Collaborator

@romanc romanc commented Jul 22, 2025

Description

This is a follow-up from NOAA-GFDL/NDSL#184 (mpi4py >= 4.1), similar to NOAA-GFDL/pace#131, removing the mpi container in favor of installing a python wheel on the bare-metal Ubuntu runner. This allows the MPI tests in this testsuite to run again.

How Has This Been Tested?

By running CI and observing that "Orchestrated dace-cpu Acoustics" is now running again. Previously, that was just skipped.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas: N/A
  • I have made corresponding changes to the documentation: N/A
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • New check tests, if applicable, are included: N/A (old ones are restored though ...)
  • Targeted model if this changed was triggered by a model need/shortcoming: N/A

@romanc romanc marked this pull request as ready for review July 22, 2025 16:49
@romanc romanc merged commit c92ddfc into NOAA-GFDL:develop Jul 22, 2025
4 checks passed
@romanc romanc deleted the romanc/wheels-mpi branch July 22, 2025 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants