Closed mattssca closed 1 year ago
Given that this PR should pass the build check, it's ready for review.
This PR has been updated to include a fix for issue https://github.com/morinlab/GAMBLR/issues/205
In addition, the description for the collate_results
example in the utility vignette has been updated and the website rebuilt with respect to this update. See issue https://github.com/morinlab/GAMBLR/issues/204https://github.com/morinlab/GAMBLR/issues/204 for more info.
This PR Includes
This small PR addresses various improvements to the website.
Firstly,
*.png
has been omitted from the.gitignore
and all example-related figures have been added to the repo (the function reference page is now complete with figures).The README has been updated to more accurately describe how to run GAMBLR on your local computer.
Hot-fix for
setup_expression_data
and doc updates toget_sample_cn_segments
.In addition, the vignettes have been updated (and re-knitted). For example, the
%VignetteIndexEntry
has been updated to reflect the name of the vignette (build_site previously threw a warning because of discrepancies between the two names).Lastly, GAMBLR docs have been updated and the site has been rebuilt with respect to the changes included in this PR.
Pull Request Checklists
Important: When opening a pull request, keep only the applicable checklist and delete all other sections.
Checklist for all PRs
Required
This can be checked and addressed by running
check_functions.pl
and responding to the prompts. Test your code after you do this.[x] I generated the documentation and checked for errors relating to the new function (e.g.
devtools::document()
) and addedNAMESPACE
and all other modified files in the root directory and underman
.[x] I have rebuilt the site with
pkgdown::build_site(lazy = TRUE)
to reflect any updated package documentation.Optional but preferred with PRs
Example: