Skip to content

Commit e51db9e

Browse files
committed
Fixed pylint
1 parent 9881bb1 commit e51db9e

File tree

2 files changed

+10
-5
lines changed

2 files changed

+10
-5
lines changed

src/sagemaker/estimator.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2631,9 +2631,8 @@ def _distribution_configuration(self, distribution):
26312631
distribution_config[self.LAUNCH_SM_DDP_ENV_NAME] = smdataparallel_enabled
26322632
distribution_config[self.INSTANCE_TYPE] = self.instance_type
26332633
if smdataparallel_enabled:
2634-
distribution_config[self.SM_DDP_CUSTOM_MPI_OPTIONS] = smdistributed["dataparallel"].get(
2635-
"custom_mpi_options", ""
2636-
)
2634+
distribution_config[self.SM_DDP_CUSTOM_MPI_OPTIONS] = \
2635+
smdistributed["dataparallel"].get("custom_mpi_options", "")
26372636

26382637
return distribution_config
26392638

tests/unit/test_estimator.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,8 +121,14 @@
121121
DISTRIBUTION_MPI_ENABLED = {
122122
"mpi": {"enabled": True, "custom_mpi_options": "options", "processes_per_host": 2}
123123
}
124-
DISTRIBUTION_SM_DDP_ENABLED = {"smdistributed": {"dataparallel": {"enabled": True,
125-
"custom_mpi_options": "options"}}}
124+
DISTRIBUTION_SM_DDP_ENABLED = {
125+
"smdistributed": {
126+
"dataparallel": {
127+
"enabled": True,
128+
"custom_mpi_options": "options"
129+
}
130+
}
131+
}
126132

127133

128134
class DummyFramework(Framework):

0 commit comments

Comments
 (0)