another try

Author: Hartorn

.github/workflows/build-python.yml

@@ -116,8 +116,6 @@ jobs:
           pdm install -G :all
           pdm list
 
-      - name: Lint code
-        run: pdm run lint
       - name: Re-install lightgbm from sources for MacOS
         if: ${{ matrix.os == 'macos-latest' }}
         run: |
@@ -160,6 +158,9 @@ jobs:
       - name: Test mlflow
         run: pdm run pytest tests/integrations/test_mlflow.py::test_fast[diabetes_dataset_with_target-linear_regression_diabetes]
 
+      - name: Lint code
+        run: pdm run lint
+
       - name: Test code (concurrency)
         run: pdm test-worker
 

tests/integrations/test_mlflow.py

@@ -14,10 +14,18 @@
 
 
 def _evaluate(dataset, model, evaluator_config):
+    import os
+    import platform
+
     with TemporaryDirectory() as f:
-        mlflow.set_tracking_uri(Path(f))
-        # MLFlow logic for windows seems to say : "Start with drive, no need for file:"
         # https://github.com/mlflow/mlflow/blob/b414a22bdead12a1865a0ab59460eb6b158db7d0/mlflow/utils/uri.py#L65C1-L66C1
+        # MLFlow logic for windows seems to say : "Start with drive, no need for file:"
+        # if platform.system() == "Windows":
+        #     f = f.replace(os.sep, "/")
+        #     f = "file://" + f
+        #     mlflow.set_tracking_uri(Path(f))
+        # else:
+        mlflow.set_tracking_uri(f"file://{f}")
         experiment_id = mlflow.create_experiment("test", artifact_location=f)
         with mlflow.start_run(experiment_id=experiment_id):
             model_info = model.to_mlflow()