pylint

Author: benieric

src/sagemaker/modules/distributed.py

@@ -95,7 +95,6 @@ def driver_dir(self) -> str:
         Returns:
             str: Path to directory containing the driver script
         """
-        pass
 
     @property
     @abstractmethod
@@ -108,7 +107,6 @@ def driver_script(self) -> str:
         Returns:
             str: Name of the driver script file
         """
-        pass
 
 
 class Torchrun(DistributedConfig):

src/sagemaker/modules/train/container_drivers/__init__.py

@@ -11,4 +11,4 @@
 # ANY KIND, either express or implied. See the License for the specific
 # language governing permissions and limitations under the License.
 """Sagemaker modules container drivers directory."""
-from __future__ import absolute_import
+from __future__ import absolute_import

src/sagemaker/modules/train/container_drivers/common/__init__.py

@@ -11,4 +11,4 @@
 # ANY KIND, either express or implied. See the License for the specific
 # language governing permissions and limitations under the License.
 """Sagemaker modules container drivers - common directory."""
-from __future__ import absolute_import
+from __future__ import absolute_import

src/sagemaker/modules/train/container_drivers/drivers/__init__.py

@@ -11,4 +11,4 @@
 # ANY KIND, either express or implied. See the License for the specific
 # language governing permissions and limitations under the License.
 """Sagemaker modules container drivers - drivers directory."""
-from __future__ import absolute_import
+from __future__ import absolute_import

src/sagemaker/modules/train/container_drivers/drivers/basic_script_driver.py

@@ -23,7 +23,7 @@
 
 sys.path.insert(0, str(Path(__file__).parent.parent))
 
-from common.utils import (  # noqa: E402
+from common.utils import (  # noqa: E402 # pylint: disable=C0413,E0611
     logger,
     get_python_executable,
     execute_commands,

src/sagemaker/modules/train/container_drivers/drivers/mpi_driver.py

@@ -29,7 +29,7 @@
 
 
 sys.path.insert(0, str(Path(__file__).parent.parent))
-from common.utils import (  # noqa: E402
+from common.utils import (  # noqa: E402 # pylint: disable=C0413,E0611
     logger,
     hyperparameters_to_cli_args,
     get_process_count,

src/sagemaker/modules/train/container_drivers/drivers/mpi_utils.py

@@ -17,14 +17,15 @@
 import sys
 import subprocess
 import time
-import paramiko
 
 from pathlib import Path
 from typing import List
 
+import paramiko
+
 sys.path.insert(0, str(Path(__file__).parent.parent))
 
-from common.utils import (  # noqa: E402
+from common.utils import (  # noqa: E402 # pylint: disable=C0413,E0611
     SM_EFA_NCCL_INSTANCES,
     SM_EFA_RDMA_INSTANCES,
     get_python_executable,

src/sagemaker/modules/train/container_drivers/drivers/torchrun_driver.py

@@ -22,7 +22,7 @@
 
 sys.path.insert(0, str(Path(__file__).parent.parent))
 
-from common.utils import (  # noqa: E402
+from common.utils import (  # noqa: E402 # pylint: disable=C0413,E0611
     logger,
     hyperparameters_to_cli_args,
     get_process_count,

src/sagemaker/modules/train/container_drivers/scripts/environment.py

@@ -24,7 +24,7 @@
 
 sys.path.insert(0, str(Path(__file__).parent.parent))
 
-from common.utils import (  # noqa: E402
+from common.utils import (  # noqa: E402 # pylint: disable=C0413,E0611
     safe_serialize,
     safe_deserialize,
     read_distributed_json,