lebeg commented on issue #12477: [WIP] Added build with nccl
URL: https://github.com/apache/incubator-mxnet/pull/12477#issuecomment-422367422
 
 
   @marcoabreu that could be an option, but for example `USE_DIST_KVSTORE` is 
not enabled as well, but as I understood has similar functionality. Enabling it 
for all builds would mean it's rather a requirement for the library. Maybe an 
option `USE_NCCL_IF_AVAILABLE` can be introduced, but information needs to be 
propagated to the API whether MXNet was compiled with NCCL to skip the test if 
needed. For now it's a separate build and a separate test - with all 
optimizations that we have on the CI these are almost trivial stages.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to