Merge branch 'main' into main

Author: pavoljuhas

.github/workflows/ci-daily.yml

@@ -71,7 +71,8 @@ jobs:
     strategy:
       matrix:
         python-version: ['3.10', '3.11']
-    runs-on: macos-latest
+    # TODO(#6577): upgrade to macos-latest when it runs Python 3.10
+    runs-on: macos-13
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5

.github/workflows/ci.yml

@@ -260,7 +260,8 @@ jobs:
     strategy:
       matrix:
         python-version: [ '3.10', '3.11' ]
-    runs-on: macos-latest
+    # TODO(#6577): upgrade to macos-latest when it runs Python 3.10
+    runs-on: macos-13
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5

cirq-core/cirq/study/sweeps.py

@@ -236,8 +236,6 @@ def keys(self) -> List['cirq.TParamKey']:
         return sum((factor.keys for factor in self.factors), [])
 
     def __len__(self) -> int:
-        if not self.factors:
-            return 0
         length = 1
         for factor in self.factors:
             length *= len(factor)

cirq-core/cirq/study/sweeps_test.py

@@ -142,6 +142,11 @@ def test_product():
     assert _values(sweep, 'b') == [4, 5, 6, 7, 4, 5, 6, 7, 4, 5, 6, 7]
 
 
+def test_empty_product():
+    sweep = cirq.Product()
+    assert len(sweep) == len(list(sweep)) == 1
+
+
 def test_slice_access_error():
     sweep = cirq.Points('a', [1, 2, 3])
     with pytest.raises(TypeError, match='<class \'str\'>'):