-
-
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]: VOTCA: multiscale frameworks for quantum and classical simulations in soft matter #6864
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: |
👋🏼 @junghans @gomartini-collab @TariniHardikar this is the review thread for the paper. All of our communications will happen here from now on. As a reviewer, the first step is to create a checklist for your review by entering @editorialbot generate my checklist as the top of a new comment in this thread. These checklists contain the JOSS requirements. As you go over the submission, please check any items that you feel have been satisfied. The first comment in this thread also contains links to the JOSS reviewer guidelines. 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 openjournals/joss-reviews#REVIEW_NUMBER so that a link is created to this thread (and I can keep an eye on what is happening). Please also feel free to comment and ask questions on this thread. In my experience, it is better to post comments/questions/suggestions as you come across them instead of waiting until you've reviewed the entire package. We aim for reviews to be completed within about 2-4 weeks. Please let me know if any of you require some more time. We can also use EditorialBot (our bot) to set automatic reminders if you know you'll be away for a known period of time. Please feel free to ping me (@srmnitc ) if you have any questions/concerns, thanks again for the submission, and for thr reviews |
Hello @junghans, here are the things you can ask me to do:
|
|
|
We cited those 3 as suggested in their documents, no DOI available.
This one has no DOI either, and the repo is also currently unavailable, still investigating, but if that persists we might drop this citation. |
@junghans thanks! My recommendation would be add the |
That was already the case. |
The repo is back online. |
|
Review checklist for @gomartini-collabConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
This is a bit ambiguous: Human and animal research: If the paper contains original data research on humans subjects or animals, does it comply with JOSS's human participants research policy and/or animal research policy? If the paper contains no such data, please check this item. |
In this case, since there is on human/animal research, checking the box would be the appropriate response. |
Review checklist for @TariniHardikarConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@gomartini-collab and @TariniHardikar just a small reminder from my side about the review! Thanks once again for your efforts. |
I think I completed my review. |
Sorry, I was traveling! Will get my full review in by July 12th. |
thanks a lot @gomartini-collab . I see that there are some unchecked boxes. If it is not applicable to the paper, please check it. If you feel the code needs improvement, could you please open an issue at the code repository, and mention here. We would need all boxes checked to go forward. Thanks once again! |
This is the kind of things I am not sure: Data sharing: If the paper contains original data, data are accessible to the reviewers. If the paper contains no original data, please check this item. If I click here, I am accepting the paper has NO original data. This is how I understood this part. Now you asked me to click on it. I am very much confuse. |
Looks good to me. |
@junghans I believe the paper converter will add first initials when needing to distinguish multiple citations from the same first author; it should be able to handle the |
@kyleniemeyer thanks. In our case there is no distinguish needed, I think we are missing a middle initial somewhere, so let me see if that fixes it. |
Ok fixed in votca/votca#1153 |
Looks good now! |
@editorialbot generate pdf |
@editorialbot set master as branch |
Done! branch is now master |
@editorialbot generate pdf |
@junghans can you review and merge votca/votca#1154 ? It fixes some small issues I noticed. |
@kyleniemeyer done, thanks for fixing |
@kyleniemeyer thanks! |
@editorialbot generate pdf |
@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... |
@kyleniemeyer before closing, could you update the title? |
I did, sorry I missed before! |
Congratulations @junghans on your article's publication in JOSS! Many thanks to @gomartini-collab and @TariniHardikar for reviewing this submission, and @srmnitc 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! 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: @junghans (Christoph Junghans)
Repository: https://github.com/votca/votca
Branch with paper.md (empty if default branch): master
Version: v2024.1
Editor: @srmnitc
Reviewers: @gomartini-collab, @TariniHardikar
Archive: 10.5281/zenodo.12750697
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
@gomartini-collab & @TariniHardikar, 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 @srmnitc 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 @gomartini-collab
📝 Checklist for @TariniHardikar
The text was updated successfully, but these errors were encountered: