[florence-bockting]

Description

We might want to update the documentation generation of roxygen2 to v8.0.0.
I simply moved on and created now a PR for this without first making an Issue. I hope that's okay for this change.

Changes

• DESCRIPTION: roxygen2 8.1.0 replaces RoxygenNote: 7.3.3 with Config/roxygen2/version: 8.010.
• man/*.Rd (42 files): all links now use one style, e.g. \link[=f]{f()} instead of \link{f}(). The rendered help does not change.
• NAMESPACE: multiple imports per package are now grouped together.

Checks

• R CMD check --as-cran (local, vignettes skipped): 0 errors, 0 warnings.

AI assistance

This PR used LLM assistance.

Copyright and Licensing

Copyright holder: Aalto University

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

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[github-actions]

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

• ✔️as_draws_array: 93ms -> 92.1ms [-2.4%, +0.5%]
• ✔️as_draws_df: 30.5ms -> 30.5ms [-1.14%, +1.26%]
• ✔️as_draws_list: 166ms -> 163ms [-4.28%, +1.08%]
• ✔️as_draws_matrix: 24.5ms -> 24.1ms [-5.42%, +2.45%]
• ✔️as_draws_rvars: 73.1ms -> 75.4ms [-3.54%, +9.88%]
• ✔️summarise_draws_100_variables: 593ms -> 594ms [-0.58%, +1.11%]
• ✔️summarise_draws_10_variables: 65.8ms -> 66.3ms [-0.24%, +1.78%]
  Further explanation regarding interpretation and methodology can be found in the documentation.

[VisruthSK]

Happened to see this and bumped to 8.1.0, hope you don't mind.

[florence-bockting]

Happened to see this and bumped to 8.1.0, hope you don't mind.

Ah okay, that works as well. Thanks. My main thought was making it consistent with loo. But perhaps we can then simply update roxygen also in loo

[VisruthSK]

Just opened a PR in loo too: stan-dev/loo#418!

[jgabry]

Just opened a PR in loo too: stan-dev/loo#418!

Thanks, merged!

[florence-bockting]

@VisruthSK, I just added you here for the review as you basically looked already at it and I don't want to merge without approval.

[VisruthSK]

Lgtm, everything's green. Didn't read the exact Rdoc changes but I haven't seen any issues in any other packages/for 8.1.0 on the whole so I think its good to go. The DESC and NAMESPACE changes both seem good too, essentially all reformatting.