-
-
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]: Simmate: a framework for materials science #4364
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 |
Failed to discover a |
Thanks @rkurchin. This project is quite large (20k lines of Python alone), so it may take some time to explore the functionality fully. |
FYI, @editorialbot commands need to be the first thing in a command |
Review checklist for @utfConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Thanks @danielskatz. |
@rkurchin @bocklund @utf Thank you for volunteering to edit and review our package! We're excited for the feedback. I just wanted to place two things upfront to help with the review. First, our test suite is currently at 84% coverage, and here's the report: Second, I just wanted to repeat my comment from the pre-review thread: |
Review checklist for @bocklundConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@rkurchin Thanks for the pings. I had some travel the last few weeks, but getting back to this now! |
@rkurchin is an explicit "Statement of Need" section required for the paper? (question here). |
Thanks for checking. @jacksund, if you could split those up I think that would be preferable. While the rendered version of that paper (from 2017) has them combined, if you look at the updated source in the docs, it has separate headers, so I think that's what's recommended currently. |
@editorialbot generate pdf |
Thanks everyone! Authors, I'll do an editorial pass over the manuscript and send any comments shortly. In the meantime, the next steps for you are:
|
Some very minor editorial comments (this is by far the smallest number of these I've ever had, so 👏 from my editor hat 🎩):
|
@editorialbot generate pdf |
@rkurchin awesome, thanks for editing the manuscript! 🥳 I've addressed your comments in the newest version. Also, I'll make a new release + create a DOI on Zenodo once I get the chance. There was a libwebp release made a few hours ago that prevents any conda-forge updates and causes my CI for MacOS to fail. They already have an issue and PR open, so it should hopefully have a fix soon. |
@editorialbot set 0.7.1 as version |
Done! version is now 0.7.1 |
@editorialbot set 10.5281/zenodo.6863068 as DOI |
I'm sorry human, I don't understand that. You can see what commands I support by typing:
|
@editorialbot set 10.5281/zenodo.6863068 as archive |
Done! Archive is now 10.5281/zenodo.6863068 |
@editorialbot recommend-accept |
|
|
👋 @openjournals/joss-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#3385, then you can now move forward with accepting the submission by compiling again with the command |
@editorialbot accept |
|
🐦🐦🐦 👉 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 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: @jacksund (Jack Sundberg)
Repository: https://github.com/jacksund/simmate
Branch with paper.md (empty if default branch): joss
Version: 0.7.1
Editor: @rkurchin
Reviewers: @bocklund, @utf
Archive: 10.5281/zenodo.6863068
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
@bocklund & @utf, 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 @rkurchin 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 @utf
📝 Checklist for @bocklund
The text was updated successfully, but these errors were encountered: