Address PR feedback

Author: DarthMax

graphdatascience/procedure_surface/api/catalog/relationships_endpoints.py

@@ -223,7 +223,7 @@ def collapse_path(
         username: str | None = None,
     ) -> CollapsePathResult:
         """
-        Collapse a path in the graph catalog.
+        Collapse each existing path in the graph into a single relationship.
 
         Parameters
         ----------
@@ -237,9 +237,9 @@ def collapse_path(
         allow_self_loops : bool, default=False
             Whether nodes in the graph can have relationships where start and end nodes are the same.
         concurrency : int | None, default=None:
-            Number of concurrent threads to use. Defaults to None.
+            Number of concurrent threads to use.
         job_id : str | None, default=None
-            Unique identifier for the job associated with the graph generation.
+            Unique identifier for the job associated with the computation.
         sudo : bool | None, default=None
             Override memory estimation limits
         log_progress : bool | None, default=None

graphdatascience/procedure_surface/arrow/config_arrow_endpoints.py

@@ -35,6 +35,7 @@ def set(
         value: Any,
         username: str | None = None,
     ) -> None:
+        key = ConfigConverter.convert_to_camel_case(key)
         deserialize(self._arrow_client.do_action_with_retry("v2/defaults.set", {key: value}))
 
     def list(
@@ -65,6 +66,7 @@ def set(
         value: Any,
         username: str | None = None,
     ) -> None:
+        key = ConfigConverter.convert_to_camel_case(key)
         deserialize(self._arrow_client.do_action_with_retry("v2/limits.set", {key: value}))
 
     def list(

graphdatascience/procedure_surface/cypher/config_cypher_endpoints.py

@@ -35,6 +35,8 @@ def set(
         value: Any,
         username: str | None = None,
     ) -> None:
+        key = ConfigConverter.convert_to_camel_case(key)
+
         params = {
             "key": key,
             "value": value,
@@ -75,6 +77,8 @@ def set(
         value: Any,
         username: str | None = None,
     ) -> None:
+        key = ConfigConverter.convert_to_camel_case(key)
+
         params = {
             "key": key,
             "value": value,

graphdatascience/procedure_surface/utils/config_converter.py

@@ -13,7 +13,7 @@ def convert_to_gds_config(**kwargs: Any | None) -> dict[str, Any]:
         return config
 
     @staticmethod
-    def _convert_to_camel_case(name: str) -> str:
+    def convert_to_camel_case(name: str) -> str:
         """Convert a snake_case string to camelCase."""
         parts = name.split("_")
 
@@ -29,7 +29,7 @@ def _process_dict_values(input_dict: dict[str, Any]) -> dict[str, Any]:
         result = {}
         for key, value in input_dict.items():
             if value is not None:
-                camel_key = ConfigConverter._convert_to_camel_case(key)
+                camel_key = ConfigConverter.convert_to_camel_case(key)
                 # Recursively process nested dictionaries
                 if isinstance(value, dict):
                     result[camel_key] = ConfigConverter._process_dict_values(value)