Merge pull request #178 from ewoks-kit/use_ruff

convert to ruff

Author: woutdenolf

.flake8

@@ -80,6 +80,11 @@ jobs:
   # Run linter / checks
   checks:
     uses: ewoks-kit/.github/.github/workflows/python-check.yml@main
+    with:
+      enable-black: "false"
+      enable-flake8: "false"
+      enable-isort: "false"
+      enable-ruff: "true"
 
   # Build documentation
   docs:

.github/workflows/test.yml

@@ -3,7 +3,6 @@
 !.gitignore
 !.github
 !.readthedocs.yaml
-!.flake8
 
 # Byte / compiled / optimized
 *.py[cod]

.gitignore

@@ -33,8 +33,7 @@ test = [
 ]
 dev = [
     "ewoksppf[test]",
-    "black >=25",
-    "flake8 >=4",
+    "ruff>=0.16.0",
 ]
 doc = [
     "ewoksppf[test]",
@@ -57,6 +56,22 @@ omit = ['*/tests/*']
 [project.entry-points."ewoks.engines"]
 "ppf" = "ewoksppf.engine:PpfWorkflowEngine"
 
-[tool.isort]
-profile = "black"
-force_single_line = "True"
+[tool.ruff.lint]
+select = [
+    "E", # pycodestyle errors
+    "F", # pyflakes
+    "I", # isort
+    "S", # flake8-bandit
+]
+ignore = [
+    "E501", # line too long
+]
+
+[tool.ruff.lint.per-file-ignores]
+"src/ewoksppf/tests/*.py" = [
+    "S101"  # allow asserts
+]
+
+[tool.ruff.lint.isort]
+force-single-line = true
+known-first-party = ["ewoksppf"]

pyproject.toml

@@ -4,23 +4,24 @@
 """
 
 import os
-import random
 import time
+from random import SystemRandom
 
 import numpy
 from ewokscore.task import Task
 from silx.io import h5py_utils
 
 from ewoksppf import execute_graph
 
+_random = SystemRandom()
+
 
 class GenerateData(
     Task,
     input_names=["nblocks", "block_size"],
     optional_input_names=["block_index"],
     output_names=["image_stack", "block_index", "finished"],
 ):
-
     def run(self):
         t0 = time.perf_counter()
         block_index = self.get_input_value("block_index", 0)
@@ -34,7 +35,7 @@ def run(self):
         self.outputs.block_index = block_index + 1
         self.outputs.finished = self.outputs.block_index >= self.inputs.nblocks
         t1 = time.perf_counter()
-        print(f"{block_size/(t1-t0)} images/sec")
+        print(f"{block_size / (t1 - t0)} images/sec")
 
 
 class IntegrateData(
@@ -43,25 +44,23 @@ class IntegrateData(
     optional_input_names=["axis", "delay"],
     output_names=["pattern_stack", "block_index"],
 ):
-
     def run(self):
         image_axis = self.get_input_value("axis", 0)
         self.outputs.pattern_stack = self.inputs.image_stack.sum(axis=image_axis + 1)
         self.outputs.block_index = self.inputs.block_index
 
         delay = self.get_input_value("delay", 0)
         if delay:
-            time.sleep(random.uniform(delay, delay * 1.5))
+            time.sleep(_random.uniform(delay, delay * 1.5))
         else:
-            time.sleep(random.uniform(0, 0.1))
+            time.sleep(_random.uniform(0, 0.1))
 
 
 class SaveData(
     Task,
     input_names=["data_stack", "block_index", "filename"],
     output_names=["hdf5_url"],
 ):
-
     def run(self):
         filename = os.path.abspath(self.inputs.filename)
         block_index = self.inputs.block_index

scripts/mapreduce_workflow.py

@@ -10,7 +10,6 @@
 
 
 class PpfWorkflowEngine(WorkflowEngine):
-
     def execute_graph(
         self,
         graph: Any,