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

Should we add 'assumption test' for Cox Hazard model? #84

Open
chandryou opened this issue Jul 3, 2019 · 4 comments
Open

Should we add 'assumption test' for Cox Hazard model? #84

chandryou opened this issue Jul 3, 2019 · 4 comments
Labels
enhancement New functionality that could be added

Comments

@chandryou
Copy link

Distriburted Research Network does not gather the whole statistical attributes, so a researcher cannot know if there was important violation of proportional hazard assumption, which may lead to wrong and misleading estimates ref.

Should we add 'assumption test' for Cox Hazard model, whose result a researcher can gather, too?

@schuemie schuemie added the enhancement New functionality that could be added label Feb 17, 2020
@schuemie
Copy link
Member

Agreed. We should compute the Schoenfeld residuals to test proportionality.

@msuchard
Copy link
Member

msuchard commented May 5, 2020

Schoenfeld $r_ij$ residuals for failure $i$ and covariate $j$ may not be terribly effective as plotting $r_ij$ as a function of covariate $j$ should be centered about 0. However, we only have two discrete values for covariate $j$ (the treatment effect), so this may be hard to see. The alternative is to plot as a function of failure time.

One will need to port the code from cox.zph() into CohortMethod or Cyclops

@msuchard
Copy link
Member

msuchard commented May 5, 2020

Another approach is to plot log(-log(S(t))) where S(t) is the survival function estimate that Cyclops (already? I believe) provides.

S(t) is also approximated into the Kaplan-Meier plots that we have already generated and packaged up across data sources. @chandryou -- maybe this is the way you want to go in your ACS study?

@msuchard
Copy link
Member

msuchard commented May 5, 2020

BTW, the formula for the Schoenfeld residuals is super-simple for a binary covariate and a time-invariant model with Breslow-like ties (which we are using):

https://documentation.sas.com/?cdcId=pgmsascdc&cdcVersion=9.4_3.2&docsetId=statug&docsetTarget=statug_phreg_details52.htm&locale=en

@schuemie schuemie moved this from Todo to In Progress in HADES Hack-a-thon Oct 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New functionality that could be added
Projects
None yet
Development

No branches or pull requests

3 participants