Run Ruff

Author: smoia

peakdet/_version.py

@@ -97,8 +97,7 @@ def run_command(commands, args, cwd=None, verbose=False, hide_stderr=False, env=
             print("unable to find command, tried %s" % (commands,))
         return None, None
     stdout = p.communicate()[0].strip()
-    if sys.version_info[0] >= 3:
-        stdout = stdout.decode()
+    stdout = stdout.decode()
     if p.returncode != 0:
         if verbose:
             print("unable to run %s (error)" % dispcmd)
@@ -116,7 +115,7 @@ def versions_from_parentdir(parentdir_prefix, root, verbose):
     """
     rootdirs = []
 
-    for i in range(3):
+    for _i in range(3):
         dirname = os.path.basename(root)
         if dirname.startswith(parentdir_prefix):
             return {
@@ -524,7 +523,7 @@ def get_versions():
         # versionfile_source is the relative path from the top of the source
         # tree (where the .git directory might live) to this file. Invert
         # this to find the root from __file__.
-        for i in cfg.versionfile_source.split("/"):
+        for _i in cfg.versionfile_source.split("/"):
             root = os.path.dirname(root)
     except NameError:
         return {

peakdet/editor.py

@@ -62,7 +62,8 @@ def __init__(self, data):
         reject = functools.partial(self.on_edit, method="reject")
         insert = functools.partial(self.on_edit, method="insert")
 
-        # Check matplotlib version rectprops is deprecated with matplotlib 3.5.0 and then obsolete
+        # Check matplotlib version rectprops is deprecated with matplotlib 3.5.0
+        # and then obsolete
         if Version(matplotlib.__version__) >= Version("3.5.0"):
             property_name = "props"
         else:

peakdet/tests/test_editor.py

@@ -16,12 +16,12 @@ def test_PhysioEditor():
     edits.on_wheel(wheel(10))
 
     # test reject / delete functionality
-    for m in range(2):
+    for _m in range(2):
         edits.on_remove(0, 10, reject=True)
         edits.on_remove(10, 20, reject=False)
 
     # undo delete + reject
-    for m in range(2):
+    for _m in range(2):
         edits.undo()
 
     # test key undo (and undo when history doesn't exist)

peakdet/tests/test_utils.py

@@ -69,7 +69,7 @@ def test_new_physio_like():
     assert new_data.fs == 50
     assert new_data.data.dtype == int
     assert new_data.history == []
-    for k, v in new_data._metadata.items():
+    for _k, v in new_data._metadata.items():
         assert v.size == 0
 
 

pyproject.toml

@@ -42,12 +42,14 @@ convention = 'numpy'
 '*.pyi' = ['E501']
 '__init__.py' = ['F401']
 '_version.py' = ['UP031']
+'conftest.py' = ['D']
+'due.py' = ['D']
 'peakdet/tests/*' = ['D']
 'peakdet/__init__.py' = ['F401', 'D']
 'peakdet/*/__init__.py' = ['F401', 'D']
 'peakdet/_version.py' = ['UP031', 'D']
-'peakdet/conftest.py' = ['D']
-'peakdet/due.py' = ['D']
+'peakdet/modalities.py' = ['D']
+'peakdet/analytics.py' = ['D']
 
 
 # ignore = E126, E402, W503, W401, W811