search

georgheinze / coxphf

2 stars 0 forks source link

Broom support #11

Closed gregorsteiner closed 1 year ago

gregorsteiner commented 1 year ago

I added support for {broom}. There are now broom::tidy(), broom::glance(), and broom::augment() methods for coxphf objects.

albamrt commented 6 months ago

The broom::tidy function is not working for me. I was trying to use it since the tidy.coxphf doesn't provide the same confidence intervals (Wald) as the coxphf function with default settings (profile likelihood).

gregorsteiner commented 6 months ago

Yes, you are right, ideally one should be able to choose the confidence intervals in tidy.coxphf. I no longer actively work on this project and don't have the time to implement and properly test this now. If you need this urgently, you could perhaps write your own tidy method by modifying the code in this PR. I think you would only need to extract the profile likelihood CI from the summary object and use that instead of the Wald CI used here.

albamrt commented 6 months ago

Sure, that is what I did. Thank you for the answer.