Skip to content
Permalink
Browse files

Deprecate DDL backend (#1649)

Transitioning to NCCL backend for the GPU collective communication library.

Signed-off-by: Nicolas V Castet <nvcastet@us.ibm.com>
  • Loading branch information
nvcastet authored and tgaddair committed Jan 9, 2020
1 parent 9da2b01 commit bb23be3e1eb5fa2a63ce087275ea037c152ed20b
Showing with 6 additions and 0 deletions.
  1. +3 −0 horovod/common/ops/ddl_operations.cc
  2. +3 −0 setup.py
@@ -15,6 +15,7 @@
// =============================================================================

#include "ddl_operations.h"
#include "../logging.h"

namespace horovod {
namespace common {
@@ -103,6 +104,8 @@ Status DDLAllreduce::Execute(std::vector<TensorTableEntry>& entries, const Respo
}

void DDLAllreduce::DDLInit(DDLContext* ddl_context, CUDAContext* cuda_context) {
LOG(WARNING) << "DDL backend has been deprecated. Please, start using the NCCL backend by "
"building Horovod with 'HOROVOD_GPU_ALLREDUCE=NCCL HOROVOD_GPU_BROADCAST=NCCL'.";
auto ddl_options = std::getenv("DDL_OPTIONS");
if (ddl_options == nullptr) {
throw std::logic_error("DDL_OPTIONS env variable needs to be set to use DDL.");
@@ -24,6 +24,7 @@
import textwrap
import traceback
import pipes
import warnings
from copy import deepcopy
from distutils.errors import CompileError, DistutilsError, \
DistutilsPlatformError, LinkError
@@ -612,6 +613,8 @@ def get_common_options(build_ext):
nccl_include_dirs = nccl_lib_dirs = nccl_libs = []

if gpu_allreduce == 'DDL':
warnings.warn("WARN: DDL backend has been deprecated. Please, start using the NCCL backend "
"by building Horovod with 'HOROVOD_GPU_ALLREDUCE=NCCL HOROVOD_GPU_BROADCAST=NCCL'.")
have_ddl = True
ddl_include_dirs, ddl_lib_dirs = get_ddl_dirs(build_ext,
cuda_include_dirs,

0 comments on commit bb23be3

Please sign in to comment.
You can’t perform that action at this time.