Use zip() or enumerate() instead of range(len())

blosc2/core.py

@@ -1275,23 +1275,23 @@ def compute_chunks_blocks(
             blocks = [blocks]
         if len(blocks) != len(shape):
             raise ValueError("blocks should have the same length than shape")
-        for i in range(len(blocks)):
-            if blocks[i] == 0:
+        for block, dim in zip(blocks, shape):
+            if block == 0:
                 raise ValueError("blocks cannot contain 0 dimension")
-            if shape[i] == 1 and blocks[i] > shape[i]:
+            if dim == 1 and block > dim:
                 raise ValueError("blocks cannot be greater than shape if it is 1")
     if chunks:
         if not isinstance(chunks, tuple | list):
             chunks = [chunks]
         if len(chunks) != len(shape):
             raise ValueError("chunks should have the same length than shape")
-        for i in range(len(chunks)):
-            if shape[i] == 1 and chunks[i] > shape[i]:
+        for chunk, dim in zip(chunks, shape):
+            if dim == 1 and chunk > dim:
                 raise ValueError("chunks cannot be greater than shape if it is 1")
 
     if chunks is not None and blocks is not None:
-        for i in range(len(blocks)):
-            if blocks[i] > chunks[i]:
+        for block, chunk in zip(blocks, chunks):
+            if block > chunk:
                 raise ValueError("blocks cannot be greater than chunks")
         return chunks, blocks
 

tests/ndarray/test_auto_parts.py

@@ -43,9 +43,9 @@ def test_compute_chunks_blocks(clevel, codec, shape: tuple, dtype):
     else:
         chunks, blocks = blosc2.compute_chunks_blocks(shape, **cparams)
     # print(chunks, blocks)
-    for i in range(len(shape)):
-        assert shape[i] >= chunks[i]
-        assert chunks[i] >= blocks[i]
+    for dim, chunk, block in zip(shape, chunks, blocks):
+        assert dim >= chunk
+        assert chunk >= block
 
 
 @pytest.mark.parametrize(
@@ -65,9 +65,9 @@ def test_compute_chunks_blocks(clevel, codec, shape: tuple, dtype):
 def test_compute_chunks(shape: tuple, blocks: tuple):
     chunks, blocks = blosc2.compute_chunks_blocks(shape, blocks=blocks)
     # print(chunks, blocks)
-    for i in range(len(shape)):
-        assert shape[i] >= chunks[i]
-        assert chunks[i] >= blocks[i]
+    for dim, chunk, block in zip(shape, chunks, blocks):
+        assert dim >= chunk
+        assert chunk >= block
 
 
 # Invalid blocks

tests/ndarray/test_lazyexpr.py

@@ -452,8 +452,8 @@ def test_save():
     a4 = blosc2.asarray(na4)
     ops = [a1, a2, a3, a4]
     op_urlpaths = ["a1.b2nd", "a2.b2nd", "a3.b2nd", "a4.b2nd"]
-    for i in range(len(op_urlpaths)):
-        ops[i] = ops[i].copy(urlpath=op_urlpaths[i], mode="w")
+    for i, urlpath in enumerate(op_urlpaths):
+        ops[i] = ops[i].copy(urlpath=urlpath, mode="w")
 
     # Construct the lazy expression with the on-disk operands
     da1, da2, da3, da4 = ops