Skip to content

Commit 2051ca1

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

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
@@ -673,7 +673,7 @@ def check(a: npt.NDArray[Any]) -> Order:
673673
def wraparound_indices(x: npt.NDArray[Any], dim_len: int) -> None:
674674
loc_neg = x < 0
675675
if np.any(loc_neg):
676-
x[loc_neg] = x[loc_neg] + dim_len
676+
x[loc_neg] += dim_len
677677

678678

679679
def boundscheck_indices(x: npt.NDArray[Any], dim_len: int) -> None:
@@ -998,8 +998,8 @@ def __init__(
998998
if stop < 0:
999999
stop = dim_numchunks + stop
10001000

1001-
start = start * dim_chunk_size
1002-
stop = stop * dim_chunk_size
1001+
start *= dim_chunk_size
1002+
stop *= dim_chunk_size
10031003
slice_ = slice(start, stop)
10041004

10051005
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)