Skip to content

Update R CMD Check GitHub Action#433

Merged
VisruthSK merged 17 commits intomasterfrom
fix-ubuntu-ci
Feb 4, 2026
Merged

Update R CMD Check GitHub Action#433
VisruthSK merged 17 commits intomasterfrom
fix-ubuntu-ci

Conversation

@VisruthSK
Copy link
Member

@VisruthSK VisruthSK commented Feb 4, 2026

Summary

Fix failing R CMD Checks by swapping to r-lib GHA steps to handle sys/R dependency resolution. Also moves from bespoke steps to r-lib or native GHA steps/config.

For Ubuntu with R 3.6, since the suggested package randomForest requires 4.1, this PR selectively deletes lines in the action to not need randomForest (in example and DESCRIPTION). Pipes in tests are also removed to run on R 3.6

Copyright and Licensing

By submitting this pull request, the copyright holder is agreeing to
license the submitted work under the following licenses:

@github-actions
Copy link

github-actions bot commented Feb 4, 2026

This is how benchmark results would change (along with a 95% confidence interval in relative change) if e608c15 is merged into master:

  • ✔️as_draws_array: 136ms -> 141ms [-2.88%, +10.2%]
  • ✔️as_draws_df: 68.5ms -> 68.8ms [-0.59%, +1.47%]
  • ✔️as_draws_list: 150ms -> 149ms [-0.78%, +0.65%]
  • ✔️as_draws_matrix: 31.1ms -> 31.1ms [-0.98%, +0.82%]
  • ✔️as_draws_rvars: 78.1ms -> 78.1ms [-0.74%, +0.87%]
  • ✔️summarise_draws_100_variables: 716ms -> 716ms [-0.3%, +0.33%]
  • ❗🐌summarise_draws_10_variables: 80ms -> 80.4ms [+0.12%, +0.89%]
    Further explanation regarding interpretation and methodology can be found in the documentation.

@VisruthSK VisruthSK changed the title Added curl R packages to Ubuntu CI downloads Update R CMD Check GitHub Action Feb 4, 2026
@VisruthSK
Copy link
Member Author

VisruthSK commented Feb 4, 2026

@paul-buerkner Dependency resolution fails on Ubuntu 3.6 since a minimum version of randomForest is pinned which requires R 4.1 (see logs). Can the versioning constraint on randomForest be relaxed? Could vignettes be skipped on 3.6?

@github-actions
Copy link

github-actions bot commented Feb 4, 2026

This is how benchmark results would change (along with a 95% confidence interval in relative change) if b71f9e1 is merged into master:

  • ❗🐌as_draws_array: 127ms -> 128ms [+0.05%, +1.18%]
  • ✔️as_draws_df: 65.9ms -> 65.8ms [-0.81%, +0.69%]
  • ✔️as_draws_list: 151ms -> 151ms [-0.13%, +0.86%]
  • ✔️as_draws_matrix: 27.6ms -> 27.6ms [-0.54%, +0.91%]
  • ✔️as_draws_rvars: 76.3ms -> 76.4ms [-0.74%, +1.12%]
  • ✔️summarise_draws_100_variables: 722ms -> 721ms [-0.37%, +0.34%]
  • ✔️summarise_draws_10_variables: 83.5ms -> 83.9ms [-0.59%, +1.58%]
    Further explanation regarding interpretation and methodology can be found in the documentation.

@github-actions
Copy link

github-actions bot commented Feb 4, 2026

This is how benchmark results would change (along with a 95% confidence interval in relative change) if eef4937 is merged into master:

  • ✔️as_draws_array: 142ms -> 141ms [-1.89%, +1.58%]
  • ✔️as_draws_df: 73.4ms -> 72.4ms [-3.58%, +0.67%]
  • ✔️as_draws_list: 168ms -> 169ms [-0.96%, +2.32%]
  • ✔️as_draws_matrix: 29ms -> 28.8ms [-2.32%, +0.89%]
  • ✔️as_draws_rvars: 81.1ms -> 80.3ms [-2.6%, +0.63%]
  • ✔️summarise_draws_100_variables: 733ms -> 732ms [-0.99%, +0.48%]
  • ✔️summarise_draws_10_variables: 84.9ms -> 84.7ms [-1.4%, +0.92%]
    Further explanation regarding interpretation and methodology can be found in the documentation.

@paul-buerkner
Copy link
Collaborator

You can just merge these github action PRs without my checks if you have the github permissions for that

@github-actions
Copy link

github-actions bot commented Feb 4, 2026

This is how benchmark results would change (along with a 95% confidence interval in relative change) if b8b7070 is merged into master:

  • ✔️as_draws_array: 144ms -> 144ms [-1.37%, +1%]
  • ✔️as_draws_df: 77.3ms -> 77.6ms [-0.85%, +1.63%]
  • ✔️as_draws_list: 168ms -> 169ms [-0.85%, +1.47%]
  • ✔️as_draws_matrix: 30.1ms -> 30ms [-0.89%, +0.83%]
  • ✔️as_draws_rvars: 81.3ms -> 81.4ms [-0.95%, +1.02%]
  • ✔️summarise_draws_100_variables: 731ms -> 730ms [-0.7%, +0.65%]
  • ✔️summarise_draws_10_variables: 85.7ms -> 86.1ms [-0.59%, +1.39%]
    Further explanation regarding interpretation and methodology can be found in the documentation.

@VisruthSK VisruthSK marked this pull request as ready for review February 4, 2026 20:08
@VisruthSK VisruthSK merged commit 30d1953 into master Feb 4, 2026
10 checks passed
@VisruthSK VisruthSK deleted the fix-ubuntu-ci branch February 4, 2026 20:08
@github-actions
Copy link

github-actions bot commented Feb 4, 2026

This is how benchmark results would change (along with a 95% confidence interval in relative change) if 7889738 is merged into master:

  • ✔️as_draws_array: 125ms -> 125ms [-0.55%, +0.71%]
  • ✔️as_draws_df: 65.1ms -> 64.8ms [-1.15%, +0.21%]
  • ✔️as_draws_list: 149ms -> 150ms [-0.12%, +1.02%]
  • ✔️as_draws_matrix: 27ms -> 26.8ms [-1.32%, +0.17%]
  • ✔️as_draws_rvars: 75.5ms -> 75.4ms [-0.93%, +0.82%]
  • ✔️summarise_draws_100_variables: 730ms -> 735ms [-1.17%, +2.51%]
  • ✔️summarise_draws_10_variables: 84.1ms -> 84.3ms [-0.54%, +1.19%]
    Further explanation regarding interpretation and methodology can be found in the documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants