Use _create_project(), not create_project() for create_offline_model_evaluation_project

Fixing a typo

Author: Val Brodsky

libs/labelbox/src/labelbox/client.py

@@ -840,7 +840,7 @@ def create_offline_model_evaluation_project(self, **kwargs) -> Project:
         kwargs.pop("append_to_existing_dataset", None)
         kwargs.pop("data_row_count", None)
 
-        return self.create_project(**kwargs)
+        return self._create_project(**kwargs)
 
     def _create_project(self, **kwargs) -> Project:
         auto_audit_percentage = kwargs.get("auto_audit_percentage")

libs/labelbox/src/labelbox/exceptions.py

@@ -153,6 +153,11 @@ class ProcessingWaitTimeout(Exception):
 
 
 def _error_message_for_unparsed_graphql_error(error_string: str) -> str:
+    """
+    Since our client only parses certain graphql errors, this function is used to
+    extract the error message from the error string when the error is not
+    parsed by the client.
+    """
     # Regex to find the message content
     pattern = r"'message': '([^']+)'"
     # Search for the pattern in the error string

libs/labelbox/src/labelbox/schema/project.py

@@ -1272,8 +1272,6 @@ def add_model_config(self, model_config_id: str) -> str:
         try:
             result = self.client.execute(query, params)
         except LabelboxError as e:
-            # unfortunately, this is the type of errors our client does not deal with and so the error message is not in the same format as the other errors
-            # needs custom parsing
             error_content = _error_message_for_unparsed_graphql_error(e.message)
             raise LabelboxError(message=error_content) from e
 
@@ -1305,12 +1303,16 @@ def delete_project_model_config(self, project_model_config_id: str) -> bool:
         return result["deleteProjectModelConfig"]["success"]
 
     def set_project_model_setup_complete(self) -> bool:
-        """ Checks if the model setup is complete for this project.
+        """
+        Sets the model setup is complete for this project.
+        Once the project is marked as "setup complete", a user can not add  / modify delete existing project model configs.
 
         Returns:
             bool, indicates if the model setup is complete.
 
         NOTE: This method should only be used for live model evaluation projects.
+            It will throw exception for all other types of projects.
+            User Project is_chat_evaluation() method to check if the project is a live model evaluation project.
         """
         query = """mutation SetProjectModelSetupCompletePyApi($projectId: ID!) {
             setProjectModelSetupComplete(where: {id: $projectId}, data: {modelSetupComplete: true}) {

libs/labelbox/tests/unit/test_exceptions.py

@@ -0,0 +1,13 @@
+import pytest
+
+from labelbox.exceptions import _error_message_for_unparsed_graphql_error
+
+
+@pytest.mark.parametrize('exception_message, expected_result', [
+    ("Unparsed errors on query execution: [{'message': 'Cannot create model config for project because model setup is complete'",
+     "Cannot create model config for project because model setup is complete"),
+    ("blah blah blah", "Unknown error"),
+])
+def test_client_unparsed_exception_messages(exception_message, expected_result):
+    assert _error_message_for_unparsed_graphql_error(
+        exception_message) == expected_result