-
Notifications
You must be signed in to change notification settings - Fork 121
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
checklist for a new study #248
Comments
@jjgao I added meta-studies to all the data files and case lists. But I don't know if CNA, segment, and expression is a must for public studies, since most publications we see only have mutations data. The MAF is the minimum genomic data file required for a study to be created. |
@ritikakundra if data does not exist, you can also check it off. The purpose of a checklist is to check one by one. And we can also evolve this list over time. To utilize this checklist, one option would be creating a pull request per study with this checklist in the PR. |
cc'ing @schultzn to see if he has anything to add. |
@jjgao ok. Added a few more things |
Thanks, @ritikakundra. Let's start to use this checklist when pushing out new studies? |
@yichaoS @rmadupuri Our checklist for any dataset that goes public. |
@ritikakundra @yichaoS @rnugraha let's also try to improve this checklist over time. For new studies, could you create a pull request per study and paste this checklist there? Thanks. |
looks good to me! |
@sandertan you might also want to check this |
@ritikakundra do we have a checklist for a new study. Could you refine the list below?
mixed
studies, let's not usemixed
and decide case by case.The text was updated successfully, but these errors were encountered: