-
Notifications
You must be signed in to change notification settings - Fork 2
Add tr gwas #26
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
base: main
Are you sure you want to change the base?
Add tr gwas #26
Conversation
gymreklab
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for submitting this PR!
I made mostly minor comments. Once those are addressed we can merge.
See in particular the comment on automatic filtering of covariates - this can be addressed in the next PR but should be high on our list to avoid accidental filtering of covariates we didn't mean to in future runs.
| data.to_csv(covar_file_path, sep="\t", index=False) | ||
|
|
||
| print(f"Done converting {args.phenotype} to plink format") | ||
| return plink_pheno,data |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't need to return these since they are not used. can remove this line
|
Thank you for all the comments! I made the changes suggested in the comments. If everything looks good I will confirm merge. |
…in the phecode cohort
…ed on GT and remap GT
add tr-gwas script and wdl. It enables running tr gwas genome wide on AoU cloud environment