Changes for RC 40 as discussed at the MPI forum Dec 7-10, 2020 for Chapters 1, 2, 8, 16-18, A, B & others

[RolfRabenseifner]

Problem

Summary of all changes reported by the reviewers for mainly Chapters 2, 8, 16-18, A, B and summarized in the attached pdf.

Proposal

As voted on the RC 40, the changes shall be stored inseveral related pull requests

Changes to the Text

See pdf and pull requests

Latest commented pdf with all changes from MPI-4.0 RC-1 (Dec. 2020) to RC-2 (Feb. 2021)
that are related to this issue:

mpi40-rc-feb-21_rab.pdf
and especially for PR547 MPI_Sessions_finalize: mpi40-issue435-PR547-sessions-2021-FEB-26-v1.pdf

List of latest pull requests that should be added to MPI-4.0 RC 2:

Both, base PR and update PR not yet voted in:

• PR528 to be updated with PR534 - in Chap. 2 Terms
• PR484 to be updated with PR533 - in Chap. 7 Groups, Contexts
• PR434 to be updated with PR544 - in Chap. 18 Semantic changes (formerly Backward incompatibilities) (+ not updated Chap. 19 and Annex B)
• PR444 to be updated with PR528 - in Annex A.1
• PR455 to be updated with PR425 and PR540 - in Chap. 14 I/O (changes by Tony)

For the following updates, the base is already voted in:

• PR351 to be updated with PR531 - in Chap. 5 Datatypes
• PR429 to be updated with PR529 - in Annex B Change-Log
• (PR412) to be updated with PR530 - in Chap. 19 Language Bindings
• Issue 341 to be updated with PR532 - in Chap. 6 Collectives

Additional changes also reported in the pdf

• Issue 437 to be updated with PR517 - in Chap. 4 Partitioned Communication
  (Additional question: is MPI_Parrived required to make progress? Answer from the Forum meeting Feb.2021: YES and change name to MPI_Ptest)

• Issue 435 to be updated with PR547 - in Chap. 11.3 Sessions Model

Impact on Implementations

None - all are corrections to the latex text only to remove errors or inconsistencies.

Impact on Users

None.

Open, merged, and other issues/PR related to this issue 405

In the latest commented pdf (see below), the following open issues are reported (see "#-TODO" and "Done" comments in the pdf):

• Merge of PR427 https://github.com/mpi-forum/mpi-standard/pull/427 Sorting of the changelog
• Merge of PR434 https://github.com/mpi-forum/mpi-standard/pull/434 Corrections to Chap. 18, 19, and Annex B
• Merge of PR439 https://github.com/mpi-forum/mpi-standard/pull/439 Corrections to Chap. 8 Topologies
• Merge of PR443 https://github.com/mpi-forum/mpi-standard/pull/442 Corrections to Chap.10 The Info Object
• Merge of PR442 https://github.com/mpi-forum/mpi-standard/pull/442 Corrections to Chap.16 Deprecated Interfaces
• Merge of PR441 https://github.com/mpi-forum/mpi-standard/pull/441 Corrections to Chap.17 Removed Interfaces
• Merge of PR444 https://github.com/mpi-forum/mpi-standard/pull/444 Corrections to Annex A. Language Binding Summaries
• Merge of PR449 https://github.com/mpi-forum/mpi-standard/pull/449 Corrections to the Callback Prototype Function Index and some smaller ones in other chapters
• Additional modifications of the PR449 mpi-user-macs.tex to better fit to the existing concept.
• Merge of PR461 https://github.com/mpi-forum/mpi-standard/pull/461 Corrections to the Change-Log
• Merge of PR486 https://github.com/mpi-forum/mpi-standard/pull/486 Additional primary Fortran entries in the Declaration Index
• Merge of PR497 https://github.com/mpi-forum/mpi-standard/pull/497 Additional changes to appLang-Const.tex according to Issue Language Bindings Summary - Appendix A (partial) RC review #375
• Merge of PR498 https://github.com/mpi-forum/mpi-standard/pull/498 Additional changes to deprecated.tex according to Issue Deprecated chapter RC review #366
• Merge of PR499 https://github.com/mpi-forum/mpi-standard/pull/499 Adding " !(_c)" postfix embiggened mpi_f08 bindings according to Issue Language Bindings Summary Chapter RC Review #364
• Merge of PR493 https://github.com/mpi-forum/mpi-standard/pull/493 Secondary callback index entries at MPI...CREATE... routines
• Merge of PR500 https://github.com/mpi-forum/mpi-standard/pull/500 Problem with 2-d arguments reported in Language Bindings Summary Chapter RC Review #364 (comment)
• Merge of PR501 https://github.com/mpi-forum/mpi-standard/pull/501 Solving MPI_CONVERSION_FN_NULL embiggening problem in Issue Bug in Fortran embiggening MPI_CONVERSION_FN_NULL #432
• Merge of PR504 https://github.com/mpi-forum/mpi-standard/pull/504 Solving Annex A problems listed in Issue Language Bindings Chapter RC Review #332
• Merge of PR506 https://github.com/mpi-forum/mpi-standard/pull/506 Solving Annex A final problems listed in Issue Language Bindings Chapter RC Review #332
• Merge of PR508 https://github.com/mpi-forum/mpi-standard/pull/508 Small corrections to Chap. 1, listed in Issue Introduction Chapter RC Review Issues #349

