-
-
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]: mlpack 4: a fast, header-only C++ machine learning library #5026
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 @sandeep-psConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @zhangjy-geConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
I have finished my initial review. I think overall, mlpack 4 is a well written package that has indicated great potential in the machine learning field, considering its good documentation, user friendliness, and the algorithms it implemented, etc. Thus, I would recommend it for publication after minor revision:
|
Hi, @osorensen, Two authors listed in the paper (#5 and #9) are affiliated with the same institution I'm affiliated with. We are from different units and haven't collaborated in the past, and we do not know each other. I request that this conflict be waived. Please let me know if there is a different process for this. Thanks. |
Thanks for letting me know @sandeep-ps. I think this should be waived, but I'll ping the editor-in-chief @openjournals/dsais-eics to hear what they think. |
@sandeep-ps Thanks for letting us know. I'm a bit reticent about waiving the COI because there are implications of reviewing work by others at your institution if you are both still employed there. I'd prefer that we seek a different reviewer and hope we can assign you to another submission where we need reviewers. Ok? Thank you for understanding. @osorensen Please assign to a different reviewer and keep @sandeep-ps in mind for other assignments! |
Okay. Sounds good. Thanks. FYI, my request was based on the COI guidelines, specifically, this statement. |
HI @sandeep-ps - thanks for bringing this up. You are correct in that it is a conflict, and that it could be waived if needed, but it's up the track editor to decide. |
Ok, @sandeep-ps, since you can attest that you are in a large organization and unlikely to be in contact with the individual profesionally, I am willing to grant the waiver. Given that we also have some independent feedback from the other reviewer, I think this might be ok. Let's proceed with you as a reviewer! |
Thanks @gkthiruvathukal. Then please continue with the review @sandeep-ps, and feel free to reach out to me if you have any questions about the review process. |
Thanks, @gkthiruvathukal, @danielskatz, and @osorensen. I will continue with the review then. |
I am not sure of the exact process, but when should I start responding to the review comments? Should I wait for @sandeep-ps's review to be complete? It would be straightforward to add some comparisons with other machine learning libraries, so I'm happy to do that. |
Thanks for asking @rcurtin. You're welcome and encouraged to start responding immediately. |
@rcurtin, to give a progress update, since I had not used this library before, I was going through the installation process and checking out some of the functionality. I ran into some issues during that step last week, and I will be continuing my review this week. |
@zhangjy-ge We do have some benchmarks on the homepage of the website. We can reference them more prominently in the paper---or do you mean to include some benchmarks specifically in the paper itself? (Personally I think it might be too long with a benchmarking table and sufficient description.)
@sandeep-ps happy to help out, just send a ping if you are having any issues. (You could even open an issue on mlpack's Github if you wanted, but whatever way is fine.) Often it can be easier to install mlpack via the system package manager, but our recent refactorings to make mlpack header-only have made the process much easier than it used to be. |
@rcurtin After double check, it seems a performance comparison is optional but we should still explicitly state the difference between mlpack and other packages (from |
@editorialbot set 4.0.1 as release |
I'm sorry human, I don't understand that. You can see what commands I support by typing:
|
@editorialbot set 4.0.1 as version |
Done! version is now 4.0.1 |
@editorialbot recommend-accept |
|
👋 @openjournals/dsais-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#3923, then you can now move forward with accepting the submission by compiling again with the command |
|
Sorry @rcurtin, I forgot to check the references. Could you please add the suggested DOIs above to your .bib file? |
@editorialbot generate pdf |
@editorialbot check references |
|
Ok @osorensen, I think I've got everything worked out. Let me know if there's anything else to do. Thanks! 👍 |
@editorialbot recommend-accept |
|
👋 @openjournals/dsais-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#3929, then you can now move forward with accepting the submission by compiling again with the command |
|
@editorialbot accept |
|
🐦🐦🐦 👉 Tweet for this paper 👈 🐦🐦🐦 |
🐘🐘🐘 👉 Toot 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... |
@sandeep-ps, @zhangjy-ge – 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 ✨ @rcurtin – 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:
|
Thanks everyone! |
Submitting author: @rcurtin (Ryan Curtin)
Repository: https://github.com/mlpack/mlpack
Branch with paper.md (empty if default branch): mlpack4_joss
Version: 4.0.1
Editor: @osorensen
Reviewers: @sandeep-ps, @zhangjy-ge
Archive: 10.5281/zenodo.7587252
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
@sandeep-ps & @zhangjy-ge, 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 @sandeep-ps
📝 Checklist for @zhangjy-ge
The text was updated successfully, but these errors were encountered: