-
Notifications
You must be signed in to change notification settings - Fork 31
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
Initial impressions and quick suggestions #2
Comments
Hi there, thank you very much for your kind words! You raise some excellent points.
Uups. Keep forgetting that. MIT License added. I also added a section to the README in order to explain what "substantial" means in our view. If you want you can comment on whether you think it is informative.
Here you are hitting on one of my preferences ;-) I have not yet found a software agnostic alternative to make that helps you to stay on top of file dependencies. As projects become large, these really can become painful to maintain and a potential source of serious error ("Uups, we forgot to update this table based on that one variable that we redefined"). Having a script that simply runs everything is an alternative but if you move to larger projects these scripts can run a while and one might get reluctant to run it. This being said, we will discuss whether having an alternative run_me script for people wihtout access to
Good point. My guess is that we won't get rid of the core tidyverse and of knit/rmarkdown. Besides ExPanDaR (see below), most (all) of the other dependencies are linked to the example payload of the package. Maybe it is a good point to discuss this in the README.
Jupp. In this case it might be even simpler to remove the df as it is no longer needed. In principle, we are trying to keep the R code as simple as possible in order to make it easy for people to assess what the code actually does.
Oh yes. Here is the workflow that we currently use
The first two functions create lists containing dataframes and And finally:
This is what our plan is. With the help of the community we believe that it can be done. Anyway: Thank you very much again for your feedback! |
Great, thank you! |
This is an excellent, commendable and much welcome effort. Many thanks!
Please take all with a grain of salt as I have not yet had the opportunity to spend enough time on using the template. Once I do I will provide more concrete and hopefully useful feedback. I'd be excited if this grows into a stable and maintained repo in the long-term.
Here are some ideas for the next version:
.df
instead ofdf
inread_config.R
.Finally, constructing tables, to me, are still a pain from RStudio to LaTeX and I have not found an easy, out-of-the-box workflow. There are many packages that provide functionality, but they are either too general or not flexible enough. What is needed is perhaps an extension of the template specifically to table styles in academic journals. R with its limitations on value and variable labels does not help, of course.
The text was updated successfully, but these errors were encountered: