-
-
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]: Diff_classifier: Parallelization of multi-particle tracking video analyses #989
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @stsievert it looks like you're currently assigned as the reviewer for this paper 🎉. ⭐ 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:
|
|
@stsievert, let me know if you have any questions! |
@stsievert, I wanted to find out how the review is coming along, and if there is anything you need from me. Thanks! |
Thanks for the reminder @ccurtis7. I've started it, and will try to finish it this week. My apologies for the delay. |
@ccurtis7 could you add DOIs for first 4 papers in |
I forgot to mention that I went and updated the |
@stsievert I hope you had a good winter break! I noticed that you have checked everything except installation and performance. I believe I have addressed all the issues you brought up. Let me know if I can do anything else to finish up! |
Thanks for the ping @ccurtis7, and my apologies for the excessive delay. I think this paper is coming to an acceptable state. Past the issues/PRs I've filed, I think the only blocker is verification of the CloudKnot parallelization claims. I'll try to do that this weekend. |
@ccurtis7 and @stsievert, how are things progressing? Thanks! |
Thanks for checking in. I’ve found a couple issues in CloudKnot, specifically on some dependencies (see Nance-Lab/diff_classifier#28 for some more detail). @ccurtis7 has resolved that issue, and I need to check the notebook again. I don’t think I would have accepted the version of diff_classifier as submitted. I think now it’s closer to being ready. |
@stsievert, looks like things are moving along then. Thank you for the update! |
Hey @cMadan, @stsievert has really given some great help on improving the quality of the package, and I've tried to be as prompt as possible in addressing issues. As he mentioned, I think there's only one last double check of the parallelization demo notebook, and it should be good to go. |
I think diff_classifier is almost ready for acceptance. Here's a quick summary of the changes since the
Here are all the commits since the |
@stsievert, thanks for the detailed update! It's good to see things moving along :) |
This review is complete! This software package is ready for acceptance. One question: Significant changes have been made since the initial submission (some changes are in #989 (comment)). I feel there should be a new release for this – I wouldn't have accepted |
@stsievert, perfect, thank you for confirming! Thanks for highlighting this difference :). After I do a final pass of the submission, I will ask @ccurtis7 to mint a new release and archive the code (@ccurtis7, wait until I do a final pass though!) and this will then be the version that is accepted. @ccurtis7, sit tight for now and I will let you know how the last steps proceed shortly. |
Thanks @cMadan! And thanks @stsievert for all your hard work through the review! |
@ccurtis7, everything look good and I'm almost ready to accept. Can you mint a new release that includes the changes from the review process? I also need a DOI for an archived version of the code (i.e., from Zenodo or figshare). |
@whedon set v0.1-beta as version |
OK. v0.1-beta is the version. |
@whedon set 10.5281/zenodo.2631862 as archive |
OK. 10.5281/zenodo.2631862 is the archive. |
@whedon accept |
|
|
@ccurtis7, it looks like a recent change broke the paper.md (or bib) |
@whedon generate pdf |
|
Hmmm, it appears that whedon has changed how it treats Latex characters in the title. Previously, I had to type Do I have to generate another release, or is the |
@ccurtis7, yes, I know you previously had the |
@whedon accept |
|
|
Check final proof 👉 openjournals/joss-papers#610 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#610, then you can now move forward with accepting the submission by compiling again with the flag
|
@openjournals/joss-eics, I think we're all set to accept here! |
@ccurtis7 — Since we will have the Zenodo archive DOI on the front page of the paper, as part of the metadata shown in the margin note, we don't include a citation to it or mention it in the paper. Can you delete that? And while you're at it, please edit:
|
@whedon generate pdf |
|
I have made the requested changes @labarba, thank you. |
@whedon accept |
|
Check final proof 👉 openjournals/joss-papers#611 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#611, then you can now move forward with accepting the submission by compiling again with the flag
|
|
@whedon accept deposit=true |
|
🚨🚨🚨 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, @ccurtis7, your JOSS paper is published! Sincere thanks to the editor: @cMadan, and reviewer: @stsievert — Merci, gracias, danke 🙏 |
🎉🎉🎉 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: @ccurtis7 (Chad Curtis)
Repository: https://github.com/ccurtis7/diff_classifier
Version: v0.1-beta
Editor: @cMadan
Reviewer: @stsievert
Archive: 10.5281/zenodo.2631862
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) 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
@stsievert, 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.theoj.org/about#reviewer_guidelines. Any questions/concerns please let @cMadan know.
✨ Please try and complete your review in the next two weeks ✨
Review checklist for @stsievert
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?The text was updated successfully, but these errors were encountered: