Fix run_asvs workflow (#36)

Author: rhshadrach

.github/workflows/process_results.yaml

@@ -11,7 +11,7 @@ env:
   # Results and other data are stored away from the main branch.
   # This keeps the main branch clean and allows us to clear out
   # objects / history after being built up for long periods of time.
-  BRANCH_NAME: pandas_20240109
+  BRANCH_NAME: pandas_20250109
   PYTHON_VERSION: 3.12
 
 permissions:
@@ -54,9 +54,9 @@ jobs:
       - name: Show environment packages
         run: uv pip freeze
 
-      - name: Move results directory
+      - name: Copy results directory
         run: |
-          mv asv-runner/data/results asv_bench/results
+          cp -r asv-runner/data/results asv_bench/results
 
       - name: Publish ASV Benchmarks
         run: cd asv_bench && asv publish

.github/workflows/run_asvs.yaml

@@ -13,7 +13,7 @@ env:
   # Results and other data are stored away from the main branch.
   # This keeps the main branch clean and allows us to clear out
   # objects / history after being built up for long periods of time.
-  BRANCH_NAME: pandas_20240109
+  BRANCH_NAME: pandas_20250109
 
 permissions:
   contents: read
@@ -66,11 +66,6 @@ jobs:
           repository: asv-runner
           file_pattern: 'data/latest_sha.txt'
 
-      - name: Move results directory
-        if: ${{ steps.new-commit.outputs.new_commit == 'yes' }}
-        run: |
-          mv asv-runner/data/results asv_bench/results
-
       - name: Set up Conda
         if: ${{ steps.new-commit.outputs.new_commit == 'yes' }}
         uses: ./.github/actions/setup-conda
@@ -87,15 +82,19 @@ jobs:
           asv run --machine=asvrunner --python=same --set-commit-hash=$(git rev-parse HEAD) --show-stderr
 
       - name: Update asv-runner branch
+        # In case there was a push by another job.
         if: ${{ steps.new-commit.outputs.new_commit == 'yes' }}
         run: |
           cd asv-runner
-          git pull
+          git fetch && git pull
 
       - name: Move results into asv-runner
         if: ${{ steps.new-commit.outputs.new_commit == 'yes' }}
         run: |
-          cp -r asv_bench/results/ asv-runner/data/results/
+          mkdir -p asv-runner/data/results/asvrunner/
+          cp asv_bench/results/benchmarks.json asv-runner/data/results/
+          cp asv_bench/results/asvrunner/machine.json asv-runner/data/results/asvrunner/
+          cp asv_bench/results/asvrunner/$(git rev-parse HEAD)-existing*.json asv-runner/data/results/asvrunner/
 
       - name: Commit results to branch
         if: ${{ steps.new-commit.outputs.new_commit == 'yes' }}

ci/make_issues.py

@@ -67,8 +67,9 @@ def run(input_path: str | Path):
         .drop_duplicates(subset="sha")
         .sort_values("date")["sha"]
         .unique()
-        .tolist()[-20:]
+        .tolist()[-40:]
     )
+    print("Number of regressions to raise issues for:", len(regression_shas))
     for sha in regression_shas:
         # Avoid GitHub rate limits
         time.sleep(2)

pyproject.toml

@@ -70,7 +70,6 @@ select = [
     "Q",   # flake8-quotes
     "RUF", # Ruff-specific rules
     "SIM", # flake8-simplify
-    "T20", # flake8-print
     "UP",  # pyupgrade
     "W",   # pycodestyle - warning
     "YTT", # flake8-2020