Related Pull Requests from other authors (see also "#-TODO" and "Done" comments in the latest commented pdf below):

• Merge of PR400 https://github.com/mpi-forum/mpi-standard/pull/400 - resolves MPI_MAX_FROM_GROUP_TAG vs. MPI_MAX_STRINGTAG_LEN issues
• Merge of PR409 https://github.com/mpi-forum/mpi-standard/pull/409 - resolves most of the MPI-T callback prototype issues
• Merge of PR423 https://github.com/mpi-forum/mpi-standard/pull/423 - removal of PMPI_T_... references in the function index
• Merge of PR429 https://github.com/mpi-forum/mpi-standard/pull/429 - backward incompatibility and change-log text for modification of MPI_T_CATEGORY_CHANGED
• Adding change-log and backward incompatibility entry for change in PR429
• Page 782 of update 7 and page 788 line 22 in mpi40-report_Issue405_PR449_v03.pdf: Is the formatting with a 8pt longer line acceptable?
• A.3, A.4, A.5: Missing Hyperrefs to the primary locations of the functions and types (see also comments in PR413 and Issue Include Hyperlinks in Language Bindings #422)
  • for the function names, see Issue Include Hyperlinks in Language Bindings #422 and PR476
  • for all type declarations
• Verfication whether the changelog is complete, e.g., references all new functions, types, and constants correctly (done in PR461)
• Checking whether all references in the MPI Indexes have a useful primary index:
  • Constants (--> may be postponed until MPI-4.1)
  • Declarations (all Fortran types should additionally get the primary references of the C types: don by PR486)
  • Callbacks (done by PR449)
  • Functions witout _c
  • Functions with _c (by PR474)
• Formatting problem "Linebreak before '*)'" in Fortran old-style binding of MPI_Waitall and MPI_Testall, reported in Issue #364

See also "#-TODO" and "Done" comments in the latest version of the commented mpi-4-rc pdf file:
mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update14.pdf

References

• mpi40-rc-nov-20_rab_Chap-8+16+17+A+B.pdf as voted for rc-40 at the MPI forum meeting Dec 7-10, 2020

• ( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update1.pdf update 1 since rc-40 vote )

• ( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update2.pdf )

• ( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update3.pdf updated after PR 439 )

• ( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update4.pdf updated after PR 443 )

• ( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update5.pdf updated after PR 444 )

• ( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update7.pdf updated after PR 449 )

• ( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update8.pdf updated after PR 461 )

• ( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update9.pdf updated after last updates of PR449 and PR 461 )

• ( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update10.pdf )

• ( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update11.pdf updated after PR 486 )

• ( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update12.pdf after revisiting Issues Language Bindings Summary Chapter RC Review #364, Deprecated chapter RC review #366, and Language Bindings Summary - Appendix A (partial) RC review #375 + PR 497 )

• ( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update13.pdf after Bug in Fortran embiggening MPI_CONVERSION_FN_NULL #432 and PR 493, 498, 499, 500, 501 )

• mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update14.pdf after Language Bindings Chapter RC Review #332 and Introduction Chapter RC Review Issues #349 with PR 504, 506, and 508.

• PR427 https://github.com/mpi-forum/mpi-standard/pull/427 Sorting of the changelog based on
  mpi40-rc-nov-20_Sort-changelog-B.1.2.xlsx
  and correction in chap-datatype that the Azure Pipelines CI works without failures.

• PR434 https://github.com/mpi-forum/mpi-standard/pull/434 Corrections to

  • Annex B. Change-log
  • (Chap 2. Terms: an additional subsection label)
  • Chap 19. Binding: Only Section 19.1.5, see change-log Items 1 and 33
  • Chap 18. Backward Compatibility. see change-log Item 33

• PR439 https://github.com/mpi-forum/mpi-standard/pull/439 Corrections to

  • Chap.8 Topologies
  • Fixes for the MPI 4.0rc in the topology chapter brought up in
    • Issue RC issues in chap-topol found by Dan #343
    • Issue Topologies Chapter RC Review Issues #353

• PR443 https://github.com/mpi-forum/mpi-standard/pull/442 Corrections to

  • Chap.10 The Info Object
  • Fixes for the MPI 4.0rc in the "Info Object" chapter brought up in
    • Issue Language Bindings Summary Chapter RC Review #364
    • and during the Dec. 2020 meeting

• PR442 https://github.com/mpi-forum/mpi-standard/pull/442 Corrections to

  • Chap.16 Deprecated Interfaces
  • Fixes for the MPI 4.0rc in the "Deprecated Interfaces" chapter brought up in
    • Issue Deprecated chapter RC review #366
    • and during the Dec. 2020 meeting

• PR441 https://github.com/mpi-forum/mpi-standard/pull/441 Corrections to

  • Chap.17 Removed Interfaces
  • Fixes for the MPI 4.0rc in the "Removed Interfaces" chapter brought up in
    • Issue Removed interfaces chapter RC review #367
    • and during the Dec. 2020 meeting

• PR444 https://github.com/mpi-forum/mpi-standard/pull/444 Corrections to

  • Annex A. Language Binding Summaries
  • Fixes for the MPI 4.0rc in the "Language Binding Summaries" annex A brought up in
    • Issue Changes for RC 40 as discussed at the MPI forum Dec 7-10, 2020 for Chapters 1, 2, 8, 16-18, A, B & others #405 and others
    • and during the Dec. 2020 meeting

• PR449 https://github.com/mpi-forum/mpi-standard/pull/449 Corrections to

  • Callback Prototype Function Index
  • and some smaller ones in other chapters
  • Fixes for the MPI 4.0rc in the "Callback Prototype Function Index" annex A brought up in
    • Issue Changes for RC 40 as discussed at the MPI forum Dec 7-10, 2020 for Chapters 1, 2, 8, 16-18, A, B & others #405 and others
    • and during the Dec. 2020 meeting

• PR461 https://github.com/mpi-forum/mpi-standard/pull/461 Corrections to

  • Change-log
  • and appropriate additions to all Index sections
  • Fixes for the MPI 4.0rc in the "Change-log" annex B brought up in
    • Issue Changes for RC 40 as discussed at the MPI forum Dec 7-10, 2020 for Chapters 1, 2, 8, 16-18, A, B & others #405 and others
    • and during the Dec. 2020 meeting

[RolfRabenseifner]

The "no more 30 character limit for MPI identifiers" is done. See Sections 2.2, 18.1, 19.1.5 page 791 Table 19.1 2B and page 793f 2nd advice to impl., B.1.2 Items 1 and 33, in following pdf:

• mpi-standard-mpi-4-rc_2020-PR432+PR434+PR435.pdf

It also includes all rc-40 updates to the change-log chapter.

Current implementation status is reported in

• mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update2.pdf

[RolfRabenseifner]

All changes by PR 439 are checked and verified in
mpi40-report_Issue405_PR439.pdf
PR 439 includes and superseeds the old PR 332. (PR332 was therefore closed.)

( mpi40-rc-nov-20_rab_Chap-8+16+17+A+B_update3.pdf
is based on the RC40 document and contains comments about the changes of this PR 439 in Chapter 8.
)

[RolfRabenseifner]

All changes by PR 441 (removed.tex) are checked and verified in
mpi40-report_Issue405_PR441.pdf

[RolfRabenseifner]

All changes by PR 442 (deprecated.tex) are checked and verified in
mpi40-report_Issue405_PR442.pdf

[RolfRabenseifner]

All changes by PR 443 (misc-2.tex - the info chapter) are checked and verified in
mpi40-report_Issue405_PR443.pdf

[RolfRabenseifner]

All changes by PR 444 (appLang-Const.tex - the Annex A.1) are checked and verified in
mpi40-report_Issue405_PR444.pdf

[RolfRabenseifner]

All changes by PR 449 (mainly callback references) are checked and verified in
mpi40-report_Issue405_PR449_v02.pdf

[RolfRabenseifner]

All changes by PR 449 (mainly callback references, including a late footnote number correction in terms) are checked and verified in
mpi40-report_Issue405_PR449_v03.pdf

[RolfRabenseifner]

All changes by PR 461 (completing the change-log) are checked and verified in
mpi40-report_Issue405_PR461_v05.pdf

It contains only small additional sentences, see pages 1040-1042. They are the basis for completing the index sections that nearly all new constants, types, callbacks and functions have a reference to the page of their change-log item.

[RolfRabenseifner]

Thank you Wesley for your help to fix the problem with github and binding-2.tex in PR449.
The corrected pdf is now:
mpi40-report_Issue405_PR449_v05.pdf

[RolfRabenseifner]

All ``_c'' in \code{...} - latest pdf of PR499:
mpi40-report_Issue405_PR499_v09.pdf

[RolfRabenseifner]

Latest pdf for PR493: mpi40-report_Issue405_PR493_v04.pdf

[wgropp]

Is there a PR for any remaining issues? If not, can this be closed?

[RolfRabenseifner]

Is there a PR for any remaining issues? If not, can this be closed?

@wgropp I'm not sure, because https://github.com/mpi-forum/mpi-standard/pull/476 did not show a pdf. Therefore, I could not check whether the MPI_Xxxx declarations in A.3 and A.4 are now again with hyperrefs to their main (i.e. first underlined) page in the Declarations Index.
Therefore the checkbox

• A.3, A.4, A.5: Missing Hyperrefs to the primary locations of the functions and types (see also comments in PR413 and Issue Include Hyperlinks in Language Bindings #422)
  for the function names, see Issue Include Hyperlinks in Language Bindings #422 and PR476
  • for all type declarations

is still not checked.
This is the last remaining checkbox before I can close this issue. (Independent of that, I still need to update my report of changes.)

[wgropp]

I believe these links are present; I see them for both the functions and MPI types.

[RolfRabenseifner]

All pull requests are merged. Therefore, I close this issue 405.

[RolfRabenseifner]

Reopened for the PRs from the Feb 2021 meeting, see first part of the description of this issue.

[wesbland]

@RolfRabenseifner I believe all of the tasks remaining in this issue have moved into their own issues so I'm going to close this. If that's not true, can you create issues for anything left?