-
Notifications
You must be signed in to change notification settings - Fork 18
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
Data validation and other tests #14
Comments
@TomHardwicke what are your thoughts on this now? Should we add a note to the project management chapter? I don't think we need a full appendix. |
Yes, something on data validation could fit quite well into the project management chapter. I very briefly mention defensive programming in the writing chapter but without any detail on how to do it practically - is it worth adding a few lines on how to do use code tests in the R Markdown appendix? |
I like the idea of referencing it in the appendix as well. If you didn't
mind adding a bit of prose to each that would be great! Thank you.
…On Thu, Jul 21, 2022 at 1:42 AM Tom Hardwicke ***@***.***> wrote:
Yes, something on data validation could fit quite well into the project
management chapter. I very briefly mention defensive programming in the
writing chapter but without any detail on how to do it practically - is it
worth adding a few lines on how to do use code tests in the R Markdown
appendix?
—
Reply to this email directly, view it on GitHub
<#14 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAI25FZTKE63W2QSD7QB7CDVVEEQVANCNFSM5DY5474A>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
A reviewer suggests we include some advice on data validation. I can add something on this, but my current practice is pretty unsophisticated - I just sprinkle my code with a few
stopifnot()
statements whenever it occurs to me.validate
, but they can get quite complex quite quickly and are potentially beyond our scope, or better addressed in an appendix.The text was updated successfully, but these errors were encountered: