Fix calibration API timestamp (#3931)

- today() does not have hours, minutes, seconds - Switch to use now().
quantumlib · Mar 19, 2021 · 0baadeb · 0baadeb
1 parent f49ab35
commit 0baadeb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/cirq/google/engine/engine.py b/cirq/google/engine/engine.py
@@ -65,7 +65,7 @@ class ProtoVersion(enum.Enum):
 def _make_random_id(prefix: str, length: int = 16):
     random_digits = [random.choice(string.ascii_uppercase + string.digits) for _ in range(length)]
     suffix = ''.join(random_digits)
-    suffix += datetime.date.today().strftime('%y%m%d-%H%M%S')
+    suffix += datetime.datetime.now().strftime('%y%m%d-%H%M%S')
     return f'{prefix}{suffix}'
 
 

diff --git a/cirq/google/engine/engine_test.py b/cirq/google/engine/engine_test.py
@@ -272,6 +272,8 @@ def test_make_random_id():
     random_id = cg.engine.engine._make_random_id('prefix-')
     time.sleep(1)
     random_id2 = cg.engine.engine._make_random_id('prefix-')
+    # Verify %H%M%S is populated
+    assert random_id[-7:] != '-000000' or random_id2[-7:] != '-000000'
     # Verify program id generate distinct even if random is seeded
     assert random_id != random_id2