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]: stanscofi and benchscofi: a new standard for drug repurposing by collaborative filtering #5973
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 |
|
Hey @jaybee84, @abhishektiwari 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
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 #5973 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 (@Nikoleta-v3) if you have any questions/concerns. 😄 🙋🏻 |
Review checklist for @abhishektiwariConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @jaybee84Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hey @abhishektiwari and @jaybee84 👋🏻, any updates on your reviews? 😄 |
I will share initial feedback early next week. |
Thank you 🙏🏻 and thank you again for your time! |
@Nikoleta-v3 Thanks for the reminder. I will also complete it within the next week. |
@clreda Initial comment on structure of Paper itself. — Just for consistency purposes, the |
@clreda I spent a few hours trying to install the package but neither Can you share what OS is supported by stanscofi? Update: Added GitHub issue with instructions to run |
Dear @abhishektiwari, thanks a lot for accepting to review this manuscript! I have tried to address the issue in the installation, please let me know how it goes. Thank you very much for the instructions to run on Docker, I will add them to the repository and documentation! EDIT: in commit ff312993 As for the comments on the manuscript itself, (I will address those concerns this week) edit: please refer to the newest version of the paper. |
@editorialbot generate pdf |
Thanks, @clreda I have now worked my way through the I am now having issue with benchscofi due it's restrictive Created ticket here: RECeSS-EU-Project/benchscofi#1 |
I have replied to your issue. I believe the best course of action is to provide for users the alternative installation guide, which relies on pip dependency conflict solver, and keep the current one to ensure smooth automated tests. Please feel free to mention other possibilities which might improve user functionality |
Hello @abhishektiwari @jaybee84👋🏻, any updates regarding your reviews? @clreda, I can see that RECeSS-EU-Project/stanscofi#2 is now closed. In the future, could you indicate the commit hash (or even better, the pull request) that resolved the issue? It will make reviewers' life a bit easier 😄 |
@Nikoleta-v3 I will revisit paper again, and expect final feedback by end of week. |
@clreda Looked at revised paper structure, still not sold on current summary. I generally think software paper summary as an elevator pitch - problem statement, solution, key value proposition, any additional detail for target audience. Revised version mentions the problem statement but not the solution. |
|
|
ID ref-luo2018computational already defined |
Dear @Nikoleta-v3 I have just fixed the reference issue mentioned above (commit d7bdc43). The process should end up successfully now. |
Thank you; that was fast 😄 |
@editorialbot recommend-accept |
|
|
👋 @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#4938, then you can now move forward with accepting the submission by compiling again with the command |
@clreda as AEiC I will now help to process this work for acceptance in JOSS. Since this concerns two packages, i.e. Please can you complete the following:
In addition:
|
@editorialbot set v.2.0.0, v.2.0.0 as version |
Done! version is now v.2.0.0, v.2.0.0 |
Dear @Kevin-Mattheus-Moerman, thanks for your help. Release version for I agree with your suggestion, and I have changed the title accordingly (commit c1e0260) in both the paper and the Zenodo archive. Please let me know if there is anything else needed. |
@editorialbot generate pdf |
@editorialbot set 10.5281/zenodo.10561760 as archive |
Done! archive is now 10.5281/zenodo.10561760 |
@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... |
Congralations on this publication @clreda ! Thanks for editing @Nikoleta-v3 And a special thank you to the reviewers: @jaybee84, @abhishektiwari |
🎉🎉🎉 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:
|
Thank you very much for coordinating the publication @Kevin-Mattheus-Moerman and a lot of thanks to @Nikoleta-v3, @jaybee84 and @abhishektiwari for their help in considerably improving the use of the softwares! |
Submitting author: @clreda (Clémence Réda)
Repository: https://github.com/RECeSS-EU-Project/stanscofi/
Branch with paper.md (empty if default branch):
Version: v.2.0.0, v.2.0.0
Editor: @Nikoleta-v3
Reviewers: @jaybee84, @abhishektiwari
Archive: 10.5281/zenodo.10561760
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
@jaybee84 & @abhishektiwari, 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 @Nikoleta-v3 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 @abhishektiwari
📝 Checklist for @jaybee84
The text was updated successfully, but these errors were encountered: