-
-
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]: Metrics As Scores: A Tool- and Analysis Suite and Interactive Application for Exploring Context-Dependent Distributions #4913
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 |
|
@mdhaber, @jstvssr: Thanks for agreeing to review. Please carry out your review in this issue by first creating a checklist ( If you have any questions or concerns please let me know. @jstvssr currently has a busy schedule, and the review is not expected to be completed before the end of January 2023. |
Review checklist for @mdhaberConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@mdhaber raised a couple of issues (opened in the repo directly) that we will begin to address next week. |
@MrShoenel Can you please give a status of the progress of resolving the raised issues? |
There were some unexpected delays on my side, but I had begun work on issues #4 (automated tests) and #2 (API documentation). I will complete some of it this week and reference the changes in these issues then. |
@editorialbot generate pdf |
Thanks for your patience. I have made substantial changes in the meantime (please check the README.md), to address all of @mdhaber's comments (thanks for these!). My to-do list is now empty :) Of course, I absolutely welcome additional comments :) |
@editorialbot generate pdf |
I got some more follow-up comments and issues from @mdhaber, all of which are addressed by now (but I suppose more is coming ;) ). |
@jstvssr: Do you have any critical points you want me to address? |
@mikldk You can see the status of my review at MrShoenel/metrics-as-scores#1. We've been steadily working through things since mid-December. Many individual issues have been addressed, but a few are still open. One of the review criteria is "Is the list of references complete...?" The paper cites some of MAS's software dependencies, but not all. How should the author decide which dependencies to cite? I am a maintainer of one such dependency, SciPy, which is used throughout MAS. Compared to other tools that are cited (e.g. Pymoo, which itself relies on SciPy), it seems to be used enough to deserve a citation, but I'm not sure that I can claim to be completely objective about this, so I'd appreciate your input. |
@editorialbot check references |
|
@editorialbot set 10.5281/zenodo.8202326 as archive |
Done! archive is now 10.5281/zenodo.8202326 |
@editorialbot set v2.8.0 as version |
Done! version is now v2.8.0 |
That one missing DOI came up previously. However, it is not valid, as it refers to a review of the book, not the book itself (which has no DOI). |
I found the same. |
@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#4443, then you can now move forward with accepting the submission by compiling again with the command |
@openjournals/joss-eics Note that the missing DOI is not correct for the book (it's for a review of the book). The book does apparently not have a DOI as far as @MrShoenel and I can find. |
If there is no DOI for the book (or any other item), you don't need to have one. This can proceed as is. |
Just checking, is there anything left I need to provide or address? |
@MrShoenel No! I will proceed with acceptance. |
@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... |
🎉🎉🎉 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 would like to thank all who were involved, especially the two reviewers @mdhaber and @kostiantyn-kucher, who helped me to significantly improve our work. I also want to thank @mikldk for a very efficient editing process. |
Submitting author: @MrShoenel (Sebastian Hönel)
Repository: https://github.com/MrShoenel/metrics-as-scores
Branch with paper.md (empty if default branch): paper
Version: v2.8.0
Editor: @mikldk
Reviewers: @mdhaber, @kostiantyn-kucher
Archive: 10.5281/zenodo.8202326
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
@mdhaber & @jstvssr, 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 @mikldk 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 @mdhaber
📝 Checklist for @kostiantyn-kucher
The text was updated successfully, but these errors were encountered: