February 2015 - mpiwg-coll - mailman3.cels.anl.gov

Re: [mpiwg-coll] Review implementation of #419+420 in chapter 7 - take 2
by Balaji, Pavan 22 Feb '15

22 Feb '15

The changes look good to me. -- Pavan > On Feb 21, 2015, at 2:56 PM, Torsten Hoefler <htor(a)inf.ethz.ch> wrote: > > Dear Chapter Committee (and anybody interested), > > Thanks to all for your review! Puri found some formatting inconsistencies (only missing spaces and line breaks) that took a while to resolve (thanks to Bill and sorry for the delay). > > While resolving them, Bill found some simple ticket-0 changes that I applied based on his comments: > > - removed the 20-year old outdated reference to EXPRESS > - slight wording change in advice to users on MPI_WEIGHTS_EMPTY > - slight wording change in example of graph_neighbors > - slight wording change in first sentence of 7.5.6 > > We can easily back them out (r2007) if we decide that we don't want them. They are rather uncontroversial though. The separate diff of these is at [1]. > > The complete diff (including all changes) is at [2] and the latest version of the chapter is at [3]. Please review this version again, hopefully a last time. > > Thanks & All the Best, > Torsten > > [1]: http://htor.inf.ethz.ch/sec/chap-topol-diff-r2003.diff > [2]: http://htor.inf.ethz.ch/sec/chap-topol-diff-r1806.diff > [3]: http://htor.inf.ethz.ch/sec/chap-topol-mpi-3_1.pdf > > -- > ### qreharg rug ebs fv crryF ------------- http://www.unixer.de/ ----- > Torsten Hoefler | Assistant Professor > Dept. of Computer Science | ETH Zürich > Universitätsstrasse 6 | Zurich-8092, Switzerland > CAB E 64.1 | Phone: +41 76 309 79 29 -- Pavan Balaji ✉️ http://www.mcs.anl.gov/~balaji

1 0

Review implementation of #419+420 in chapter 7 - take 2
by Torsten Hoefler 21 Feb '15

21 Feb '15

Dear Chapter Committee (and anybody interested), Thanks to all for your review! Puri found some formatting inconsistencies (only missing spaces and line breaks) that took a while to resolve (thanks to Bill and sorry for the delay). While resolving them, Bill found some simple ticket-0 changes that I applied based on his comments: - removed the 20-year old outdated reference to EXPRESS - slight wording change in advice to users on MPI_WEIGHTS_EMPTY - slight wording change in example of graph_neighbors - slight wording change in first sentence of 7.5.6 We can easily back them out (r2007) if we decide that we don't want them. They are rather uncontroversial though. The separate diff of these is at [1]. The complete diff (including all changes) is at [2] and the latest version of the chapter is at [3]. Please review this version again, hopefully a last time. Thanks & All the Best, Torsten [1]: http://htor.inf.ethz.ch/sec/chap-topol-diff-r2003.diff [2]: http://htor.inf.ethz.ch/sec/chap-topol-diff-r1806.diff [3]: http://htor.inf.ethz.ch/sec/chap-topol-mpi-3_1.pdf -- ### qreharg rug ebs fv crryF ------------- http://www.unixer.de/ ----- Torsten Hoefler | Assistant Professor Dept. of Computer Science | ETH Zürich Universitätsstrasse 6 | Zurich-8092, Switzerland CAB E 64.1 | Phone: +41 76 309 79 29

1 0

Re: [mpiwg-coll] Review implementation of #419+420 in chapter 7
by Daniel Holmes 11 Feb '15

11 Feb '15

