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

Matched probe Language Bindings / Language Bindings Support addition #274

Closed
mpiforumbot opened this issue Jul 24, 2016 · 14 comments
Closed

Comments

@mpiforumbot
Copy link
Collaborator

mpiforumbot commented Jul 24, 2016

Originally by brbarret on 2011-05-10 13:21:46 -0500


The matched probe addition did not update two chapters impacted by the change: the Language Bindings and Language Bindings Support chapters.

In the Language Bindings chapter, the following is required in section 16.3.4 in the list of conversion functions:

MPI_Message MPI_Message_f2c(MPI_Fint message)
MPI_Fint MPI_Message_c2f(MPI_Message message)

The Language Bindings Support Annex is more complicated

In the NULL handles table:

  MPI_MESSAGE_NULL
    MPI_Message / INTEGER

In section A.1.2, C handles to assorted structures, add:

MPI_Message
@mpiforumbot
Copy link
Collaborator Author

Originally by gropp on 2011-05-11 11:25:00 -0500


I have reviewed this and compared it to Annex A in the 3.0 draft document, and it looks complete.

@mpiforumbot
Copy link
Collaborator Author

Originally by htor on 2011-05-11 11:42:49 -0500


Looks good to me too! Please not that this is an addendum/fix for ticket #38.

Torsten

@mpiforumbot
Copy link
Collaborator Author

Originally by jsquyres on 2011-05-11 12:32:37 -0500


Do you need to say "(use C value)" in the NULL handles table?

I'm comparing to the "Named Predefined Datatypes" table on p516, where it says "(use C datatype handle)" for all the new MPI-2.2 datatypes for which corresponding C++ datatype handles exist.

Everything else looks good.

@mpiforumbot
Copy link
Collaborator Author

Originally by jsquyres on 2011-05-11 12:57:24 -0500


Brian and I chatted; he added "Not defined for C++" in the C++ column, just like the "Status size and reserved index values" table on page 515.

@mpiforumbot
Copy link
Collaborator Author

Originally by brbarret on 2011-07-18 14:27:55 -0500


Attachment added: mpi-report.pdf (2291.4 KiB)

@mpiforumbot
Copy link
Collaborator Author

Originally by brbarret on 2011-07-18 14:31:21 -0500


Not sure why the final pdf wasn't on the ticket; it's the exact same pdf that was presented at the May meeting.

@mpiforumbot
Copy link
Collaborator Author

Originally by buntinas on 2011-10-25 10:55:21 -0500


No changes need to be applied to Info Object chapter.

@mpiforumbot
Copy link
Collaborator Author

Originally by jsquyres on 2012-06-01 13:28:15 -0500


Attachment added: merged-binding-2.pdf (369.6 KiB)
Merged result on the SVN "approved" branch

@mpiforumbot
Copy link
Collaborator Author

Originally by jsquyres on 2012-06-01 13:29:05 -0500


Bindings chapter changes merged to the SVN "approved" branch, and "merged-binding-2.pdf" is attached with the results. Please review.

(I can't speak for the other chapters involved)

@mpiforumbot
Copy link
Collaborator Author

Originally by RolfRabenseifner on 2012-06-28 13:46:58 -0500


appLang committed (SVN 1200)

@mpiforumbot
Copy link
Collaborator Author

Originally by jsquyres on 2012-07-03 07:18:52 -0500


appLang commit reviewed by Jeff.

@mpiforumbot
Copy link
Collaborator Author

Originally by jsquyres on 2012-07-18 09:59:01 -0500


Appears to be fully committed. Moving to "Waiting for PDF reviews".

@mpiforumbot
Copy link
Collaborator Author

Originally by chaarawi on 2012-07-18 11:19:07 -0500


reviewed. Looks good.

@mpiforumbot
Copy link
Collaborator Author

Originally by RolfRabenseifner on 2013-01-07 11:23:42 -0600


Since Sep. 21, 2012, this ticket is included in MPI-3.0 and the pdf is checked according to https://svn.mpi-forum.org/svn/mpi-forum-docs/trunk/meetings/2012-07-jul/mpi3-tickets.xlsx

Therefore, by proxy / on behalf of the ticket owner, I close it with priority "Ticket complete", resolution "Text committed".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant