-
-
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]: Re-Envisioning Numerical Information Field Theory (NIFTy.re): A Library for Gaussian Processes and Variational Inference #6593
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: |
|
@Abinashbunty, @apizzuto — This is the review thread for the paper. All of our correspondence will happen here from now on. Thanks again for agreeing to participate! 👉 Please read the "Reviewer instructions & questions" in the first comment above, and generate your checklists by commenting 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 the review process to be completed within about 4-6 weeks but please try to make a start ahead of this as JOSS reviews are by their nature iterative and any early feedback you may be able to provide to the author will be very helpful in meeting this schedule. Please get your review started as soon as possible! |
Review checklist for @AbinashbuntyConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @apizzutoConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hello authors — thank you for a lovely piece of software! I enjoyed installing and playing around with this tool. Overall, I found the paper to be extremely clear, the API docs and installation directions were well done, and the tutorials were a really nice introduction to solving real problems with the software. I left a couple of tiny nits as issues in the repository, they are pretty much all documentation based, so hopefully they won't be hard to address. The only remaining technical parts of my review involve reproducing plots from the paper, which I will try to do in the next few days. I will only benchmark a few performance metrics just to make sure that things seem to line up with your performance plot. I will circle back once I've completed those parts (and I'll check off the remaining boxes in my review once the issues I opened up are addressed). Cheers! |
Many thanks for the review! I am very happy to hear that you enjoyed playing around with NIFTy.re! :) Thanks for filing these issues! They are all things that definitely still should be improved! I am currently in the midst of putting the finishing touches on my PhD thesis but I'll work on these issues as soon as possible, hopefully resolving them within the next week or so. Regarding the performance validation, feel free to use the [benchmarking script in the repo])https://github.com/NIFTy-PPL/NIFTy/blob/paper/paper/minimal_benchmark.py) to validate the results. I used it together with this visualization script to create the figure in the paper. |
Thanks for the constructive feedback! I incorporated all of your comments into the code :) |
@Abinashbunty Do you now approximately when you'll be able to review NIFTy.re? :) |
@Edenhofer Hi! Perhaps 2 more days. I'm executing some parts of the package and then the review will be over. 👍🏻 |
All of your fixes for the issues I opened were great, and I have completed the last items on my checklist. Everything looks good from my end. |
@Abinashbunty — I'll check in here too. Let me know if you have any blockers to finishing your review. (@Edenhofer — Please let me do the reviewer reminders in the future!) |
@editorialbot commands |
Hello @Abinashbunty, here are the things you can ask me to do:
|
Thanks @apizzuto and @Abinashbunty for reviewing the submission and for providing valuable and constructive feedback! Thanks also @dfm for guiding us through the review process and for your fixes! I'm currently on vacation without my Laptop for one more week but will incorporate the feedback and finalize the submission at the end of next week. |
Thanks for your all of your feedback! I've updated the references accordingly. Note, I left out the DOI if the journal paper doesn't have one even if there is a arXiv paper with a DOI. I left out the DOI as the cited paper technically has no unique identifier and the arXiv paper might be slightly different and/or outdated. I'm happy to add them in if the consensus is to include DOIs for arXiv papers in this case. I tagged the most recent release with v8.5.1 and pushed a release to zenodo (DOI 10.5281/zenodo.11441976). |
Hi @dfm I approve of the citing of the references to the respective research papers that do not have a DOI apart from the one in arXiv. And hence also approve of the changes implemented by the authors. I'm just going to run the bot command for checking references as a final report generation. 👍🏻 |
@editorialbot check references |
|
@editorialbot set 10.5281/zenodo.11441976 as archive |
Done! archive is now 10.5281/zenodo.11441976 |
@editorialbot set v8.5.1 as version |
Done! version is now v8.5.1 |
@editorialbot generate pdf |
@editorialbot recommend-accept |
|
|
👋 @openjournals/aass-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#5505, then you can now move forward with accepting the submission by compiling again with the command |
@editorialbot accept |
|
Ensure proper citation by uploading a plain text CITATION.cff file to the default branch of your repository. If using GitHub, a Cite this repository menu will appear in the About section, containing both APA and BibTeX formats. When exported to Zotero using a browser plugin, Zotero will automatically create an entry using the information contained in the .cff file. You can copy the contents for your CITATION.cff file here: CITATION.cff
If the repository is not hosted on GitHub, a .cff file can still be uploaded to set your preferred citation. Users will be able to manually copy and paste the citation. |
🐘🐘🐘 👉 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... |
@Abinashbunty, @apizzuto — Many thanks for your reviews here! JOSS relies upon the volunteer effort of people like you and we simply wouldn't be able to do this without you!! @Edenhofer — 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:
|
Submitting author: @Edenhofer (Gordian Edenhofer)
Repository: https://github.com/nifty-ppl/nifty
Branch with paper.md (empty if default branch): paper
Version: v8.5.1
Editor: @dfm
Reviewers: @Abinashbunty, @apizzuto
Archive: 10.5281/zenodo.11441976
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
@Abinashbunty & @apizzuto, 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 @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 ✨
Checklists
📝 Checklist for @Abinashbunty
📝 Checklist for @apizzuto
The text was updated successfully, but these errors were encountered: