-
Notifications
You must be signed in to change notification settings - Fork 2
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
Package size #159
Comments
We can start with removing the plot sandbox, the vignette produces a number of plots and tables. I'm not sure if that will have a large impact as it reuses citation data from the screening phase vignette. I'll remove it and test to see where that gets us. |
Removing the plot sandbox did it! I'm not sure exactly what size it's sitting at now but would like to figure that out. We may need to consolidate the vignettes a bit more. The CMD check had a warning for being over 6.4MB, however, the guidelines say that the package should not be over 5MB. @rootsandberries what are your thoughts on removing the "Assessing Usefulness of Databases for Evidence Synthesis" vignette as it does much the same as the "Source Analysis Across Screening Phases" vignette? Happy to work to make sure that one is meeting all of our needs for the use case. |
Not sure if this is actually fixed. When running the devtools::check(remote = TRUE, manual = TRUE) I get the message Size of tarball: 7725382 bytes, which is 7.7MB... hard to know, but believe we still need to reduce. |
Yes, I think removing "Assessing Usefulness of Databases for Evidence Synthesis" makes a lot of sense. Its very repetitive with "Source Analysis Across Screening Phases" at this point. Hopefully, that will bring the size down sufficiently! |
The current package size without vignettes is 3.04MB - the vignettes are about 20MB with two (Benchmarking & Source Analysis Across Screening Phases) each at about 9MB each. Let's discuss what options we might have to trim down vignettes, move vignettes somewhere outside the package, etc. Video vignettes are still planned so we can also point to those and keep them updated pretty easily. |
The current size is 7.2MB and we'll need to get it under 5MB.
The text was updated successfully, but these errors were encountered: