Skip to content
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

The data wrangling vignette appears to still be incomplete and need checking #39

Closed
adamhsparks opened this issue Aug 7, 2020 · 3 comments
Assignees

Comments

@adamhsparks
Copy link
Member

From the text of the file itself.

This file is not complete and there are a number of things that need checking - Mean squares are only calculated for the triazoles and controls - If the years for each trial are input correctly - If the disease and yield errors are calculated and that they are shuffled correctly into the data frame - AUDPC needs to be checked for starting dates and if trials have been omitted - Check that what is generated from this file matches the main file and update - write.csv should have row.names = FALSE

If this isn't complete, then it needs to be checked and completed so that we can finish the analysis and then the paper that we've been working on.

It doesn't appear to be a documented vignette, rather it's several chunks of code just pasted in a Markdown file and run, to what purpose? How does this fit in the analysis workflow?

Also, I don't understand why it's not a part of the workflow itself if it is actually a part of the analysis that was performed.

@PaulMelloy
Copy link
Collaborator

I'll look over the file to make sure the comments are addressed in the code.
Thinking more on this should it be a chapter, part of the bookdown. It could be 'Import and format data' or part of the 'Prep data' chapter?

@adamhsparks
Copy link
Member Author

Thanks! I'd think it should be chapter 2, "Import and format Data".

@PaulMelloy
Copy link
Collaborator

DataWrangle vignette has been moved to chapter 2 of the bookdown document and reminders at the top double-checked and removed

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

No branches or pull requests

2 participants