RMA, AMO, Reduction edits

openshmem-org / specification

OpenSHMEM Application Programming Interface

http://www.openshmem.org

50 stars 38 forks source link

RMA, AMO, Reduction edits #362

Closed shamisp closed 4 years ago

shamisp commented 4 years ago

Adding macro for subsubsubsection
Moving blocking and non_blocking RMA into one subsubsection
Moving blocking and non_blocking AMO into one subsubsection
Updating AMO introduction to reflect support for non_blocking operations
Reduction paragraph was replaced with subsubsubsection to make the text consistent

Signed-off-by: Pavel Shamis (Pasha) pasharesearch@gmail.com

shamisp commented 4 years ago

Fixing #216

shamisp commented 4 years ago

@nspark this PR touched collectives. Reduction used paragraphs and I made it consistent across chapters. Please let me know if this is okay. Otherwise I can open separate PR just with the reduction change.

shamisp commented 4 years ago

@manjugv can you please review as well. thanks

jdinan commented 4 years ago

Please move this PR to the relevant section committee branch.

shamisp commented 4 years ago

@jdinan How do I do it and what is the relevant branch ? thanks !

jdinan commented 4 years ago

You're the chair, so please create a branch on the current openshmem-org:master head for RMA/AMO/Signal section edits on your fork: shamisp:sec/rma-amo. Post the changes on this PR as a PR against that section committee branch, request reviews from your committee members, and merge into section edits once section committee approves. Thanks!

shamisp commented 4 years ago

@jdinan - got it. thanks.

shamisp commented 4 years ago

As @jdinan requested moved the PR here https://github.com/shamisp/specification/pull/1

I guess I have to ask @jdinan @nspark @manjugv re-approve the new PR. I will close this one and just reference the discussion from the new PR.

jdinan commented 4 years ago

@shamisp IIUC, this PR was superseded by https://github.com/shamisp/specification/pull/1. Can it be closed?