-
-
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]: colorspace: A Python Toolbox for Manipulating and Assessing Colors and Palettes #7120
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:
|
Software report:
Commit count by author:
|
Paper file info: 📄 Wordcount for ✅ The paper includes a |
|
License info: 🟡 License found: |
👋🏼 @retostauffer @hollowscene @dmreagan this is the review thread for the paper. All of our communications will happen here from now on. As a reviewer, the first step is to create a checklist for your review by entering
as the top of a new comment in this thread. These checklists contain the JOSS requirements. As you go over the submission, please check any items that you feel have been satisfied. The first comment in this thread also contains links to the JOSS reviewer guidelines. The JOSS review is different from most other journals. Our goal is to work with the authors to help them meet our criteria instead of merely passing judgment on the submission. As such, the reviewers are encouraged to submit issues and pull requests on the software repository. When doing so, please mention We aim for reviews to be completed within about 2-4 weeks. Please let me know if any of you require some more time. We can also use EditorialBot (our bot) to set automatic reminders if you know you'll be away for a known period of time. I encourage you to over-communicate and let everyone know that you're on the task every now and then, instead of waiting several weeks to collect all your comments, questions, or suggestions. Please feel free to ping me (@jromanowska) if you have any questions/concerns. |
Review checklist for @hollowsceneConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hey everyone, just thought I'd make an early start to some of the housekeeping tickboxes. As a heads up - I'll most likely hold off on actually installing and playing around with the package until I've got a bit more time this weekend. I hate to be nitpicky but just for the statement of need tickboxes, I feel like I'm not getting an explicit designation of who the target audience is. I definitely acknowledge it does feel a bit redundant for this package considering how ubiquitous its usage should be, and it sort of is implied within the current statement of need. But just wanted to double check this before ticking it off as I'm not entirely sure how strict we need to be! |
Good point! The tickboxes are very general - to suit all the different types of software submitted to JOSS. As long as you feel that the "Statement of need" section in the paper is alright, you can check off the box. However, you could also ask the authors to have a second look at it and improve if you feel something is missing. And thanks for starting right off - we encourage here over-commenting, so let's keep it like that 👍 |
Review checklist for @dmreaganConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hello, just to update, I've made a bit more of a dent into my checklist. Hoping to be able to work through the rest over the week ahead |
I also wondered about the explicit target audience as in #7120 (comment). But I think it is implied that the intended users are the same users of the "most graphics packages" the authors refer to in the second paragraph. It's a ✅ for me. |
Question about references: is it OK to have web links in the body of the paper? The authors have included direct URLs to a few of their own web resources while putting all 3rd-party links down in the References section. |
For me, that seems logical. However, if you think it doesn't suit, go ahead and suggest another way of doing it. JOSS does not have strict formatting rules for the papers. |
Just finished my review checklist. I did encounter one issue with some tests failing due to some Windows OS discrepancies which I've raised a GitHub issue for. Regardless, automated tests are thorough and well documented so I've still ticked it off in the checklist. Regarding references, adding 3rd-party links seems reasonable and is more quickly accessible so I've also given it a tick. Overall, the paper is well written and the package has had significant work dedicated to it. I have no doubt this package will be incredibly useful for both casual and veteran data vizzers. Amazing work :) |
Andrew @hollowscene, thank you so much for the positive and constructive feedback along with the issues and pull requests in the GitHub project. Reto is currently on vacation but we'll incorporate all your helpful suggestions in the software and paper when he's back! Thanks & best wishes, Achim |
@zeileis , @retostauffer - how is it going? Let us know when you're done with the modifications. |
Julia @jromanowska, thanks for the reminder and apologies for the delay. Reto will be back from his vacations this week. I already took some notes and I expect that we will have a revision and response to the helpful reviews very soon. |
Another follow-up, unfortunately just to say that we will need a bit more time because corona had other plans for us this week. But we promise that we will get back to this as soon as possible. Apologies again! |
Thanks for letting us know and stay safe. Get healthy while we're waiting patiently :) |
Dear @jromanowska, @hollowscene, and @dmreagan, First of all, we would like to sincerely thank you for handling our submission, as well as for the constructive and valuable feedback on our work. We apologize for the delayed response due to vacations and covid. We have thoroughly addressed all comments from the review as well as the pre-review (handled by Daniel S. Katz). We have also updated the README to acknowledge the contributions of the reviewers and editors involved in this process. Your efforts in enhancing the quality of our work are greatly appreciated. Paper
SoftwareKey Changes
Additional Changes
Once again, thank you for your time and thoughtful feedback. We look forward to your further comments and suggestions. Best regards, |
@editorialbot generate pdf |
Submitting author: @retostauffer (Reto Stauffer)
Repository: https://github.com/retostauffer/python-colorspace
Branch with paper.md (empty if default branch):
Version: v0.5.0
Editor: @jromanowska
Reviewers: @hollowscene, @dmreagan
Archive: Pending
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
@hollowscene & @dmreagan, 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 @jromanowska 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 @hollowscene
📝 Checklist for @dmreagan
The text was updated successfully, but these errors were encountered: