Track tasks and feature requests
Join 36 million developers who use GitHub issues to help identify, assign, and keep track of the features and bug fixes your projects need.
Sign up for free See pricing for teams and enterprises[REVIEW]: lifelines: survival analysis in Python #1317
Comments
whedon
assigned
trallard
Mar 12, 2019
whedon
added
the
review
label
Mar 12, 2019
whedon
referenced this issue
Mar 12, 2019
Closed
[PRE REVIEW]: lifelines: survival analysis in Python #1305
This comment has been minimized.
This comment has been minimized.
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @becarioprecario, it looks like you're currently assigned as the reviewer for this paper 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:
|
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Please use this issue to comment, make suggestions and discuss anything related to the submission. If you need to request changes or go in detail on something it is advisable to make an issue on the software repo https://github.com/camdavidsonpilon/lifelines and reference this issue to keep track of the process. If any of you or @CamDavidsonPilon has any questions at any point feel free to ping me. Happy reviewing |
This comment has been minimized.
This comment has been minimized.
Hi @becarioprecario and @sunhwan I have not seen much activity on this repo for a bit so I was wondering if there is anything you need help with? |
This comment has been minimized.
This comment has been minimized.
Thanks for reminding me. I'll work on this soon. |
This comment has been minimized.
This comment has been minimized.
Thanks for reminding me. I’ll try to look into this asap.
Virgilio
… El 1 abr 2019, a las 19:37, Tania Allard ***@***.***> escribió:
Hi @becarioprecario and @sunhwan I have not seen much activity on this repo for a bit so I was wondering if there is anything you need help with?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
This comment has been minimized.
This comment has been minimized.
@trallard I went through the checklist and I was satisfied with most of the item in the list. I found a few problems running some script/file. I left a comment in the repository. CamDavidsonPilon/lifelines#703 This package looks great and I'm okay to accept after the author address the minor issue. Are reviewers expected to write a separate review? |
This comment has been minimized.
This comment has been minimized.
Fantastic @sunhwan thanks a lot for your review. @becarioprecario is there anything we can help with to move this review forward ? |
whedon
assigned
becarioprecario and
sunhwan
May 6, 2019
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Jun 3, 2019
Hi @becarioprecario, just wanted to check on your review. |
This comment has been minimized.
This comment has been minimized.
ping! @becarioprecario do you have any updates on the review? |
This comment has been minimized.
This comment has been minimized.
Hi,
I have been a bit stuck with other stuff. Please, may I ask you to send me the guidelines again on how to make the review.
Thanks!
Virgilio
|
This comment has been minimized.
This comment has been minimized.
Hi @becarioprecario the checklist for the review is at the top of this issue 👆🏼 For reference the guidelines for reviewers are located here: https://joss.theoj.org/about#reviewer_guidelines |
This comment has been minimized.
This comment has been minimized.
Hi, I am done with my review now. The package and its documentation are excellent!! I had some troubles with the installation, but due to my local configuration. Installing the package with pip run smoothly. Best, Virgilio |
This comment has been minimized.
This comment has been minimized.
Thanks for your time and reviews @becarioprecario and @sunhwan 🙏🏼 @CameronDevine I can confirm your paper has been accepted now!
|
trallard
added
the
accepted
label
Jul 1, 2019
This comment has been minimized.
This comment has been minimized.
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@whedon check references |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
@CamDavidsonPilon, I just wanted to make sure you saw this since I was mentioned by mistake. |
This comment has been minimized.
This comment has been minimized.
HI @CamDavidsonPilon I was wondering if you'd make any progress with the last few tasks for acceptance? |
This comment has been minimized.
This comment has been minimized.
CamDavidsonPilon
commented
Jul 8, 2019
hi @trallard, I have addressed the missing DOI. Anything else? Is there a list I missed? |
This comment has been minimized.
This comment has been minimized.
@CamDavidsonPilon Can you please paste the DOI here so I can set it as the archive? As well as the latest version of the package and I can finalise the acceptance |
This comment has been minimized.
This comment has been minimized.
CamDavidsonPilon
commented
Jul 18, 2019
Missing DOI entry: https://github.com/CamDavidsonPilon/lifelines/blob/master/paper/paper.bib#L144-L153 Latest version is 0.21.1 |
This comment has been minimized.
This comment has been minimized.
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@whedon check references |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
@whedon set 0.21.1 as version |
This comment has been minimized.
This comment has been minimized.
OK. 0.21.1 is the version. |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
@whedon set 10.5281/zenodo.805993 as archive |
This comment has been minimized.
This comment has been minimized.
OK. 10.5281/zenodo.805993 is the archive. |
This comment has been minimized.
This comment has been minimized.
@whedon accept |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
Check final proof If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#847, then you can now move forward with accepting the submission by compiling again with the flag
|
This comment has been minimized.
This comment has been minimized.
@CamDavidsonPilon Lifelines is ready for acceptance to JOSS, let's pass this over to our editor in chief to proceed @becarioprecario and @sunhwan thanks for the time and effort put in this review. Your contribution to JOSS is deeply appreciated. 🙏🏼🙌🏼 @openjournals/joss-eics this paper is ready for acceptance |
openjournals
deleted a comment from
whedon
Jul 19, 2019
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Jul 19, 2019
@trallard - you've jumped the gun slightly. Let's say that you believe Lifelines is ready to be accepted to JOSS - now @openjournals/joss-eics need to do some final checks. |
This comment has been minimized.
This comment has been minimized.
ooops wrong wording you are correct!!! sorry! |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Jul 19, 2019
@CamDavidsonPilon - please accept the changes in CamDavidsonPilon/lifelines#781, or disagree with the ones that are not adding needed spaces :) |
whedon commentedMar 12, 2019
•
edited
Submitting author: @CamDavidsonPilon (Cameron Davidson-Pilon)
Repository: https://github.com/camdavidsonpilon/lifelines
Version: 0.21.1
Editor: @trallard
Reviewer: @becarioprecario, @sunhwan
Archive: 10.5281/zenodo.805993
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) 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
@becarioprecario & @sunhwan, 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.theoj.org/about#reviewer_guidelines. Any questions/concerns please let @trallard know.
Review checklist for @becarioprecario
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?Review checklist for @sunhwan
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?