Use ruff check (#131)

* Use ruff check

* lint

* skip

Author: xadupre

_cmake/clang_format.sh

@@ -1,7 +1,7 @@
 #!/bin/bash
 clear
 echo "--ruff--"
-ruff .
+ruff check .
 echo "--cython-lint--"
 cython-lint .
 echo "--clang-format--"

_unittests/ut_plotting/test_plot_gallery.py

@@ -4,6 +4,7 @@
 import unittest
 import warnings
 import http.client
+import urllib.error
 import numpy
 from mlinsights.ext_test_case import ExtTestCase, unzip_files
 from mlinsights.plotting import plot_gallery_images
@@ -62,9 +63,8 @@ def test_plot_gallery_url(self):
 
         try:
             fig, ax = plot_gallery_images(files, return_figure=True)
-        except http.client.RemoteDisconnected as e:
-            warnings.warn(f"Unable to fetch image {e}'")
-            return
+        except (http.client.RemoteDisconnected, urllib.error.HTTPError) as e:
+            raise unittest.SkipTest(f"Unable to fetch image {e} (url={root!r})")
         self.assertNotEmpty(fig)
         self.assertNotEmpty(ax)
 

azure-pipelines.yml

@@ -24,7 +24,7 @@ jobs:
   - script: pip install -r requirements-dev.txt
     displayName: 'Install Requirements dev'
   - script: |
-      ruff .
+      ruff check .
     displayName: 'Ruff'
   - script: |
       black --diff .
@@ -98,7 +98,7 @@ jobs:
   - script: pip install -r requirements-dev.txt
     displayName: 'Install Requirements dev'
   - script: |
-      ruff .
+      ruff check .
     displayName: 'Ruff'
   - script: |
       black --diff .

mlinsights/mlbatch/pipeline_cache.py

@@ -1,6 +1,10 @@
 from sklearn.base import clone
 from sklearn.pipeline import Pipeline, _fit_transform_one
-from sklearn.utils import _print_elapsed_time
+
+try:
+    from sklearn.utils._user_interface import _print_elapsed_time
+except ImportError:
+    from sklearn.utils import _print_elapsed_time
 from sklearn.utils.validation import check_memory
 from .cache_model import MLCache
 

pyproject.toml

@@ -146,10 +146,10 @@ disable_error_code = ["override", "index"]
 exclude = [".eggs", ".git", "build", "dist"]
 line-length = 88
 
-[tool.ruff.mccabe]
+[tool.ruff.lint.mccabe]
 max-complexity = 10
 
-[tool.ruff.per-file-ignores]
+[tool.ruff.lint.per-file-ignores]
 "_unittests/ut_plotting/test_dot.py" = ["E501"]
 "mlinsights/mlbatch/__init__.py" = ["F401"]
 "mlinsights/metrics/__init__.py" = ["F401"]