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

Fix GitHub Actions spec for zip_studies #2049

Merged
merged 1 commit into from
Aug 2, 2024
Merged

Conversation

jamesqo
Copy link
Contributor

@jamesqo jamesqo commented Aug 2, 2024

The main culprit was the missing parentheses, I also double-checked my changes from #2048 and fixed some variables that I had forgotten to update. /cc @rmadupuri

checks

For all pull requests:

  • Passes validation

For a new study (in addition to above):

  • Does study name and study ID follow our convention? e.g. Tumor_Type (Institue, Journal Year); brca_mskcc_2015
  • is study meta data complete? e.g. pmid, group of PUBLIC
  • were all samples profiled with WES/WGS? If not, is gene panel file curated?
  • are oncotree codes of all samples curated; Cancer Type and Cancer Type Detailed needs to be added in addition to Oncotree Code
  • clinical sample and patient data with meta files
  • mutations data with meta files
  • MAF is based on hg19
  • MAF with 2 isoforms: uniprot and mskcc
  • CNA data with meta files
  • CNA segment data with meta files
  • Expression data including z-scores with meta files
  • Case-lists for all profiles.
  • Manual checking (Niki or JJ): Triage or private Portal link here

@rmadupuri rmadupuri merged commit fc4b313 into master Aug 2, 2024
2 checks passed
@rmadupuri rmadupuri deleted the fix-zip-studies branch August 2, 2024 16:28
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

Successfully merging this pull request may close these issues.

2 participants