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]: Surprise: A Python library for recommender systems #2174
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @sara-02, @ejhigson it looks like you're currently assigned to review this paper 🎉. ⭐ Important ⭐ If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews 😿 To fix this do the following two things:
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:
|
PDF failed to compile for issue #2174 with the following error: Can't find any papers to compile :-( |
@whedon generate pdf from branch joss_paper |
|
@sara-02, @ejhigson - please carry out your review in this issue by updating the checklist above and giving feedback in this issue. The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html Any questions/concerns please let @terrytangyuan know. |
Hi @NicolasHug - congratulations on making a really awesome and very popular software package! The install and examples all worked perfectly for me. The API documentation is very thorough too. I have couple of questions:
|
Thanks for the review @ejhigson I didn't include a comparison with other similar packages. Should I? There are instructions to run the tests in the contributing guidelines: https://github.com/NicolasHug/Surprise/blob/master/CONTRIBUTING.md . Also note that a CI instance is ran at every commit, there's a "build passing badge in the readme. |
@NicolasHug If not an exhaustive comparison then even an overview would do. I assume most projects on Recommendation Systems have been written as part of research projects and not maintained after that. |
I agree with @sara-02 that an overview would be useful.
I think this is useful information to include in the paper.
Aha sorry I missed this! With these instructions all the tests work perfectly for me. |
👋 @NicolasHug - today we reopened JOSS for new submissions and are checking in on our existing reviews. How are you getting on making these updates to your paper? |
@terrytangyuan I have updated my reviews, waiting for @NicolasHug to update the paper w.r.t |
👋 @NicolasHug - could you update us on when you might be able to address @sara-02's request above? ☝️ |
Thank you for the reviews so far and apologies for the silence! I'm still short on time ATM but it will get better soon and should be able to address the reviews by mid |
@whedon generate pdf from branch joss_paper |
|
@whedon generate pdf from branch joss_paper |
|
Thanks again everyone! I have updated the paper with a section on related libraries, as suggested by @sara-02 . |
👋 @sara-02 Could you review again when you get a chance? Thanks! |
@terrytangyuan @NicolasHug LGTM 🥳 |
@NicolasHug Minor suggestion for the first line of the paper can we replace the phrase |
Thanks @sara-02 , and good catch. I have modified the first sentence accordingly. |
|
PDF failed to compile for issue #2174 with the following error: error: pathspec 'joss-paper' did not match any file(s) known to git. |
@whedon generate pdf from branch joss_paper |
|
👋 @Kevin-Mattheus-Moerman - is this ready to go? |
@NicolasHug - in the references, I think "ECML pkdd workshop" should be "ECML PKDD Workshop" Similarly the booktitle for Gantner should be changed to title case (by protecting it in the .bib file with {}s After these are fixed, please regenerate the pdf |
@danielskatz yes this looks good to me. I see you raised a minor issue ☝️, once that is resolved it looks good to go. You can process acceptance if you like or let me know if you want me to. |
I'll do it once this issue is resolved - thanks |
@whedon generate pdf from branch joss_paper |
|
Thanks @danielskatz , capitalization is hopefully fixed now |
@whedon accept |
|
PDF failed to compile for issue #2174 with the following error: Can't find any papers to compile :-( |
@whedon accept from branch joss_paper |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#1628 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#1628, then you can now move forward with accepting the submission by compiling again with the flag
|
@whedon accept deposit=true from branch joss_paper |
|
🐦🐦🐦 👉 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... |
Thanks to @sara-02 and @ejhigson for reviewing and @terrytangyuan for editing! Congratulations to @NicolasHug (Nicolas Hug)!! |
🎉🎉🎉 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! 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 so much everyone! 🎉 |
Submitting author: @NicolasHug (Nicolas Hug)
Repository: https://github.com/NicolasHug/Surprise
Version: v1.1.1
Editor: @terrytangyuan
Reviewer: @sara-02, @ejhigson
Archive: 10.5281/zenodo.3959188
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
@sara-02 & @ejhigson, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @terrytangyuan know.
✨ Please try and complete your review in the next two weeks ✨
Review checklist for @sara-02
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @ejhigson
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: