remote translator client is responiblity of the remote class

* remove remote attributes from base `Translator`
* always set self.client when `_load_translator()` is called

Signed-off-by: Jeffrey Martin <jemartin@nvidia.com>

garak/translators/base.py

@@ -147,8 +147,6 @@ class Translator(Configurable):
     def __init__(self, config_root: dict = {}) -> None:
         self._load_config(config_root=config_root)
 
-        self.translator = None
-        self.nmt_client = None
         self.source_lang, self.target_lang = self.language.split("-")
 
         self._validate_env_var()

garak/translators/remote.py

@@ -61,17 +61,16 @@ def _load_translator(self):
 
         import riva.client
 
-        if self.client is None:
-            auth = riva.client.Auth(
-                None,
-                self.use_ssl,
-                self.uri,
-                [
-                    ("function-id", self.function_id),
-                    ("authorization", "Bearer " + self.api_key),
-                ],
-            )
-            self.client = riva.client.NeuralMachineTranslationClient(auth)
+        auth = riva.client.Auth(
+            None,
+            self.use_ssl,
+            self.uri,
+            [
+                ("function-id", self.function_id),
+                ("authorization", "Bearer " + self.api_key),
+            ],
+        )
+        self.client = riva.client.NeuralMachineTranslationClient(auth)
 
     # TODO: consider adding a backoff here and determining if a connection needs to be re-established
     def _translate(self, text: str) -> str:
@@ -118,8 +117,7 @@ def _load_translator(self):
                 f"Language pair {self.source_lang}-{self.target_lang} is not supported for this translator service."
             )
 
-        if self.client is None:
-            self.client = Translator(self.api_key)
+        self.client = Translator(self.api_key)
 
     def _translate(self, text: str) -> str:
         try: