Skip to content

Conversation

@raafatfeki
Copy link
Contributor

Changes are done for vulcan and dynamic_gen2
1- Instead of using a temporary buffer for non contiguous data to receive data from aggregator, we define a derived datatype to describe the received data layout.
2- Remove send/recv operations with empty data.
3- Make common directives in header file

Signed-off-by: raafatfeki fekiraafat@gmail.com

…ypes

Changes are done for vulcan and dynamic_gen2
1- Instead of using a temporary buffer for non contiguous data to receive data from aggregator, we define a derived datatype to describe the received data layout.
2- Remove send/recv operations with empty data.
3- Make common directives in header file

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

bot:retest

@jsquyres
Copy link
Member

jsquyres commented Feb 4, 2021

You can ignore the readthedocs CI error -- it's only there for a pending PR (#8329). This PR has nothing to do with readthedocs, so whatever error it caused over there is ignoreable (and it may have been a transient RTD error, anyway).

@raafatfeki
Copy link
Contributor Author

Thanks for the clarification.

@raafatfeki raafatfeki merged commit e894138 into open-mpi:master Feb 4, 2021
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.

3 participants