Skip to content

Commit 7a73bf1

Browse files
committed
Revert "Merge pull request #359 from stan-dev/master"
This reverts commit 8d83fc2, reversing changes made to cddb7cc.
1 parent 9d199fb commit 7a73bf1

3 files changed

Lines changed: 15 additions & 34 deletions

File tree

Lines changed: 8 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -1,46 +1,25 @@
11
name: Continuous Benchmarks (Comment)
22

33
concurrency:
4-
group: ${{ github.workflow }}-${{ github.run_id }}
4+
group: ${{ github.workflow }}-${{ github.head_ref }}
55
cancel-in-progress: true
66

77
on:
88
workflow_run:
99
workflows: ["Continuous Benchmarks (Receive)"]
10-
types: [completed]
10+
types:
11+
- completed
1112

1213
jobs:
13-
comment:
14+
upload:
1415
runs-on: ubuntu-latest
1516
permissions:
1617
actions: read
1718
pull-requests: write
18-
if: ${{ github.event.workflow_run.event == 'pull_request' }}
19+
statuses: write
20+
if: >
21+
${{ github.event.workflow_run.event == 'pull_request' }}
1922
steps:
20-
- name: Download Touchstone artifact
21-
id: download
22-
uses: actions/download-artifact@v8
23-
with:
24-
name: pr
25-
github-token: ${{ github.token }}
26-
repository: ${{ github.repository }}
27-
run-id: ${{ github.event.workflow_run.id }}
28-
29-
# defensive since issues could cause commenting in random places
30-
- name: Read PR number
31-
id: pr
32-
shell: bash
33-
run: |
34-
number="$(tr -cd '0-9' < ./NR)"
35-
test -n "$number"
36-
echo "number=$number" >> "$GITHUB_OUTPUT"
37-
38-
- name: Create or update sticky PR comment
39-
id: comment
40-
uses: marocchino/sticky-pull-request-comment@v3
23+
- uses: lorenzwalthert/touchstone/actions/comment@main
4124
with:
4225
GITHUB_TOKEN: ${{ github.token }}
43-
number_force: ${{ steps.pr.outputs.number }}
44-
header: touchstone
45-
path: ./info.txt
46-
skip_unchanged: true

touchstone/config.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
2-
"os": "ubuntu-24.04",
3-
"r": "4.5.3",
2+
"os": "ubuntu-22.04",
3+
"r": "4.4.3",
44
"rspm": "https://packagemanager.posit.co/cran/__linux__/jammy/latest",
55
"benchmarking_repo": "",
66
"benchmarking_ref": "",

touchstone/script.R

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,14 @@
44
# installs branches to benchmark
55
touchstone::branch_install()
66

7-
# make log lik available to tests
87
touchstone::pin_assets("touchstone/wine.rds")
98

9+
# These synthetic workloads are large enough to expose real slowdowns in the
10+
# core `loo()` paths, but still short enough to keep PR feedback reasonably fast.
1011
touchstone::benchmark_run(
1112
expr_before_benchmark = {
1213
suppressPackageStartupMessages(library(loo))
14+
# benchmark_run() evaluates in a callr subprocess, so load pinned assets here.
1315
wine_log_lik_matrix <- readRDS(touchstone::path_pinned_asset(
1416
"touchstone/wine.rds"
1517
))
@@ -24,7 +26,7 @@ touchstone::benchmark_run(
2426
)
2527
)
2628
},
27-
n = 60
29+
n = 10
2830
)
2931

3032
touchstone::benchmark_run(
@@ -48,7 +50,7 @@ touchstone::benchmark_run(
4850
)
4951
)
5052
},
51-
n = 60
53+
n = 10
5254
)
5355

5456
# create artifacts used downstream in the GitHub Action

0 commit comments

Comments
 (0)