Hi Torsten, I have reviewed the changes for #420 - all looks good to me. My only comments are that the diff shows changes to most of the lines of the example, even lines that have not apparently changed, which makes picking out the actual changes a little more difficult than necessary. Also, the alignment of the first set numbers in the "[s/r]displs = ( xxx + ..." lines is off by one character for the single digit numbers. Minor quibbles that do not need to be fixed unless you have lots of free time! Cheers, Dan. On 07/02/2015 21:35, Torsten Hoefler wrote: > Dear Chapter Committee (and anybody interested), > > Could you please review the implementation of tickets 419 and 420 [1] in > the new pdf [2] or the diff [3] (which includes some more ticket-0 > changes) before February 16. > > I don't have Chulho Kim's email address but I hope he's subscribed here. > > Martin, can you please update the table on > http://meetings.mpi-forum.org/MPI_3.1_main_page.php . I rely on this > table that there is nothing else missing in chapters 5 or 7. Please let > me know if there is still something open. > > Thanks & Best, > Torsten > > [1]: https://svn.mpi-forum.org/trac/mpi-forum-web/ticket/419 > https://svn.mpi-forum.org/trac/mpi-forum-web/ticket/420 > [2]: http://htor.inf.ethz.ch/sec/chap-topol-mpi-3_1.pdf > [3]: http://htor.inf.ethz.ch/sec/chap-topol-diff-r1806.diff > -- Dan Holmes Applications Consultant in HPC Research EPCC, The University of Edinburgh James Clerk Maxwell Building The Kings Buildings Peter Guthrie Tait Road Edinburgh EH9 3FD T: +44(0)131 651 3465 E: dholmes(a)epcc.ed.ac.uk *Please consider the environment before printing this email.* The University of Edinburgh is a charitable body, registered in Scotland, with registration number SC005336.

1 0

Re: [mpiwg-coll] Review implementation of #419+420 in chapter 7
by Jeff Hammond 07 Feb '15

07 Feb '15

I BCC'd Chulho's Lenovo address just in case. Jeff Sent from my iPhone > On Feb 7, 2015, at 1:35 PM, Torsten Hoefler <htor(a)inf.ethz.ch> wrote: > > Dear Chapter Committee (and anybody interested), > > Could you please review the implementation of tickets 419 and 420 [1] in > the new pdf [2] or the diff [3] (which includes some more ticket-0 > changes) before February 16. > > I don't have Chulho Kim's email address but I hope he's subscribed here. > > Martin, can you please update the table on > http://meetings.mpi-forum.org/MPI_3.1_main_page.php . I rely on this > table that there is nothing else missing in chapters 5 or 7. Please let > me know if there is still something open. > > Thanks & Best, > Torsten > > [1]: https://svn.mpi-forum.org/trac/mpi-forum-web/ticket/419 > https://svn.mpi-forum.org/trac/mpi-forum-web/ticket/420 > [2]: http://htor.inf.ethz.ch/sec/chap-topol-mpi-3_1.pdf > [3]: http://htor.inf.ethz.ch/sec/chap-topol-diff-r1806.diff > > -- > ### qreharg rug ebs fv crryF ------------- http://www.unixer.de/ ----- > Torsten Hoefler | Assistant Professor > Dept. of Computer Science | ETH Zürich > Universitätsstrasse 6 | Zurich-8092, Switzerland > CAB E 64.1 | Phone: +41 76 309 79 29 > > > _______________________________________________ > mpiwg-coll mailing list > mpiwg-coll(a)lists.mpi-forum.org > http://lists.mpi-forum.org/mailman/listinfo.cgi/mpiwg-coll

1 0

Review implementation of #419+420 in chapter 7
by Torsten Hoefler 07 Feb '15

07 Feb '15

Dear Chapter Committee (and anybody interested), Could you please review the implementation of tickets 419 and 420 [1] in the new pdf [2] or the diff [3] (which includes some more ticket-0 changes) before February 16. I don't have Chulho Kim's email address but I hope he's subscribed here. Martin, can you please update the table on http://meetings.mpi-forum.org/MPI_3.1_main_page.php . I rely on this table that there is nothing else missing in chapters 5 or 7. Please let me know if there is still something open. Thanks & Best, Torsten [1]: https://svn.mpi-forum.org/trac/mpi-forum-web/ticket/419 https://svn.mpi-forum.org/trac/mpi-forum-web/ticket/420 [2]: http://htor.inf.ethz.ch/sec/chap-topol-mpi-3_1.pdf [3]: http://htor.inf.ethz.ch/sec/chap-topol-diff-r1806.diff -- ### qreharg rug ebs fv crryF ------------- http://www.unixer.de/ ----- Torsten Hoefler | Assistant Professor Dept. of Computer Science | ETH Zürich Universitätsstrasse 6 | Zurich-8092, Switzerland CAB E 64.1 | Phone: +41 76 309 79 29

1 0