New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Expanded the loading data vignette #780
Conversation
R/utils-data.R
Outdated
#' the dataloader. `.getbatch` must work for batches of size larger or equal to 1. | ||
#' | ||
#' the dataloader. `.getbatch` must work for batches of size larger or equal to 1. For more | ||
#' on this see the the vignette on data loading. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This ideally would have a link to the vignette
Also if this is helpful I bet I could do it for other pages too! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a super improvement!
Thanks very much @jnolis !
R/utils-data.R
Outdated
#' the dataloader. `.getbatch` must work for batches of size larger or equal to 1. | ||
#' | ||
#' the dataloader. `.getbatch` must work for batches of size larger or equal to 1. For more | ||
#' on this see the the vignette on data loading. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will make at least the pkgdown website to create a link:
#' on this see the the vignette on data loading. | |
#' on this see the the `vignette("loading-data")`. |
Up to you if you prefer adding this or not. I think most users will read it in the Rd format.
Let me know what you prefer and I'll merge this PR.
Yeah! Please do, that's super helpful! |
I expanded the load data vignette to give more of a gentle introduction for new users. I also added a part at the end about optimizing data loads that mentions getbatch and avoiding datasets entirely.
I also added a sentence or two to the
dataset
docs, although I wasn't sure how to have a hyperlink to the vignette (I'm not an R package expert)