Skip to content

ABI: Missing keys "mpi_{integer|logical}16_supported" #979

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

Open
dalcinl opened this issue Mar 24, 2025 · 4 comments
Open

ABI: Missing keys "mpi_{integer|logical}16_supported" #979

dalcinl opened this issue Mar 24, 2025 · 4 comments
Assignees
Labels
chap-abi ABI Chapter Committee Chapter Committee Change Changes to be made by the respective chapter committee(s) mpi-5.0 For inclusion in the MPI 5.0 standard wg-abi ABI Working Group

Comments

@dalcinl
Copy link

dalcinl commented Mar 24, 2025

The listing of predefined info keys in MPI_ABI_GET_FORTRAN_INFO is missing the "mpi_{integer|logical}16_supported" entries.

@jeffhammond simply forgot to remove the comment in two lines of the LaTeX sources:

diff --git a/chap-abi/abi.tex b/chap-abi/abi.tex
index 72280f73..21355944 100644
--- a/chap-abi/abi.tex
+++ b/chap-abi/abi.tex
@@ -433,12 +433,12 @@ The following keys are predefined for this object:
 \item[\infokeymain{mpi\_logical2\_supported}:]  (boolean) \mpidtype{MPI\_LOGICAL2} is supported.
 \item[\infokeymain{mpi\_logical4\_supported}:]  (boolean) \mpidtype{MPI\_LOGICAL4} is supported.
 \item[\infokeymain{mpi\_logical8\_supported}:]  (boolean) \mpidtype{MPI\_LOGICAL8} is supported.
-%\item[\infokeymain{mpi\_logical16\_supported}:] (boolean) \mpidtype{MPI\_LOGICAL16} is supported.
+\item[\infokeymain{mpi\_logical16\_supported}:] (boolean) \mpidtype{MPI\_LOGICAL16} is supported.
 \item[\infokeymain{mpi\_integer1\_supported}:]  (boolean) \mpidtype{MPI\_INTEGER1} is supported.
 \item[\infokeymain{mpi\_integer2\_supported}:]  (boolean) \mpidtype{MPI\_INTEGER2} is supported.
 \item[\infokeymain{mpi\_integer4\_supported}:]  (boolean) \mpidtype{MPI\_INTEGER4} is supported.
 \item[\infokeymain{mpi\_integer8\_supported}:]  (boolean) \mpidtype{MPI\_INTEGER8} is supported.
-%\item[\infokeymain{mpi\_integer16\_supported}:] (boolean) \mpidtype{MPI\_INTEGER16} is supported.
+\item[\infokeymain{mpi\_integer16\_supported}:] (boolean) \mpidtype{MPI\_INTEGER16} is supported.
 \item[\infokeymain{mpi\_real2\_supported}:]     (boolean) \mpidtype{MPI\_REAL2} is supported.
 \item[\infokeymain{mpi\_real4\_supported}:]     (boolean) \mpidtype{MPI\_REAL4} is supported.
 \item[\infokeymain{mpi\_real8\_supported}:]     (boolean) \mpidtype{MPI\_REAL8} is supported.
@dalcinl dalcinl added chap-abi ABI Chapter Committee Chapter Committee Change Changes to be made by the respective chapter committee(s) mpi-5.0 For inclusion in the MPI 5.0 standard wg-abi ABI Working Group labels Mar 24, 2025
@github-project-automation github-project-automation bot moved this to To Do in MPI 5.0 Mar 24, 2025
@wesbland
Copy link
Member

@jeffhammond Is this something you plan to do as a no-no for MPI 5?

@jeffhammond
Copy link
Member

If necessary, but it's a very clear latex typo that should be fixed as an editorial change. If we don't make this change, we'll have an inconsistent document.

@wesbland
Copy link
Member

Cool. So.... PR then?

@dalcinl
Copy link
Author

dalcinl commented Apr 17, 2025

Cool. So.... PR then?

mpi-forum/mpi-standard#1120

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chap-abi ABI Chapter Committee Chapter Committee Change Changes to be made by the respective chapter committee(s) mpi-5.0 For inclusion in the MPI 5.0 standard wg-abi ABI Working Group
Projects
Status: To Do
Development

No branches or pull requests

3 participants