Merge pull request #83 from MatthewSZhang/ruff-formatter

MNT black to ruff

Author: MatthewSZhang

.github/workflows/ci.yml

@@ -11,8 +11,29 @@ jobs:
   call-test:
     uses: ./.github/workflows/test.yml
     secrets: inherit
+  
+  build-sdist:
+    runs-on: ubuntu-latest
+    needs: call-test
+    steps:
+      - uses: actions/checkout@v4
+      - uses: prefix-dev/[email protected]
+        with:
+          environments: dev
+          cache: true
+      - name: Re-install local
+        run: |
+          pixi reinstall -e dev --frozen fastcan
+      - name: Build SDist
+        run: |
+          pixi run build-sdist
+      - name: Store artifacts
+        uses: actions/upload-artifact@v4
+        with:
+          name: cibw-sdist
+          path: dist/*.tar.gz
 
-  build:
+  build-wheels:
     strategy:
       fail-fast: false
       matrix:

.github/workflows/lint.yml

@@ -0,0 +1,32 @@
+name: Static
+
+on:
+  workflow_call:
+
+jobs:
+  static:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v4
+      - uses: prefix-dev/[email protected]
+        with:
+          environments: static
+          cache: true
+
+      - name: Re-install local
+        run: |
+          pixi reinstall -e static --frozen fastcan
+
+      - name: Linter
+        run: |
+          pixi run lint
+      - name: Lint Cython
+        run: |
+          pixi run cython-lint
+      - name: Formatter
+        run: |
+          pixi run fmt
+      - name: Type check
+        run: |
+          pixi run type

.github/workflows/static.yml

@@ -9,8 +9,7 @@ on:
 
 jobs:
   call-lint:
-    uses: ./.github/workflows/lint.yml
-    secrets: inherit
+    uses: ./.github/workflows/static.yml
 
   test:
     strategy:
@@ -23,7 +22,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: prefix-dev/[email protected]
         with:
-          environments: default
+          environments: dev
           cache: true
 
       - name: Re-install local
@@ -44,3 +43,13 @@ jobs:
       - name: Test nogil
         run: |
           pixi run nogil-eta
+      - name: Test coverage
+        if: runner.os == 'Linux'
+        shell: bash
+        run: |
+          FMT=xml pixi run test-coverage
+      - name: Upload coverage reports to Codecov
+        if: runner.os == 'Linux'
+        uses: codecov/codecov-action@v5
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}

.github/workflows/test.yml

@@ -93,7 +93,7 @@ def auto_duffing_equation(y, t):
 
 sol = odeint(duffing_equation, [0.6, -0.8], t)
 u_test = 2.5 * np.cos(2 * np.pi * t).reshape(-1, 1)
-y_test = sol[:, 0]+ e_test
+y_test = sol[:, 0] + e_test
 
 # %%
 # One-step-head VS. multi-step-ahead NARX

examples/plot_narx_msa.py

@@ -178,12 +178,12 @@ def baseline(X, y, t):
 time_eta = np.zeros(n_features_max, dtype=float)
 for i in range(n_features_max):
     time_h[i] = timeit(
-        f"s = FastCan({i+1}, verbose=0).fit(X, y)",
+        f"s = FastCan({i + 1}, verbose=0).fit(X, y)",
         number=10,
         globals=globals(),
     )
     time_eta[i] = timeit(
-        f"s = FastCan({i+1}, eta=True, verbose=0).fit(X, y)",
+        f"s = FastCan({i + 1}, eta=True, verbose=0).fit(X, y)",
         number=10,
         globals=globals(),
     )

examples/plot_speed.py

@@ -13,8 +13,8 @@
 
 __all__ = [
     "FastCan",
-    "refine",
     "minibatch",
     "narx",
+    "refine",
     "utils",
 ]

fastcan/__init__.py

@@ -17,7 +17,7 @@
 from sklearn.utils._param_validation import Interval
 from sklearn.utils.validation import check_is_fitted, validate_data
 
-from ._cancorr_fast import _forward_search  # type: ignore
+from ._cancorr_fast import _forward_search  # type: ignore[attr-defined]
 
 
 class FastCan(SelectorMixin, BaseEstimator):

fastcan/_fastcan.py

@@ -13,7 +13,7 @@
 from sklearn.utils._param_validation import Interval, validate_params
 from sklearn.utils.validation import check_X_y
 
-from ._cancorr_fast import _forward_search  # type: ignore
+from ._cancorr_fast import _forward_search  # type: ignore[attr-defined]
 from ._fastcan import _prepare_search
 
 

fastcan/_minibatch.py

@@ -13,7 +13,7 @@
 from sklearn.utils._param_validation import Interval, StrOptions, validate_params
 from sklearn.utils.validation import check_is_fitted
 
-from ._cancorr_fast import _forward_search  # type: ignore
+from ._cancorr_fast import _forward_search  # type: ignore[attr-defined]
 from ._fastcan import FastCan, _prepare_search