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

[Question]: More information needed about how univariate summarize_coxph() inputs are passed to survival::coxph() #1267

Open
3 tasks done
chelseadickens opened this issue Jul 12, 2024 · 10 comments
Labels
question Further information is requested

Comments

@chelseadickens
Copy link

chelseadickens commented Jul 12, 2024

What is your question?

  1. In trying to reproduce the estimates produced by the examples for summarize_coxph() in Cox proportional hazards regression using survival::coxph(), @adcascone and I were able to deduce that it's necessary to supply ties = "exact" as input. Is it possible to add details about this in function's documentation?
  2. Is it possible to add details about how the univariate and multivariate use cases differ in the documentation? From what we can tell, the univariate case takes any covariates and passes them to sequential models fitted with survival::coxph() (e.g., in the examples shown on the website, the first model contains only ARM as a predictor, the second model contains ARM + COVAR1 as predictors, and the third model contains ARM + COVAR2 as predictors, like below). Then the estimates for ARM only are extracted and presented in the outputs. However, for the multivariate use case, all predictors and covariates are passed to a single model. Is that correct?
coxph_summary1 <- summary(survival::coxph(Surv(TIME, STATUS) ~ ARM, 
                                          data = data.frame(dta_bladder), ties = "exact"))
coxph_summary1

coxph_summary2 <- summary(survival::coxph(Surv(TIME, STATUS) ~ ARM + COVAR1, 
                                          data = data.frame(dta_bladder), ties = "exact"))
coxph_summary2

coxph_summary3 <- summary(survival::coxph(Surv(TIME, STATUS) ~ ARM + COVAR2, 
                                          data = data.frame(dta_bladder), ties = "exact"))
coxph_summary3
  1. In addition, we're a bit confused why the output from summarize_coxph() provides the hazard ratios and associated CIs/pvalues for the ARM term but the labels would imply the hazard ratios are from the covariates (indicated by "A Covariate Label" and "Sex (F/M)". Is this done because there is a primary interest in the hazard ratios for the ARM variable (thus only the corresponding hazard ratios are shown and no outputs for the covariates that are supplied)?

tern_coxph

Thank you!

Code of Conduct

  • I agree to follow this project's Code of Conduct.

Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines.

Security Policy

  • I agree to follow this project's Security Policy.
@chelseadickens chelseadickens added the question Further information is requested label Jul 12, 2024
@shajoezhu
Copy link
Contributor

Dear @chelseadickens , hope you are well. Could you please repost the question to the StackOverflow, and nest-tern and nest-rtables. Thanks! We will be able to track the answers there.

@chelseadickens
Copy link
Author

Hi @shajoezhu, thanks for the reply. Hope you are well also. I tried posting to StackOverflow and tagging nest-tern and nest-rtables. However, nest-tern is not a current tag and requires 1500 reputation to create a new tag. In addition, I'm not able to upload the images supplied in my GitHub issue, as that requires 10 reputation to include images, so I tried removing the image. Then went to add the nest-tern tag and was faced with not having enough reputation also. See below.

For new users, using StackOverflow for package-specific questions will likely be somewhat limiting.

image

image

@shajoezhu
Copy link
Contributor

hi @chelseadickens , thanks for the reply, we will create the tag for "nest-tern" asap. Hi @ddsjoberg , I was wondering if you could help us to create the tag please. Thanks!

@ddsjoberg
Copy link

hi @chelseadickens , thanks for the reply, we will create the tag for "nest-tern" asap. Hi @ddsjoberg , I was wondering if you could help us to create the tag please. Thanks!

Can you link to the stackoverflow post and I'll add/create the tag?

@chelseadickens
Copy link
Author

I haven't created the post yet since the tag was not available. I'm new to StackOverflow. Can a tag be added after the post is created?

@ddsjoberg
Copy link

Thank you @chelseadickens !!

@shajoezhu we spoke about creating nest-rtables and nest-teal tags. I think we decided previously that tern questions would use the nest-rtables tag. Is that still the case?

@chelseadickens
Copy link
Author

Thanks for your help @ddsjoberg and @shajoezhu !

@shajoezhu
Copy link
Contributor

Thank you @chelseadickens !!

@shajoezhu we spoke about creating nest-rtables and nest-teal tags. I think we decided previously that tern questions would use the nest-rtables tag. Is that still the case?

hi @ddsjoberg , yes we did. But I think we might need "nest-tern" tag as well. especially this case it will be more related to tern. Thanks so much Daniel!

@Melkiades
Copy link
Contributor

Hi @chelseadickens could you double check if now the information is more readable? Thanks! @edelarua @ayogasekaram what do you think about question 2 and 3?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants