-
-
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]: BART-Survival: A Bayesian machine learning approach to survival analyses in Python #7213
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: |
@turgeonmaxime and @WeakCha - Thank you for agreeing to review this submission. This is the review thread for the paper. All of our communications will happen here from now on. As mentioned above, you can use the command There are also links to the JOSS reviewer guidelines (https://joss.readthedocs.io/en/latest/reviewer_guidelines.html) 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, reviewers are encouraged to submit issues and pull requests on the software repository. When doing so, please mention #7213 so that a link is created to this thread for visibility. 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 you require additional time. We can also use editorialbot (our bot) to set automatic reminders if you know you'll be away for a known period. Please feel free to ping me (@mahfuz05062) if you have any questions/concerns. |
Review checklist for @WeakChaConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Thanks for inviting me for reviewing this paper and I like the idea of extending BART from R to Python. I have taken a quick overview of the paper and the software and I have these comments. For the paper:
For the software:
I may give more comments when I have chance to test this software on my PC, but in general, the idea is interesting, although some refinement may be needed. Please feel free to let me know if you have any questions, or correct me if I am wrong, thanks! |
@WeakCha As far as point 10. We did do an extensive validation analysis for our method with simulations and real data, in which we compared our algorithm to the R based algorithm. However, I was under the impression that JOSS did not necessarily want this type of analysis in the JOSS papers. We were planning to put together the validation study in a separate paper. I could provide an example comparing our algorithm and the R algorithm using a simulated dataset if you think that would be sufficient. Or if you have other thoughts? |
@twj8CDC Thank you for the reply and you mentioned a good point. Regardless of any preference of JOSS I would simply add a few sentences saying something like "based on our evaluation test, our software successfully reproduces the functionalities in the R version". But I have no idea about this kind of preference. @mahfuz05062 Do you have thoughts on this? If this is not necessarily shown on JOSS, you do not need to modify anything on the paper. But it would be great if you could provide an example of comparison, and put these comparison code in your github example folder. (could be a py file with the compared R code commented out) Definitely open to other thoughts! |
Review checklist for @turgeonmaximeConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@mahfuz05062 sounds great, I will add an example of this sort to the paper. |
Hi @twj8CDC ! Out of curiosity, have you generated the PDF for us to further review? |
Hi @WeakCha. Sorry for the delay, but yes I did just update the pdf! Thanks |
@editorialbot generate pdf |
@mahfuz05062 @twj8CDC Thank you so much for editing your paper, I agree that your paper looks much better! Here is a list of my new comments:
I will take a look at the github repo soon and will get back to you if I have more comments. Thanks a lot for your editing, I think the paper now is much richer and more detailed! |
@twj8CDC Hi, I come back for the GitHub repo! For your Github repo,
|
Hi @twj8CDC, can you make necessary change according to the suggestions from @WeakCha ? |
@turgeonmaxime I haven't seen much activity from your side. Is there anything that I can help with or do you need more time? |
@mahfuz05062 Yes I will complete these changes over the next couple days! Thanks |
Sorry @mahfuz05062, I've been busy with work, but I'm actually reading the newest version right now! I should have some comments to share later today. |
@twj8CDC Thanks for your updates to the paper. I think one crucial point that is still lacking from the manuscript is what role Similarly, I think you did a good job explaining the need for Beyond that, I have a few more comments, some general and some more targeted:
I'll have a look at the repo itself and report back. |
@twj8CDC One more thing: could you add a short description of the |
Done! version is now v.0.1.1 |
@editorialbot generate pdf |
@editorialbot generate checklist |
I'm sorry human, I don't understand that. You can see what commands I support by typing:
|
Review checklist for @rich2355Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@mahfuz05062 Done! |
@editorialbot generate pdf |
@editorialbot remove WeakCha from reviewers |
I can't add that reviewer: WeakCha is not a username |
@editorialbot remove @WeakCha from reviewers |
@WeakCha removed from the reviewers list! |
@editorialbot generate pdf |
@editorialbot check references |
|
Thanks! I removed your old name from reviewers list. We should be good now. |
@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#6282, then you can now move forward with accepting the submission by compiling again with the command |
Hi @mahfuz05062, Do you know how long it typically take for the publication process to be finalized? Thanks |
@twj8CDC I have seen the finalization process to be completed within 1-2 weeks. However, as the last couple of weeks were holidays, it may be slower than usual. The next step is the track editor will go through the article and may provide some suggestions for modification, and then officially approve the publication. But, yes, you don't have anything to do until then. |
@twj8CDC as AEiC for JOSS I will now help to process this submission for acceptance in JOSS. Below are some final checks, some of which may require your attention: Checks on repository
Checks on review issue
Checks on archive
Checks on paper
Remaining points:As you can see, most seems in order, however the below are some points that require your attention 👇 :
|
@Kevin-Mattheus-Moerman is there anyway to make an exception on the word limit? I was asked by the reviewers to extend the content to the length the paper is now. Feels like a waste of effort to drop these sections at this point. |
Hi @twj8CDC, so sorry for this, I was not notified or aware of the word limit. As suggested by the AEiC, it would be great to move these 2 sections to a separate link with your documentation. Moving these sections out of paper does not mean throwing them away, people will still be interested in your method and demonstration and will refer to your documentation. |
@rich2355 @Kevin-Mattheus-Moerman, No worries, just thought I would ask. I will work on this change. Thanks! |
Submitting author: @twj8CDC (Jacob Tiegs)
Repository: https://github.com/CDCgov/BART-Survival
Branch with paper.md (empty if default branch):
Version: v.0.1.1
Editor: @mahfuz05062
Reviewers: @turgeonmaxime, @rich2355
Archive: 10.5281/zenodo.14105188
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
@turgeonmaxime & @WeakCha, 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 @mahfuz05062 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 @turgeonmaxime
📝 Checklist for @WeakCha
📝 Checklist for @rich2355
The text was updated successfully, but these errors were encountered: