Move Critique Clients to their own folder (stanford-crfm#1885)

Author: JosselinSomervilleRoberts

src/helm/proxy/clients/auto_client.py

@@ -18,7 +18,7 @@
     TokenizationRequestResult,
 )
 from helm.proxy.clients.client import Client
-from helm.proxy.clients.critique_client import CritiqueClient
+from helm.proxy.critique.critique_client import CritiqueClient
 from helm.proxy.clients.huggingface_model_registry import get_huggingface_model_config
 from helm.proxy.clients.toxicity_classifier_client import ToxicityClassifierClient
 from helm.proxy.retry import NonRetriableException, retry_request
@@ -363,17 +363,17 @@ def get_critique_client(self) -> CritiqueClient:
             return self._critique_client
         critique_type = self.credentials.get("critiqueType")
         if critique_type == "random":
-            from helm.proxy.clients.critique_client import RandomCritiqueClient
+            from helm.proxy.critique.critique_client import RandomCritiqueClient
 
             self._critique_client = RandomCritiqueClient()
         elif critique_type == "mturk":
-            from helm.proxy.clients.mechanical_turk_critique_client import (
+            from helm.proxy.critique.mechanical_turk_critique_client import (
                 MechanicalTurkCritiqueClient,
             )
 
             self._critique_client = MechanicalTurkCritiqueClient()
         elif critique_type == "surgeai":
-            from helm.proxy.clients.surge_ai_critique_client import (
+            from helm.proxy.critique.surge_ai_critique_client import (
                 SurgeAICritiqueClient,
             )
 
@@ -382,15 +382,15 @@ def get_critique_client(self) -> CritiqueClient:
                 raise ValueError("surgeaiApiKey credentials are required for SurgeAICritiqueClient")
             self._critique_client = SurgeAICritiqueClient(surgeai_credentials, self._build_cache_config("surgeai"))
         elif critique_type == "model":
-            from helm.proxy.clients.model_critique_client import ModelCritiqueClient
+            from helm.proxy.critique.model_critique_client import ModelCritiqueClient
 
             model_name: Optional[str] = self.credentials.get("critiqueModelName")
             if model_name is None:
                 raise ValueError("critiqueModelName is required for ModelCritiqueClient")
             client: Client = self._get_client(model_name)
             self._critique_client = ModelCritiqueClient(client, model_name)
         elif critique_type == "scale":
-            from helm.proxy.clients.scale_critique_client import ScaleCritiqueClient
+            from helm.proxy.critique.scale_critique_client import ScaleCritiqueClient
 
             scale_credentials = self.credentials.get("scaleApiKey")
             scale_project = self.credentials.get("scaleProject", None)

src/helm/proxy/clients/critique_client.py src/helm/proxy/critique/critique_client.py

@@ -2,9 +2,9 @@
     CritiqueRequest,
     CritiqueRequestResult,
 )
-from helm.proxy.clients.critique_client import CritiqueClient
-from helm.proxy.clients.mechanical_turk_critique_exporter import export_request
-from helm.proxy.clients.mechanical_turk_critique_importer import import_request_result
+from helm.proxy.critique.critique_client import CritiqueClient
+from helm.proxy.critique.mechanical_turk_critique_exporter import export_request
+from helm.proxy.critique.mechanical_turk_critique_importer import import_request_result
 
 
 class MechanicalTurkCritiqueClient(CritiqueClient):

src/helm/proxy/clients/mechanical_turk_critique_client.py src/helm/proxy/critique/mechanical_turk_critique_client.py

@@ -9,7 +9,7 @@
 from helm.common.critique_request import CritiqueQuestionTemplate, CritiqueRequest, CritiqueTaskTemplate, QuestionType
 from helm.common.general import ensure_directory_exists
 from helm.common.hierarchical_logger import hlog
-from helm.proxy.clients.mechanical_turk_utils import replace_emoji_characters
+from helm.proxy.critique.mechanical_turk_utils import replace_emoji_characters
 
 
 def _indent_to_level(text: str, level: int) -> str:

src/helm/proxy/clients/mechanical_turk_critique_exporter.py src/helm/proxy/critique/mechanical_turk_critique_exporter.py

@@ -13,7 +13,7 @@
     CritiqueRequestResult,
 )
 from helm.common.hierarchical_logger import hlog
-from helm.proxy.clients.mechanical_turk_utils import replace_emoji_characters
+from helm.proxy.critique.mechanical_turk_utils import replace_emoji_characters
 
 # A representation of fields that can be used as a dict key.
 _CritiqueRequestKey = Tuple[Tuple[str, str], ...]

src/helm/proxy/clients/mechanical_turk_critique_importer.py src/helm/proxy/critique/mechanical_turk_critique_importer.py

@@ -10,11 +10,15 @@
     CritiqueTaskTemplate,
 )
 from helm.common.hierarchical_logger import hlog
+from helm.common.optional_dependencies import handle_module_not_found_error
 from helm.common.request import Request, RequestResult, Sequence
 from helm.proxy.clients.client import Client
-from helm.proxy.clients.critique_client import CritiqueClient
+from helm.proxy.critique.critique_client import CritiqueClient
 
-import anthropic
+try:
+    import anthropic
+except ModuleNotFoundError as e:
+    handle_module_not_found_error(e)
 
 
 class CritiqueParseError(Exception):

src/helm/proxy/clients/mechanical_turk_utils.py src/helm/proxy/critique/mechanical_turk_utils.py

@@ -15,7 +15,7 @@
     CritiqueResponse,
 )
 from helm.common.optional_dependencies import handle_module_not_found_error
-from helm.proxy.clients.critique_client import CritiqueClient
+from helm.proxy.critique.critique_client import CritiqueClient
 
 try:
     import scaleapi

src/helm/proxy/clients/model_critique_client.py src/helm/proxy/critique/model_critique_client.py

@@ -12,7 +12,7 @@
 )
 from helm.common.hierarchical_logger import hlog
 from helm.common.optional_dependencies import handle_module_not_found_error
-from helm.proxy.clients.critique_client import CritiqueClient
+from helm.proxy.critique.critique_client import CritiqueClient
 
 try:
     import surge