Skip to content

Create pkgdown reference index#113

Merged
nikosbosse merged 2 commits intomasterfrom
pkgdown-index
Jul 20, 2021
Merged

Create pkgdown reference index#113
nikosbosse merged 2 commits intomasterfrom
pkgdown-index

Conversation

@Bisaloo
Copy link
Copy Markdown
Contributor

@Bisaloo Bisaloo commented Jul 20, 2021

scoringutils exports a lot of functions and while I was trying to learn more about the package functionality, it was difficult to create mental categories of which function serves which purpose. In this PR, I make these categories explicit by creating a reference index for the pkgdown website.

I can revisit this PR as much as necessary, e.g., by adding a short description for each category. I also see that you are probably in the process of submitting to CRAN so feel free to delay merging this PR as you might not want to introduce extra sources of potential issues at the last minute.

For reference, here is the relevant pkgdown help page where you can see all the options for the reference index: https://pkgdown.r-lib.org/reference/build_reference.html

A screenshot of the output:

Screen Shot 2021-07-20 at 07 18 07-fullpage

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 20, 2021

Codecov Report

Merging #113 (3c95cf0) into master (c091495) will increase coverage by 0.11%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
+ Coverage   48.74%   48.85%   +0.11%     
==========================================
  Files          18       18              
  Lines        1356     1357       +1     
==========================================
+ Hits          661      663       +2     
+ Misses        695      694       -1     
Impacted Files Coverage Δ
R/eval_forecasts_helper.R 100.00% <ø> (ø)
R/pairwise-comparisons.R 42.14% <ø> (ø)
R/utils.R 63.49% <ø> (ø)
R/sharpness.R 75.00% <0.00%> (ø)
R/eval_forecasts_continuous_integer.R 69.64% <0.00%> (ø)
R/eval_forecasts_quantile.R 94.05% <0.00%> (+0.05%) ⬆️
R/eval_forecasts.R 66.07% <0.00%> (+0.89%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c091495...3c95cf0. Read the comment docs.

@Bisaloo Bisaloo requested a review from nikosbosse July 20, 2021 08:04
@nikosbosse
Copy link
Copy Markdown
Collaborator

This is really nice! So when you build the package it automatically builds a html file with the contents of the package?

@Bisaloo
Copy link
Copy Markdown
Contributor Author

Bisaloo commented Jul 20, 2021

Ah yes, sorry, I should have mentioned it. The current page is here. It contains the list of documented functions in scoringutils (even if they're not exported, as long as they're documented). By default, it's a single list but you can group the functions in categories to make discovery easier.

In short, it will not change the package itself, neither its functionality nor its built-in documentation. It will only affect the online documentation available at https://epiforecasts.io/scoringutils/.

@Bisaloo
Copy link
Copy Markdown
Contributor Author

Bisaloo commented Jul 20, 2021

And yes, with the change proposed here, the pkgdown.yaml action will automatically build this references/index.html file after each push to the master branch.

@nikosbosse
Copy link
Copy Markdown
Collaborator

sweet! Thanks :)

@nikosbosse nikosbosse merged commit edf3d63 into master Jul 20, 2021
@Bisaloo Bisaloo deleted the pkgdown-index branch July 21, 2021 07:19
@Bisaloo Bisaloo mentioned this pull request Jul 27, 2021
53 tasks
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