Open mahsasarafrazi opened 2 years ago
Please check off boxes as applicable, and elaborate in comments below. Your review is not limited to these topics, as described in the reviewer guide
The package includes all the following forms of documentation:
URL
, BugReports
and Maintainer
(which may be autogenerated via Authors@R
).Estimated hours spent reviewing: 1.5 hours
You may consider adding the codecov badge.
There is room to do more tests on the code as the code coverage is currently at 45%. It would be great if you are able to test more edge cases to make your package more robust against user error.
You may consider adding more details in the Roxygen2 of the functions, there are no examples for the functions.
Only function get_store() works for me, seems there is namespace issue with other 3 functions.
Example & usage is easy to follow, you may also consider adding links / tutorials on how to generate the personal ‘bearer token’ for the get_store() function.
You may describe where your packages fit into the R ecosystem (are there any other R packages that have the same/similar functionality) in the readme.
Please check off boxes as applicable, and elaborate in comments below. Your review is not limited to these topics, as described in the reviewer guide
The package includes all the following forms of documentation:
URL
, BugReports
and Maintainer
(which may be autogenerated via Authors@R
).Estimated hours spent reviewing: 1h
Please check off boxes as applicable, and elaborate in comments below. Your review is not limited to these topics, as described in the reviewer guide
The package includes all the following forms of documentation:
URL
, BugReports
and Maintainer
(which may be autogenerated via Authors@R
).Estimated hours spent reviewing: 1 hr
Please check off boxes as applicable, and elaborate in comments below. Your review is not limited to these topics, as described in the reviewer guide
The package includes all the following forms of documentation:
URL
, BugReports
and Maintainer
(which may be autogenerated via Authors@R
).Estimated hours spent reviewing: 1h
Amazing work! The package works as expected and as described. I love how the name conveys the benefit of being able to generate plots quickly. The peer reviewers above me provided detailed feedback that I agree with. In addition, I noted some minor revisions, as follows.
Aside from the minor issues mentioned above, the package has been so comprehensive and easy to use overall. I really appreciate that you can use this package to make data visualization easier for non-technical people. Thanks!
Submitting Author Name: Mahsa Sarafrazi Submitting Author Github Handle: !--author1-->@mahsasarafrazi<!--end-author1-- Other Package Authors Github handles: (comma separated, delete if none) Amir Shojakhani: @amirshoja, Shiva Shankar Jena: @shivajena, Mahmood Rahman: @mahm00d27 Repository: https://github.com/UBC-MDS/rtweetlytics Version submitted: Standard Submission type: Standard Reviewers: Xiaohan Chen, James Kim, Brandon Lam, Zihan Zhou Archive: TBD Version accepted: TBD Language: en
Scope
Please indicate which category or categories from our package fit policies this package falls under: (Please check an appropriate box below. If you are unsure, we suggest you make a pre-submission inquiry.):
Explain how and why the package falls under these categories (briefly, 1-2 sentences): The package rtweetlytics is a package intended to give insight about a topic on Twitter through some functions. The intention is that a user with little knowledge about data science can quickly call a function to analyze how the topics and trends are on Twitter. Internally, the package uses the official Twitter API, stores the data as a .json and .csv file, perform data cleaning, data analysis, and plotting.
Who is the target audience and what are scientific applications of this package? Any data professionals at the entry-level would like to conduct a quick exploratory data analysis on Twitter.
Are there other R packages that accomplish the same thing? If so, how does yours differ or meet our criteria for best-in-category?
from Twitter API and token to plotting, all the procedures that a person should accomplish to get the result, now combine in one package
(If applicable) Does your package comply with our guidance around Ethics, Data Privacy and Human Subjects Research?
If you made a pre-submission inquiry, please paste the link to the corresponding issue, forum post, or another discussion, or @tag the editor you contacted.
Explain reasons for any
pkgcheck
items which your package is unable to pass.Technical checks
Confirm each of the following by checking the box.
This package:
Publication options
[ ] Do you intend for this package to go on CRAN?
[ ] Do you intend for this package to go on Bioconductor?
[ ] Do you wish to submit an Applications Article about your package to Methods in Ecology and Evolution? If so:
MEE Options
Code of conduct