-
-
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]: easyunfold: A Python package for unfolding electronic band structures #5974
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 |
|
@srmnitc and @awvwgk thanks for your help reviewing this work! This is where the review happens. I kindly ask you to now formally start the review. Follow the instructions above ☝️ to generate a check box list for yourself here to guide you through the process. Let me know if you have any questions. |
@zhubonan this is where the review takes place. Please keep an eye out for comments here from the reviewers, as well as any issues opened by them on your software repository. I recommend you aim to respond to these as soon as possible, and you can address them straight away as they come in if you like, to ensure we do not loose track of the reviewers. |
Review checklist for @srmnitcConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @awvwgkConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hi @mbarzegary I have started the review, and opened two issues on the repo already. I will finish the reminder of my review this week. |
@mbarzegary I have now finished my review. The package is in a really good state, with adequate documentation and example use cases to get a user started. It has already been used in a number of publications, and therefore definitely would make a good addition to JOSS. I only have three points that I need to check, for which I have opened issues in the repository: Automated tests, Community guidelines, and references. Addressing these issues would then complete the review. Thanks to the developers for this nice package! |
I finished my review as well, easyunfold provides a comprehensive documentation with examples, tutorials, theory, and technical reference. The repository provides all the necessary parts to install the package and to follow the documentation. Besides the already raised issues, I opened a request to evaluate the need for tailored implementation of IO interfaces to Vasp and Castep, especially with a standard package like ASE which can handle and extract band structures already (see SMTG-Bham/easyunfold#41). Otherwise this package is good to publish in JOSS. |
@mbarzegary Sure, sorry for the delay and @srmnitc and @awvwgk thanks for review this package. |
@editorialbot generate pdf |
@editorialbot check references |
|
👋 @openjournals/bcm-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#4870, then you can now move forward with accepting the submission by compiling again with the command |
@zhubonan as AEiC I will now help to process this work for acceptance in JOSS. I have checked your repository, this review, the archive link, and the paper. Most seems in order. However, I do have the below points that require your attention.
Has now been published as a paper. Please use this reference rather than the preprint: https://doi.org/10.1038/s41597-020-00638-4 (unless the pre-print version is intended to be cited e.g. if it is different). |
@editorialbot set 10.5281/zenodo.10396925 as archive |
Done! archive is now 10.5281/zenodo.10396925 |
@editorialbot generate pdf |
@editorialbot set v0.3.5 as version |
Done! version is now v0.3.5 |
@editorialbot check references |
|
@zhubonan as an author (who happens to also be an editor) please refrain from using the editor commands here. |
@editorialbot set 10.5281/zenodo.10510884 as archive |
Done! archive is now 10.5281/zenodo.10510884 |
@zhubonan please edit the archive listed license to match your software license. Note also that I updated the archive link here to use the one that relates to the new version |
Err sorry...
Sure, I have just updated the archive's licence to MIT to match the code. |
@openjournals/dev can we make it that an author cannot run the editor command for the issue they are listed as author for (such that the commands do not work even if the submitting author happens to be a JOSS editor?)? |
@zhubonan thanks for making those changes. I think all looks good now, so we can proceed. |
@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... |
@zhubonan congratulations on this JOSS paper!!! Thanks for editing @mbarzegary !! |
🎉🎉🎉 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:
|
I understand the request, but I think this is best handled with training/guidelines rather than code. |
Submitting author: @zhubonan (Bonan Zhu)
Repository: https://github.com/SMTG-UCL/easyunfold
Branch with paper.md (empty if default branch): paper
Version: v0.3.5
Editor: @mbarzegary
Reviewers: @srmnitc, @awvwgk
Archive: 10.5281/zenodo.10510884
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
@srmnitc & @awvwgk, 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 @mbarzegary 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 @srmnitc
📝 Checklist for @awvwgk
The text was updated successfully, but these errors were encountered: