-
-
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]: TidyTensor: Utilities for multidimensional arrays as named hierarchical structures #3543
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @nhejazi, @omid-s 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:
|
Wordcount for |
|
|
@oneilsh Would you please add the missing DOI? |
@taless474 done :) (I think) |
@whedon check references |
|
Hi @taless474, quick question: I just noticed a couple of minor typos in the documentation. Should I fix them right away or not make any adjustments to the codebase while the review is ongoing? |
Hi @oneilsh. Please go ahead and make a pull request, so the reviewers realize what the recent changes are if they were reviewing those parts. |
👋 @nhejazi, please update us on how your review is going (this is an automated reminder). |
👋 @omid-s, please update us on how your review is going (this is an automated reminder). |
Execute my belated reply, just saw the messages here :/. I will be done soon :) |
Hi @oneilsh, there are two license files, repo's license link goes to a file with two lines that sets copy right to you, though I can see another "license.md" that has a brief MIT Licence description, I was wondering which one are you planning to keep? |
Hi @omid-s, I believe that's an artifact of R's restrictions on the LICENSE file not containing the actual text of the license, whereas LICENSE.md is used more commonly as an optional extra. This gitlab issue describes some frustrations developers have had there with the duplicate. LICENSE + LICENSE.md is how |
Hi @taless474, I'm done with my review, everything looks good to me, I particularly loved the guides and the documentation! Very through and I enjoyed following them. |
Hi @omid-s, thank you for the thorough review! I've made a branch with the fixes, here's a quick link to the compiled paper: https://github.com/oneilsh/tidytensor/suites/3607702673/artifacts/87088522 If it looks good I will merge it in and close the issue. |
Sorry to have missed the comments above, I've been somewhat delayed in completing my review due to outstanding work commitments; however, I can have my review completed by the end of next week. |
@oneilsh, I apologize for my belated response. I am reviewing reviews and will comment as soon as possible. |
Hi @taless474, just checking in to see how the review is going. |
Hi @oneilsh @taless474, I'm the Associate Editor-in-Chief on duty this week, and since it looks like this submission is essentially ready to be accepted, I'm going to step in and help get it past the finish line! |
Yes, the JOSS publication is "tied" to the version that was reviewed, but we do usually expect that changes made in response to the review are reflected in that version—so if you have bumped it we can update the version here. Regarding changes made after JOSS publication, you may want to begin archiving future versions yourself, so that others can cite a version they use for reproducibility purposes. If things change substantially in the future, you could even submit to JOSS again for a new review. |
@whedon generate pdf |
OK, everything looks fine to me. @oneilsh, can you please do the following?
|
Hi @kyleniemeyer, thanks! The version that got reviewed is v0.9.1, which has Zenodo DOI 10.5281/zenodo.5098818. The "all versions" Zenodo DOI is 10.5281/zenodo.4968727. I was contemplating minting a v1.0.0 identical to v0.9.1 for purposes of labeling as the JOSS-reviewed version (and I have some documentation fixups I'd like to add after with subsequent minor version updates), but I'll defer to your wisdom on that. Supposing someone comes across the published PDF, how does it link to the reviewed version? Does the Zenodo version DOI and version number get listed in the PDF somewhere? |
It's fine if you have some additional documentation improvements that you want to include in a v1.0.0, as long as the software itself is not changing from what was reviewed. If you want to do that, I will hold this open until you report the associated DOI (the one specifically pointing to that version). The published PDF does link to the specific DOI associated with the software archive (see https://joss.theoj.org/papers/10.21105/joss.03523 for an example). |
Ok, v1.0.0 created with minor doc improvements and DOI 10.5281/zenodo.5555276. Thank you again! |
@whedon set 10.5281/zenodo.5555276 as archive |
OK. 10.5281/zenodo.5555276 is the archive. |
@whedon set v1.0.0 as version |
OK. v1.0.0 is the version. |
@whedon accept |
To recommend a paper to be accepted use |
@whedon recommend-accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#2655 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#2655, 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... |
Congratulations @oneilsh on your article's publication in JOSS! Many thanks to @nhejazi and @omid-s for reviewing this submission, and @taless474 for editing. |
🎉🎉🎉 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:
|
Submitting author: @oneilsh (Shawn T. O'Neil)
Repository: https://github.com/oneilsh/tidytensor
Version: v1.0.0
Editor: @taless474
Reviewer: @nhejazi, @omid-s
Archive: 10.5281/zenodo.5555276
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
@nhejazi & @omid-s, 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 @taless474 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 @nhejazi
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @omid-s
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: