place config in helper function

Author: mshannon-sil

silnlp/common/environment.py

@@ -174,12 +174,7 @@ def copy_pt_project_from_bucket(self, name: Union[str, Path], patterns: Union[st
             raise Exception(
                 f"No paratext project name is given.  Data still in the cache directory of {self.pt_projects_dir}"
             )
-        config = Config(
-            s3={"addressing_style": "path"},
-            retries={"mode": "adaptive", "max_attempts": 10},
-            connect_timeout=600,
-            read_timeout=600,
-        )
+        config = generate_s3_config()
         s3 = boto3.resource("s3", config=config)
         data_bucket = s3.Bucket(str(self.data_dir).strip("\\/"))
         len_silnlp_path = len(pt_projects_path)
@@ -213,12 +208,7 @@ def copy_experiment_from_bucket(self, name: Union[str, Path], patterns: Union[st
             raise Exception(
                 f"No experiment name is given.  Data still in the cache directory of {self.mt_experiments_dir}"
             )
-        config = Config(
-            s3={"addressing_style": "path"},
-            retries={"mode": "adaptive", "max_attempts": 10},
-            connect_timeout=600,
-            read_timeout=600,
-        )
+        config = generate_s3_config()
         s3 = boto3.resource("s3", config=config)
         data_bucket = s3.Bucket(str(self.data_dir).strip("\\/"))
         len_silnlp_path = len(experiments_path)
@@ -251,12 +241,7 @@ def copy_experiment_to_bucket(self, name: str, patterns: Union[str, Sequence[str
                 f"No experiment name is given.  Data still in the temp directory of {self.mt_experiments_dir}"
             )
         experiment_path = str(self.mt_dir.relative_to(self.data_dir) / "experiments") + "/"
-        config = Config(
-            s3={"addressing_style": "path"},
-            retries={"mode": "adaptive", "max_attempts": 10},
-            connect_timeout=600,
-            read_timeout=600,
-        )
+        config = generate_s3_config()
         s3 = boto3.resource("s3", config=config)
         data_bucket = s3.Bucket(str(self.data_dir).strip("\\/"))
         temp_folder = str(self.mt_experiments_dir / name)
@@ -300,12 +285,7 @@ def download_if_s3_paths(paths: Iterable[S3Path]) -> List[Path]:
             if not s3_setup:
                 temp_root = Path(tempfile.TemporaryDirectory().name)
                 temp_root.mkdir()
-                config = Config(
-                    s3={"addressing_style": "path"},
-                    retries={"mode": "adaptive", "max_attempts": 10},
-                    connect_timeout=600,
-                    read_timeout=600,
-                )
+                config = generate_s3_config()
                 s3 = boto3.resource("s3", config=config)
                 data_bucket = s3.Bucket(str(SIL_NLP_ENV.data_dir).strip("\\/"))
                 s3_setup = True
@@ -357,4 +337,14 @@ def get_env_path(name: str, default: str = ".") -> str:
     return path
 
 
+def generate_s3_config() -> Config:
+    s3_config = Config(
+        s3={"addressing_style": "path"},
+        retries={"mode": "adaptive", "max_attempts": 10},
+        connect_timeout=600,
+        read_timeout=600,
+    )
+    return s3_config
+
+
 SIL_NLP_ENV = SilNlpEnv()