Apply ruff/Pylint rule PLR6104

PLR6104 Use `+=`, `*=`, `/=` to perform an augmented assignment directly

Author: DimitriPapadopoulos

src/zarr/api/asynchronous.py

@@ -173,7 +173,7 @@ async def consolidate_metadata(
     store_path = await make_store_path(store)
 
     if path is not None:
-        store_path = store_path / path
+        store_path /= path
 
     group = await AsyncGroup.open(store_path, zarr_format=zarr_format, use_consolidated=False)
     group.store_path.store._check_writable()
@@ -294,7 +294,7 @@ async def open(
     store_path = await make_store_path(store, mode=mode, storage_options=storage_options)
 
     if path is not None:
-        store_path = store_path / path
+        store_path /= path
 
     if "shape" not in kwargs and mode in {"a", "w", "w-"}:
         try:
@@ -403,7 +403,7 @@ async def save_array(
     mode = kwargs.pop("mode", None)
     store_path = await make_store_path(store, mode=mode, storage_options=storage_options)
     if path is not None:
-        store_path = store_path / path
+        store_path /= path
     new = await AsyncArray.create(
         store_path,
         zarr_format=zarr_format,
@@ -584,7 +584,7 @@ async def group(
 
     store_path = await make_store_path(store, mode=mode, storage_options=storage_options)
     if path is not None:
-        store_path = store_path / path
+        store_path /= path
 
     if chunk_store is not None:
         warnings.warn("chunk_store is not yet implemented", RuntimeWarning, stacklevel=2)
@@ -699,7 +699,7 @@ async def open_group(
 
     store_path = await make_store_path(store, mode=mode, storage_options=storage_options)
     if path is not None:
-        store_path = store_path / path
+        store_path /= path
 
     if attributes is None:
         attributes = {}
@@ -885,7 +885,7 @@ async def create(
 
     store_path = await make_store_path(store, mode=mode, storage_options=storage_options)
     if path is not None:
-        store_path = store_path / path
+        store_path /= path
 
     return await AsyncArray.create(
         store_path,
@@ -1073,7 +1073,7 @@ async def open_array(
     mode = kwargs.pop("mode", None)
     store_path = await make_store_path(store, mode=mode)
     if path is not None:
-        store_path = store_path / path
+        store_path /= path
 
     zarr_format = _handle_zarr_version_or_format(zarr_version=zarr_version, zarr_format=zarr_format)
 

src/zarr/codecs/sharding.py

@@ -252,7 +252,7 @@ def create_empty(
     def __setitem__(self, chunk_coords: ChunkCoords, value: Buffer) -> None:
         chunk_start = len(self.buf)
         chunk_length = len(value)
-        self.buf = self.buf + value
+        self.buf += value
         self.index.set_chunk_slice(chunk_coords, slice(chunk_start, chunk_start + chunk_length))
 
     def __delitem__(self, chunk_coords: ChunkCoords) -> None:

src/zarr/core/indexing.py

@@ -673,7 +673,7 @@ def check(a: npt.NDArray[Any]) -> Order:
 def wraparound_indices(x: npt.NDArray[Any], dim_len: int) -> None:
     loc_neg = x < 0
     if np.any(loc_neg):
-        x[loc_neg] = x[loc_neg] + dim_len
+        x[loc_neg] += dim_len
 
 
 def boundscheck_indices(x: npt.NDArray[Any], dim_len: int) -> None:
@@ -998,8 +998,8 @@ def __init__(
                 if stop < 0:
                     stop = dim_numchunks + stop
 
-                start = start * dim_chunk_size
-                stop = stop * dim_chunk_size
+                start *= dim_chunk_size
+                stop *= dim_chunk_size
                 slice_ = slice(start, stop)
 
             else:

tests/v3/test_codecs/test_sharding.py

@@ -193,7 +193,7 @@ def test_sharding_partial_overwrite(
     read_data = a[0:10, 0:10, 0:10]
     assert np.array_equal(data, read_data)
 
-    data = data + 10
+    data += 10
     a[:10, :10, :10] = data
     read_data = a[0:10, 0:10, 0:10]
     assert np.array_equal(data, read_data)