-
-
Notifications
You must be signed in to change notification settings - Fork 36
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]: Your: Your Unified Reader #2750
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @pravirkr, @paulray it looks like you're currently assigned to review this paper 🎉. Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post. ⭐ Important ⭐ If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews 😿 To fix this do the following two things:
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:
|
|
The software package I have opened the following issues within the main repository:
Most of these are very minor issues. Apologies for nitpicking (reviewing for the first time). |
@pravirkr, thanks for the issues. We have addressed those from our end, feel free to take a look and close them. |
I'm supposed to answer this question: "Does the full list of paper authors seem appropriate and complete?" |
Hi @paulray, I can answer that here: |
Thanks, that is extremely helpful! |
Regarding my review, I want to disclose the following potential COIs.
I just now realized that the submitting author is a NANOGrav member. I don't think that is a serious conflict since I wasn't even aware of it. According to the statement above, my other potential COIs generally have a largely advisory role in this project or have prior code that was subsumed into this project. I don't think these potential COIs affect my ability to make an impartial review, so I propose that they be waived. |
@paulray: Thanks for bringing these up! I agree that these potential conflicts seem ok to be waived. @KshitijAggarwal: do you or co-authors don't have any concerns about this? Please feel free to let me know here or offline via email if you'd prefer. Thanks! |
I enjoyed reviewing the paper and the code and trying it out on same data I had. I found it well explained, easy to install and the documentation was very clear. Nice work! |
@pravirkr @dfm, we are submitting an NSF proposal for a project (the petabyte project) which is going to rely on this library. The NSF proposal is due pretty soon. I also noticed that only a few checklist items are left in the review. So I wanted to ask if it would be possible to complete this review in the next day or so? This way we can properly cite it in our proposal, and also upload it to arxiv. |
You're welcome to post the pre-print to ArXiv, but sometimes ArXiv doesn't accept JOSS papers so no guarantees that it will work! @pravirkr: you definitely shouldn't feel pressured to finish your review faster than planned! |
@dfm: Thanks. I was waiting for the resolution of the above issues. I also recommend acceptance of this work. |
@whedon generate pdf |
@whedon check references |
|
@pravirkr, @paulray: Thanks for your reviews!! @KshitijAggarwal: I'm going to go through the paper and do some final checks. Then I'll give you a few instructions to prepare for final review. This will include minting a new release so no need to do that yet! |
@KshitijAggarwal: I've just submitted a small pull request with some formatting changes. After you take a look at that, here are the steps to run:
Let me know if you have any issues or questions! |
@whedon set 10.5281/zenodo.4269947 as archive |
OK. 10.5281/zenodo.4269947 is the archive. |
@whedon set 0.6.0 as version |
OK. 0.6.0 is the version. |
@KshitijAggarwal: I found a few more remaining formatting issues with the manuscript. Take a look at that PR and then we should be good to go! |
@dfm merged! |
@whedon generate pdf |
@whedon check references |
|
@whedon accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#1907 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#1907, then you can now move forward with accepting the submission by compiling again with the flag
|
@whedon accept deposit=true |
|
🐦🐦🐦 👉 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... |
Congrats on your new publication @KshitijAggarwal! Many thanks to editor @dfm and reviewers @pravirkr and @paulray for your time and expertise!! 🎉 |
🎉🎉🎉 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! 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:
|
Thanks a lot, everyone!!! :) :) |
Submitting author: @KshitijAggarwal (Kshitij Aggarwal)
Repository: https://github.com/thepetabyteproject/your
Version: 0.6.0
Editor: @dfm
Reviewer: @pravirkr, @paulray
Archive: 10.5281/zenodo.4269947
Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.
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
@pravirkr & @paulray, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @dfm 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 ✨
Review checklist for @pravirkr
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @paulray
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: