-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
[REVIEW]: datawizard: An R Package for Easy Data Preparation and Statistical Transformations #4684
Comments
Hello humans, I'm @editorialbot, a robot that can help you with some common editorial tasks. For a list of things I can do to help you, just type:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
|
Wordcount for |
|
Review checklist for @garretrcConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
👋 @tomfaulkenberry, @garretrc, could you please update us on how it's going with your reviews? |
Should have it done today! I'm sorry for my delay...our semester got started right at the same time the review period got started. I appreciate the reminder :) |
Review checklist for @tomfaulkenberryConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
thanks for the reminder, I'll start taking a look at the repo and docs today but may not be able to test functionality until next week |
@osorensen We needed to make a new CRAN release. Therefore, can you please bump the version to |
@IndrajeetPatil, we set the final version once the paper is ready for acceptance, so no need to bump the version right now. |
But I can do it anyway: |
@editorialbot set 0.6.0 as version |
Done! version is now 0.6.0 |
👋 @tomfaulkenberry, @garretrc, could you please update us on how it's going with your reviews? Feel free to add comments to your reviews in this thread, or to open issues in the source repository |
I've had the chance to incoporate many of the datawizard functions into my own workflow to test the functionality, I'll wrap up checking some of the remaining functions that I haven't been able to test today. This package has a lot of useful functions even for experienced R users, so I'm also thinking about ways this package could best communicate that. |
Review finished. Everything I've tested returns the correct results and runs quickly. A couple comments:
The above points are small nitpicks, but I think the selection of functions in the package provides an accessible route to many common data manipulation tasks for new R users. For experienced R users who have previously coded the included operations on their own/through tidyverse or other packages, it seems just as fast to learn a new datawizard function than to look up/code up an equivalent solution. The datawizard functions are well-curated toward tasks that are annoying to remember/implement, so I believe they will be easier to recall and quicker to access than equivalent solutions from more general packages. |
Thanks a lot for your review @garretrc! |
Dear @garretrc, Thanks a lot for your wonderful assessment of I will respond to your nitpicks point-by-point.
In this video, I demonstrate how, depending on the supplied argument, if you hit tab, the IDE will provide the accurate argument list. But, if put the cursor on the function name and hit tab, it always displays the same tooltip, irrespective of which S3 method is dispatched. ide.movThanks again for your review, and let us know if you have any other comments or suggestions. |
@IndrajeetPatil I'll leave a comment on that issue if I think of anything! I didn't know you could supply the first argument like that to change the tooltip, it definitely improves the experience. Not really in your control to make sure a user inputs the first argument before pressing tab. |
@tomfaulkenberry could you please update us on how it's going with your review? |
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉📄 Download article If the paper PDF and the deposit XML files look good in openjournals/joss-papers#3578, then you can now move forward with accepting the submission by compiling again with the command |
@osorensen I have checked the final proofs and everything looks good to me. Let me know if there is anything else that I need to do. Thanks. |
@editorialbot accept |
|
🐦🐦🐦 👉 Tweet for this paper 👈 🐦🐦🐦 |
🚨🚨🚨 THIS IS NOT A DRILL, YOU HAVE JUST ACCEPTED A PAPER INTO JOSS! 🚨🚨🚨 Here's what you must now do:
Any issues? Notify your editorial technical team... |
@tomfaulkenberry, @garretrc – many thanks for your reviews here and to @osorensen for editing this submission! JOSS relies upon the volunteer effort of people like you and we simply wouldn't be able to do this without you ✨ @IndrajeetPatil – your paper is now accepted and published in JOSS ⚡🚀💥 |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 If you would like to include a link to your paper from your README use the following code snippets:
This is how it will look in your documentation: We need your help! The Journal of Open Source Software is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:
|
Hey @arfon, the value for the archive is incorrect, it should be set to As reported here by @sdruskat |
@editorialbot set 10.5281/zenodo.7143971 as archive |
Done! archive is now 10.5281/zenodo.7143971 |
@editorialbot reaccept |
|
|
The error is caused by the name of the directory containing the paper having a space in it : |
@xuanxu I've done it |
@editorialbot reaccept |
|
🌈 Paper updated! New PDF and metadata files 👉 openjournals/joss-papers#5241 |
Submitting author: @IndrajeetPatil (Indrajeet Patil)
Repository: https://github.com/easystats/datawizard
Branch with paper.md (empty if default branch):
Version: 0.6.2
Editor: @osorensen
Reviewers: @tomfaulkenberry, @garretrc
Archive: 10.5281/zenodo.7143971
Status
Status badge code:
Reviewers and authors:
Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) by leaving comments in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)
Reviewer instructions & questions
@tomfaulkenberry & @garretrc, your review will be checklist based. Each of you will have a separate checklist that you should update when carrying out your review.
First of all you need to run this command in a separate comment to create the checklist:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @osorensen know.
✨ Please start on your review when you are able, and be sure to complete your review in the next six weeks, at the very latest ✨
Checklists
📝 Checklist for @garretrc
📝 Checklist for @tomfaulkenberry
The text was updated successfully, but these errors were encountered: