Skip to content
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

minimum version of the survival dependency #11

Closed
davraam opened this issue May 31, 2022 · 0 comments
Closed

minimum version of the survival dependency #11

davraam opened this issue May 31, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@davraam
Copy link

davraam commented May 31, 2022

We got the following error with the ds.cox.zphSLMA function and we found out that this particular cohort had version 2.44-1.1 of the survival package. In another cohort with version 3.2-11 of the survival the function worked fine. So I suggest you to specify a minimum required version of the survival package in the DESCRIPTION file

[1] "Command 'cox.zphSLMADS("coxph_serverside", "km", FALSE, FALSE, TRUE)' failed on 'moba': Error while evaluating 'dsSurvival::cox.zphSLMADS("coxph_serverside", "km", FALSE, FALSE, TRUE)' -> Error in survival::cox.zph(fit = fit_model, transform = transform, terms = terms, : \n unused arguments (terms = terms, singledf = singledf)\n"

@neelsoumya neelsoumya self-assigned this Jun 1, 2022
neelsoumya added a commit that referenced this issue Jun 1, 2022
possible fix to issue #11 minimum dependency of survival package. The prototype of survival::cox.zph() changed in some version
@neelsoumya neelsoumya added the enhancement New feature or request label Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants