Skip to content

Commit a1d08de

Browse files
Apply ruff/Pylint rule PLR6104
PLR6104 Use `+=`, `*=`, `/=` to perform an augmented assignment directly
1 parent d563577 commit a1d08de

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

src/zarr/codecs/sharding.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -252,7 +252,7 @@ def create_empty(
252252
def __setitem__(self, chunk_coords: ChunkCoords, value: Buffer) -> None:
253253
chunk_start = len(self.buf)
254254
chunk_length = len(value)
255-
self.buf = self.buf + value
255+
self.buf += value
256256
self.index.set_chunk_slice(chunk_coords, slice(chunk_start, chunk_start + chunk_length))
257257

258258
def __delitem__(self, chunk_coords: ChunkCoords) -> None:

src/zarr/core/indexing.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -675,7 +675,7 @@ def check(a: npt.NDArray[Any]) -> Order:
675675
def wraparound_indices(x: npt.NDArray[Any], dim_len: int) -> None:
676676
loc_neg = x < 0
677677
if np.any(loc_neg):
678-
x[loc_neg] = x[loc_neg] + dim_len
678+
x[loc_neg] += dim_len
679679

680680

681681
def boundscheck_indices(x: npt.NDArray[Any], dim_len: int) -> None:
@@ -1000,8 +1000,8 @@ def __init__(
10001000
if stop < 0:
10011001
stop = dim_numchunks + stop
10021002

1003-
start = start * dim_chunk_size
1004-
stop = stop * dim_chunk_size
1003+
start *= dim_chunk_size
1004+
stop *= dim_chunk_size
10051005
slice_ = slice(start, stop)
10061006

10071007
else:

tests/test_codecs/test_sharding.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -229,7 +229,7 @@ def test_sharding_partial_overwrite(
229229
read_data = a[0:10, 0:10, 0:10]
230230
assert np.array_equal(data, read_data)
231231

232-
data = data + 10
232+
data += 10
233233
a[:10, :10, :10] = data
234234
read_data = a[0:10, 0:10, 0:10]
235235
assert np.array_equal(data, read_data)

0 commit comments

Comments
 (0)