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

master: Bull 2020 update of coll/han #8462

Conversation

EmmanuelBRELLE
Copy link
Contributor

@EmmanuelBRELLE EmmanuelBRELLE commented Feb 8, 2021

This completes and fixes current code for coll/Han:

  • a barrier is added
  • a "simple" scatter is added
  • some Doxygen documentation is added

Fix:

  • compilation and cppcheck warnings
  • corner case errors when parsing a rule file

Signed-off-by: Emmanuel Brelle [email protected]

This completes and fixes current code for coll/Han:
- a barrier is added
- a "simple" scatter is added
- some Doxygen documentation is added

Fix:
- compilation and cppcheck warnings
- corner case errors when parsing a rule file

Signed-off-by: Emmanuel Brelle <[email protected]>
@ompiteam-bot
Copy link

Can one of the admins verify this patch?

@EmmanuelBRELLE EmmanuelBRELLE changed the title Bull update of coll/han master: Bull 2020 update of coll/han Feb 8, 2021
@ggouaillardet
Copy link
Contributor

ok to test

@ibm-ompi
Copy link

ibm-ompi commented Feb 8, 2021

The IBM CI (PGI) build failed! Please review the log, linked below.

Gist: https://gist.github.com/039abe769567af0561aabf4505a4ac4c

@ibm-ompi
Copy link

ibm-ompi commented Feb 8, 2021

The IBM CI (XL) build failed! Please review the log, linked below.

Gist: https://gist.github.com/67857e77c32482be7217d179f6dcdd7f

@jjhursey
Copy link
Member

jjhursey commented Feb 8, 2021

IBM Ci had an issue over the weekend. Retrying. bot:ibm:retest

@bosilca bosilca self-requested a review February 8, 2021 15:08
@EmmanuelBRELLE
Copy link
Contributor Author

Is there anything I can do to help the review ? Otherwise, could this PR be merged ?

@ggouaillardet
Copy link
Contributor

since @bosilca already approved this PR, I am merging it

@ggouaillardet ggouaillardet merged commit 7017a07 into open-mpi:master Feb 16, 2021
@EmmanuelBRELLE EmmanuelBRELLE deleted the patch_Han_Bull_2020H2-master-rebased branch February 16, 2021 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants