Skip to content

Conversation

raafatfeki
Copy link
Contributor

No description provided.

@ibm-ompi
Copy link

ibm-ompi commented Aug 1, 2019

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

Gist: https://gist.github.com/69f79aa394f94e38e7ed010f21a25c92

@ibm-ompi
Copy link

ibm-ompi commented Aug 1, 2019

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

Gist: https://gist.github.com/68fc4b7fa79ded0bc580e146a52036fb

@raafatfeki raafatfeki force-pushed the pr/ompio_coll_write_clean branch from 7b0516b to 1960c37 Compare August 1, 2019 19:54
@ibm-ompi
Copy link

ibm-ompi commented Aug 1, 2019

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

Gist: https://gist.github.com/f97634b544186c2880b9d94901869546

@ibm-ompi
Copy link

ibm-ompi commented Aug 1, 2019

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

Gist: https://gist.github.com/3991ab6b352f6deefb74ec62847024aa

@raafatfeki raafatfeki force-pushed the pr/ompio_coll_write_clean branch from 1960c37 to a568925 Compare August 1, 2019 20:46
Within the shuffle iteration, the aggregators have to set a displacement array needed to receive data from other processes. The array had 1 extra element. We adjust the displacement index to match the number of elements.

Signed-off-by: raafatfeki <fekiraafat@gmail.com>
Within the shuffle iteration, the aggregators have to set a displacement array needed to receive data from other processes. The array had 1 extra element. We adjust the displacement index to match the number of elements.

Signed-off-by: raafatfeki <fekiraafat@gmail.com>
Copy link
Member

@edgargabriel edgargabriel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All tests passed, feel free to merge @raafatfeki

@raafatfeki raafatfeki merged commit 7877743 into open-mpi:master Sep 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants