-
-
Notifications
You must be signed in to change notification settings - Fork 39
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]: TorchSurv: A Lightweight Package for Deep Survival Analysis #7341
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:
|
Software report:
Commit count by author:
|
|
Paper file info: 📄 Wordcount for ✅ The paper includes a |
License info: ✅ License found: |
@melodiemonod, @WeakCha, @LingfengLuo0510, This is the review thread for the paper. All of our communications will happen here from now on. Please read the "Reviewer instructions & questions" in the first comment above. For @WeakCha, @LingfengLuo0510 - Both reviewers have checklists at the top of this thread (in that first comment) with the JOSS requirements. As you go over the submission, please check any items that you feel have been satisfied. There are also links to the JOSS reviewer guidelines. As you are probably already aware, 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 #7341 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 the review process to be completed within about 4-6 weeks but please make a start well ahead of this as JOSS reviews are by their nature iterative and any early feedback you may be able to provide to the author will be very helpful in meeting this schedule. Thanks in advance and let me know if you have any questions!! |
Review checklist for @WeakChaConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Thank you so much for this package, this work looks super interesting! Here are my initial comments: For the paper:
For your tutorial (https://opensource.nibr.com/torchsurv/notebooks/introduction.html), could you please show what Thank you so much for your work! I am looking forward to reviewing your updated paper and package! |
Dear @WeakCha, Thank you for your early comments. Regarding your last one ( For the first notebook ( # PyTorch boilerplate - see https://github.com/Novartis/torchsurv/blob/main/docs/notebooks/helpers_introduction.py
from helpers_introduction import Custom_dataset, plot_losses I would discourage adding the code directly in the notebook because it would overcrowd the notebook:
If our suggestion isn't satisfying, I can look into other option (one could be to have the cell hidden, but requires another package thus making it rely on more dependancies) |
Hello everyone, We modified the package according to the first comments. Please use the Thanks |
Hi @tcoroller Thanks for the update! My further comments about your tutorial are listed below:
seems to emphasize the word "preferred" with a code font. I guess you want to use bold or italic instead? There are many words like this in your tutorials.
I will get back to you with paper reviews if any. Thanks a lot for your work! |
@editorialbot generate pdf |
@tcoroller @melodiemonod @kanishkan91 It seems that your paper is unchanged. I guess this may be an error as I found an updated version here: https://github.com/Novartis/torchsurv/blob/58-comments-joss-241018/paper/paper.md, but I cannot get this when following the prompt by the editoralbot. |
dear @WeakCha , Thank you for your comments, please see my replies below:
Thank you, |
To expand on point 4 about model comparison: our package is designed to provide all the necessary functions to assist in model fitting (i.e.., with the negative log-likelihood function) and evaluate its performance (i.e., with evaluation metric functions), rather than providing functions for directly fitting a model. Unlike packages like |
@tcoroller @melodiemonod Thank you so much for your reply and I agree with most of them. I would wait to review
Again, thanks a lot for your work! |
@editorialbot commands |
Hello @melodiemonod, here are the things you can ask me to do:
|
@WeakCha removed from the reviewers list! |
@rich2355 - This should be fixed (Your new name is added). To be safe, could you regenerate your checklist and complete it? Its just a safety measure. |
@tcoroller , @melodiemonod I have recommended this for acceptance now. I will be reading through the paper for typos etc. shortly. The AEiC in this submission track will review shortly and if all goes well this will go live soon! Big thank you to @rich2355 and @XinyiEmilyZhang for reviewing! JOSS is volunteer run and relies heavily on researchers such as yourself. |
Review checklist for @rich2355Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@kanishkan91 Done! |
@editorialbot generate pdf 🔍 checking out the following:
|
@melodiemonod @tcoroller - I guess one last minor comment from my side. The paper is a bit too long. We generally only recommend 3-4 pages of text for a software paper. I recommend removing the "Comprehensive example" starting page 4. This is probably covered in your documentation. |
Dear @kanishkan91, thank you for your feedback. We are open to removing the example, provided that @rich2355, who originally suggested it, agrees. Please let us know your thoughts so we can proceed accordingly. |
I agree!
…On Fri, Dec 20, 2024 at 2:39 PM Mélodie Monod ***@***.***> wrote:
Dear @kanishkan91 <https://github.com/kanishkan91>, thank you for your
feedback. We are open to removing the example, provided that @rich2355
<https://github.com/rich2355>, who originally suggested it, agrees.
Please let us know your thoughts so we can proceed accordingly.
—
Reply to this email directly, view it on GitHub
<#7341 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALQQQR2Y37ZWSJLPLDHHGMD2GRW5ZAVCNFSM6AAAAABPVX3BWKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJXGYYTMNBVHA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@editorialbot generate pdf |
Dear @kanishkan91, we have removed the comprehensive example from the manuscript. Please see the updated version above. |
Thanks. The EiC will take care of this paper here forward. |
👋 @melodiemonod - I just need you to address the following before I move to accept this for publication: In the archive:
In the paper:
After you make these changes let me know. Thanks. |
@editorialbot generate pdf |
Dear @crvernon, thank you for your comments. All the changes, on the Zenodo and on the manuscript, have been made. |
@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 👈 🐘🐘🐘 |
🦋🦋🦋 👉 Bluesky post 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 new publication @melodiemonod! Many thanks to @kanishkan91 for editing and @XinyiEmilyZhang and @rich2355 for your time, hard work, and expertise!! JOSS wouldn't be able to function nor succeed without your efforts. Please consider becoming a reviewer for JOSS if you are not already: https://reviewers.joss.theoj.org/join |
🎉🎉🎉 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:
|
We are delighted by this news. Many thanks for your helpful reviews and for ensuring a swift reviewing process. Wishing you all lovely holidays. |
Submitting author: @melodiemonod (Mélodie Monod)
Repository: https://github.com/Novartis/torchsurv
Branch with paper.md (empty if default branch): main
Version: v0.1.4
Editor: @kanishkan91
Reviewers: @XinyiEmilyZhang, @rich2355
Archive: 10.5281/zenodo.14517267
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
@WeakCha & @LingfengLuo0510, 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 @kanishkan91 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 @XinyiEmilyZhang
📝 Checklist for @rich2355
The text was updated successfully, but these errors were encountered: