Merge branch 'feature/job-decorator' into cd-results

amazon-braket · Oct 13, 2023 · 6a4dd1e · 6a4dd1e
2 parents 6a2600e + 0efbd82
commit 6a4dd1e
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 51 deletions.
diff --git a/examples/job.py b/examples/job.py
diff --git a/src/braket/jobs/hybrid_job.py b/src/braket/jobs/hybrid_job.py
@@ -306,15 +306,14 @@ def _sanitize(hyperparameter: Any) -> str:
 
     sanitized = (
         string_hp
-        # replace newlines with spaces
+        # replace forbidden characters with close matches
         .replace("\n", " ")
-        # replace forbidden characters with "?"
         .replace("$", "?")
-        .replace("(", "?")
-        .replace("&", "?")
-        .replace("`", "?")
+        .replace("(", "{")
+        .replace("&", "+")
+        .replace("`", "'")
         # not technically forbidden, but to avoid mismatched parens
-        .replace(")", "?")
+        .replace(")", "}")
     )
 
     # max allowed length for a hyperparameter is 2500

diff --git a/test/integ_tests/test_create_quantum_job.py b/test/integ_tests/test_create_quantum_job.py
@@ -220,7 +220,7 @@ def decorator_job(a, b: int, c=0, d: float = 1.0, **extras):
         with open(hp_file, "r") as f:
             hyperparameters = json.load(f)
         assert hyperparameters == {
-            "a": "MyClass(value)",
+            "a": "MyClass{value}",
             "b": "2",
             "c": "0",
             "d": "5",

diff --git a/test/unit_tests/braket/jobs/test_hybrid_job.py b/test/unit_tests/braket/jobs/test_hybrid_job.py
@@ -481,7 +481,7 @@ def my_job():
         ),
         (
             "with weird chars: (&$`)",
-            "with weird chars: ?????",
+            "with weird chars: {+?'}",
         ),
         (
             "?" * 2600,