Open arijeetchatterjee 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:
setup.py
file or elsewhere.Readme requirements The package meets the readme requirements below:
The README should include, from top to bottom:
Reviewers are encouraged to submit suggestions (or pull requests) that will improve the usability of the package as a whole. Package structure should follow general community best-practices. In general please consider:
OS: Windows 10
Could not perform primary method of installation - encountered the following error
ERROR: Failed building wheel for wordcloud
Could not perform secondary method of installation - encountered the following error
git@github.com: Permission denied (publickey).
fatal: Could not read from remote repository.
Estimated hours spent reviewing:
30 minutes.
Summary: Looks like a very cool package - unfortunately wasn't able to test it out due to some issues with installation. Will circle back to this once resolved!
pip install pytextprep
) due to dependency wordcloud
.@LukeAC Thanks for your feedback!
Regarding the installation failure, we've updated our README to include clearer instructions on how to use our package. We think that the failure could be because pip install wordcloud
was not working for some reason. As a workaround, you can install wordcloud using conda install -c conda-forge wordcloud -y
in the virtual environment and then run pip install pytextprep
.
Let us know if this works for you!
Also, to generate a word cloud, you do not need to pass the tweets through the other functions first. It's all handled in the word cloud function itself.
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:
setup.py
file or elsewhere.Readme requirements The package meets the readme requirements below:
The README should include, from top to bottom:
Reviewers are encouraged to submit suggestions (or pull requests) that will improve the usability of the package as a whole. Package structure should follow general community best-practices. In general please consider:
Note: Be sure to check this carefully, as JOSS's submission requirements and scope differ from pyOpenSci's in terms of what types of packages are accepted.
The package contains a paper.md
matching JOSS's requirements with:
Estimated hours spent reviewing: 2 hours
A cool and useful package. I followed your suggestion to install the package in windows platform, and it did work.
Here are some suggestions:
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:
setup.py
file or elsewhere.Readme requirements The package meets the readme requirements below:
The README should include, from top to bottom:
Reviewers are encouraged to submit suggestions (or pull requests) that will improve the usability of the package as a whole. Package structure should follow general community best-practices. In general please consider:
Note: Be sure to check this carefully, as JOSS's submission requirements and scope differ from pyOpenSci's in terms of what types of packages are accepted.
The package contains a paper.md
matching JOSS's requirements with:
First of all congratulations on creating a wonderful and useful package. The team has done a great job and I really found the documentation, docstrings and examples to be very good. They easily guided me through the process of working with your package. I have added my recommendations below as minor changes I feel could make this already very good package slightly better:
In general, this is great work and I enjoyed using your package!
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:
setup.py
file or elsewhere.Readme requirements The package meets the readme requirements below:
The README should include, from top to bottom:
Reviewers are encouraged to submit suggestions (or pull requests) that will improve the usability of the package as a whole. Package structure should follow general community best-practices. In general please consider:
Note: Be sure to check this carefully, as JOSS's submission requirements and scope differ from pyOpenSci's in terms of what types of packages are accepted.
The package contains a paper.md
matching JOSS's requirements with:
Estimated hours spent reviewing: 1 hour 10 mins
The package looks well-structured and it was relatively easy for me to follow the installation instructions on Windows 11. I have some proposals to make the package even more appealing!
tweets_list = ["Make America Great Again! @DonalTrump", "It's a new day in ####^America"]
Using the input above gave me this output: ['', '', '', '']
None
type would be really beneficial.Python
file was listed as API Reference in Documentation.
WordCloud
object is also a return variable.
Great work team 21!!!
Submitting Authors: Philson Chan (@PhilsChan)
Melisa Maidana (@mmaidana24318) Arijeet Chatterjee (@arijc76) Joshua Sia (@joshsia)
Package Name: pytextprep One-Line Description of Package: Python package that offers additional text preprocessing functionality specifically designed for tweets Repository Link: https://github.com/UBC-MDS/pytextprep Version submitted: v1.0.5 Editor: TBD
Reviewer 1: Luke Collins (@LukeAC) Reviewer 2: Kyle Ahn (@AraiYuno) Reviewer 3: Tianwei WANG () Reviewer 4: Linhan Cai (@lipcai)
Description
Scope
* Please fill out a pre-submission inquiry before submitting a data visualization package. For more info, see notes on categories of our guidebook.
Explain how the and why the package falls under these categories (briefly, 1-2 sentences):
Who is the target audience and what are scientific applications of this package?
Are there other Python packages that accomplish the same thing? If so, how does yours differ?
If you made a pre-submission enquiry, please paste the link to the corresponding issue, forum post, or other discussion, or
@tag
the editor you contacted:Technical checks
For details about the pyOpenSci packaging requirements, see our packaging guide. Confirm each of the following by checking the box. This package:
Publication options
JOSS Checks
- [ ] The package has an **obvious research application** according to JOSS's definition in their [submission requirements][JossSubmissionRequirements]. Be aware that completing the pyOpenSci review process **does not** guarantee acceptance to JOSS. Be sure to read their submission requirements (linked above) if you are interested in submitting to JOSS. - [ ] The package is not a "minor utility" as defined by JOSS's [submission requirements][JossSubmissionRequirements]: "Minor ‘utility’ packages, including ‘thin’ API clients, are not acceptable." pyOpenSci welcomes these packages under "Data Retrieval", but JOSS has slightly different criteria. - [ ] The package contains a `paper.md` matching [JOSS's requirements][JossPaperRequirements] with a high-level description in the package root or in `inst/`. - [ ] The package is deposited in a long-term repository with the DOI: *Note: Do not submit your package separately to JOSS*Are you OK with Reviewers Submitting Issues and/or pull requests to your Repo Directly?
This option will allow reviewers to open smaller issues that can then be linked to PR's rather than submitting a more dense text based review. It will also allow you to demonstrate addressing the issue via PR links.
Code of conduct
P.S. *Have feedback/comments about our review process? Leave a comment here
Editor and Review Templates
Editor and review templates can be found here