Fix alerts detected by newer ruff 0.1.7

Author: DimitriPapadopoulos

bench/pack_tensor.py

@@ -48,7 +48,7 @@
     ctoc = time.time()
     tc = (ctoc - ctic) / NREP
     print(
-        "  Time for tensorflow (tf.io.serialize):\t[:.3f} s ({:.2f} GB/s)) ".format(tc, ((N * 8 / tc) / 2**30)),
+        "  Time for tensorflow (tf.io.serialize):\t{:.3f} s ({:.2f} GB/s)) ".format(tc, ((N * 8 / tc) / 2**30)),
         end="",
     )
     print("\tcr: {:5.1f}x".format(in_.size * in_.dtype.itemsize * 1.0 / len(c)))

blosc2/core.py

@@ -1115,13 +1115,14 @@ def get_chunksize(blocksize, l3_minimum=2**21, l3_maximum=2**25):
         # value only when it is an actual int.
         # Also, sometimes cpuinfo does not return a correct L3 size;
         # so in general, enforcing L3 > L2 is a good sanity check.
-        l2_cache_size = cpu_info.get("l2_cache_size", "Not found")
-        if type(l3_cache_size) is int and type(l2_cache_size) is int and l3_cache_size > l2_cache_size:
-            chunksize = l3_cache_size
+        if isinstance(l3_cache_size, int):
+            l2_cache_size = cpu_info.get("l2_cache_size", "Not found")
+            if isinstance(l2_cache_size, int) and l3_cache_size > l2_cache_size:
+                chunksize = l3_cache_size
     else:
         # Chunksize should be at least the size of L2
         l2_cache_size = cpu_info.get("l2_cache_size", "Not found")
-        if type(l2_cache_size) is int and l2_cache_size > chunksize:
+        if isinstance(l2_cache_size, int) and l2_cache_size > chunksize:
             chunksize = l2_cache_size
 
     # Ensure a minimum size

blosc2/info.py

@@ -22,7 +22,7 @@ def info_text_report(items: list) -> str:
     max_key_len = max(len(k) for k in keys)
     report = ""
     for k, v in items:
-        if type(v) is dict:
+        if isinstance(v, dict):
             # rich way, this is disabled because it doesn't work well in the notebooks
             # with io.StringIO() as buf:
             #     v_sorted = {k: val for k, val in sorted(v.items())}

blosc2/ndarray.py

@@ -368,9 +368,9 @@ def squeeze(self):
 
 
 def _check_shape(shape):
-    if type(shape) is int:
+    if isinstance(shape, int):
         shape = (shape,)
-    if type(shape) not in (tuple, list):
+    elif not isinstance(shape, (tuple, list)):
         raise ValueError("shape should be a tuple or a list!")
     return shape
 

examples/btune.py

@@ -28,7 +28,7 @@
 
 # Set the Btune configuration to use
 btune_conf = {"tradeoff": 0.3, "perf_mode": blosc2_btune.PerformanceMode.DECOMP}
-blosc2_btune.set_params_defaults(**kwargs)
+blosc2_btune.set_params_defaults()
 
 # Create the SChunk
 data = np.arange(200 * 1000 * nchunks)

tests/test_compress2.py

@@ -109,7 +109,7 @@ def test_raise_error(object, cparams, dparams, gil):
         blosc2.decompress2(c, dst=dest)
 
     dest3 = blosc2.decompress2(c)
-    if type(object) is bytes:
+    if isinstance(object, bytes):
         assert dest3 == object
     else:
         assert dest3 == object.tobytes()

tests/test_decompress.py

@@ -83,13 +83,13 @@ def test_raise_error(object, codec):
         blosc2.decompress(c, dst=dest)
 
     dest3 = blosc2.decompress(c)
-    if type(object) is bytes:
+    if isinstance(object, bytes):
         assert dest3 == object
     else:
         assert dest3 == object.tobytes()
 
     dest4 = blosc2.decompress(c, as_bytearray=True)
-    if type(object) is bytes:
+    if isinstance(object, bytes):
         assert dest4 == object
     else:
         assert dest4 == object.tobytes()

tests/test_pathlib.py

@@ -6,8 +6,6 @@
 # LICENSE file in the root directory of this source tree)
 #######################################################################
 
-import os
-
 import numpy as np
 import pytest
 import pathlib

tests/test_ucodecs.py

@@ -142,11 +142,10 @@ def test_dynamic_ucodecs_error(cparams, dparams):
     chunk_len = 100
     dtype = np.dtype(np.int32)
     nchunks = 1
-    fill_value = 341
     data = np.arange(chunk_len * nchunks, dtype=dtype)
 
     with pytest.raises(RuntimeError):
-        schunk = blosc2.SChunk(
+        _ = blosc2.SChunk(
             chunksize=chunk_len * dtype.itemsize,
             data=data,
             cparams=cparams,