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

Firth penalized likelihood function in assocCoxPH #8

Open
ryshi06 opened this issue Jun 15, 2021 · 2 comments
Open

Firth penalized likelihood function in assocCoxPH #8

ryshi06 opened this issue Jun 15, 2021 · 2 comments

Comments

@ryshi06
Copy link

ryshi06 commented Jun 15, 2021

Hi there,

I am wondering is assocCoxph support firth's penalized likelihood function or other possible functions adjusting for monotone likelihood issues?

Thanks a lot!

Best,
Ruyu

@smgogarten
Copy link
Owner

No, it does not. While assocRegression has an option to use the Firth test (via the logistf package), assocCoxPH does not have that option. There is a package coxphf that would provide the test you're looking for, but I didn't know it existed until I looked it up just now to respond to your question.

This is not a feature I will have time to add anytime soon, but if you're interested in adding it to GWASTools yourself, you could submit a pull request.

@ryshi06
Copy link
Author

ryshi06 commented Jun 16, 2021

Thanks for your reply. I 'll try the assocCoxPH first to decide whether I do need firth's correction. Thanks again!

Best,
Ruyu

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

No branches or pull requests

2 participants