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

Okay to include PEER covariants? #55

Closed
boxiangliu opened this issue Oct 9, 2016 · 2 comments
Closed

Okay to include PEER covariants? #55

boxiangliu opened this issue Oct 9, 2016 · 2 comments

Comments

@boxiangliu
Copy link

Hi Graham,

The final combined_test.py can take principle components into account. Would it be okay to use PEER factors, age, gender and other covariates?

Bosh

@gmcvicker
Copy link
Collaborator

In principle this should work, but I have not tried this myself. I have also not tested the PC option of the CHT for a long time, so let us know if you try this and run into any problems.

@bmvdgeijn
Copy link
Owner

Since the coefficients are fit one at a time, it would likely work better
if the covariates are orthogonal (as PCs are ). I might suggest
orthogonalizing the covariate matrix (perhaps using PCA).

On Mon, Oct 10, 2016 at 2:28 PM, Graham McVicker notifications@github.com
wrote:

In principle this should work, but I have not tried this myself. I have
also not tested the PC option of the CHT for a long time, so let us know if
you try this and run into any problems.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#55 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADSDguM89IN11l4qstjEiWbrjNfmTp7Pks5qyoPCgaJpZM4KSJx2
.

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

3 participants