Skip to content

Commit

Permalink
Remove dependency on Fairring
Browse files Browse the repository at this point in the history
Summary: We intend to remove/archive Fairring, since it hasn't been developed in 2 years. For that we need to first clean up the projects that depend on it.

Reviewed By: bottler

Differential Revision: D60222759

fbshipit-source-id: 182c9496644e0408a582f916aae7f5f3391b0de1
  • Loading branch information
lw authored and facebook-github-bot committed Jul 25, 2024
1 parent fceef73 commit c9e2255
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 14 deletions.
2 changes: 1 addition & 1 deletion et_replay/comm/pytorch_backend_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
logger = logging.getLogger(__name__)

supportedDevices = ["cpu", "cuda", "rocm", "tpu"]
supportedC10dBackends = ["nccl", "gloo", "mpi", "ucc", "xla", "fairring"]
supportedC10dBackends = ["nccl", "gloo", "mpi", "ucc", "xla"]
supportedCollectives = [
"reduce",
"all_reduce",
Expand Down
6 changes: 0 additions & 6 deletions et_replay/comm/pytorch_dist_backend.py
Original file line number Diff line number Diff line change
Expand Up @@ -1008,12 +1008,6 @@ def __init__(self, bootstrap_info, commsParams):
raise RuntimeError("Unable to import initialize_ucc_plugin")
else:
initialize_ucc_plugin(backend)
# Import Fairring
if backend == "fairring":
try:
import fairring # noqa
except ImportError:
raise RuntimeError("Unable to import Fairring")

def get_new_pg(self, group_ranks, backend):
if self.use_ext_dist:
Expand Down
2 changes: 1 addition & 1 deletion train/comms/pt/pytorch_backend_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
logger = logging.getLogger(__name__)

supportedDevices = ["cpu", "cuda", "rocm", "tpu"]
supportedC10dBackends = ["nccl", "gloo", "mpi", "ucc", "xla", "fairring"]
supportedC10dBackends = ["nccl", "gloo", "mpi", "ucc", "xla"]
supportedCollectives = [
"reduce",
"all_reduce",
Expand Down
6 changes: 0 additions & 6 deletions train/comms/pt/pytorch_dist_backend.py
Original file line number Diff line number Diff line change
Expand Up @@ -1010,12 +1010,6 @@ def __init__(self, bootstrap_info, commsParams):
raise RuntimeError("Unable to import initialize_ucc_plugin")
else:
initialize_ucc_plugin(backend)
# Import Fairring
if backend == "fairring":
try:
import fairring # noqa
except ImportError:
raise RuntimeError("Unable to import Fairring")

def get_new_pg(self, group_ranks, backend):
if self.use_ext_dist:
Expand Down

0 comments on commit c9e2255

Please sign in to comment.