llnl / llnl/wrap

Newer MPI_Session handle is not recognized by wrap.py

Open
#25 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
37
Forks
24
PR merge metrics
No merged PRs in 30d

Description

Newer openmpi 5.0.* introduced a new handle MPI_Session which was not recognized by wrap.py. The code diff below adds the new definition to the handles list. The change addresses the build issue seen with spack. Apparently MPI_Session is in mpi standard 3.1.

```

66 | 166 |  
-- | -- | --
167 | 167 | # Set of MPI Handle types
168 | 168 | mpi_handle_types = set(["MPI_Comm", "MPI_Errhandler", "MPI_File", "MPI_Group", "MPI_Info",
169 |   | - "MPI_Op", "MPI_Request", "MPI_Status", "MPI_Datatype", "MPI_Win" ])
  | 169 | + "MPI_Op", "MPI_Request", "MPI_Status", "MPI_Datatype", "MPI_Win", "MPI_Session" ])
170 | 170 |  
171 | 171 | # MPI Calls that have array parameters, and mappings from the array parameter positions to the position

```
Just letting the wrap team know that this issue is out there. We fixed our copy of wrap.py that we have in our code.
Thanks,
Jim G

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in wrap.py at the mpi_handle_types set around the lines shown in the issue. Confirm that MPI_Session from the Open MPI 5.0 build failure is represented there, then verify that wrap.py completes the affected generation path without the reported build error.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
tooling
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.