-
Notifications
You must be signed in to change notification settings - Fork 590
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
VS-530 VDS creation documentation for AoU #8169
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## ah_var_store #8169 +/- ##
================================================
Coverage ? 86.219%
Complexity ? 35515
================================================
Files ? 2191
Lines ? 166339
Branches ? 17901
================================================
Hits ? 143416
Misses ? 16543
Partials ? 6380 |
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.
Some small edits for clarity...etc. before actually trying to generate the VDS.
Co-authored-by: Bec Asch <[email protected]>
Co-authored-by: Bec Asch <[email protected]>
Co-authored-by: Miguel Covarrubias <[email protected]>
0b1a7a7
to
b051652
Compare
47bbffb
to
e7400c5
Compare
1. GCS location of the VDS (the `--vds-path` input from `hail_gvs_import.py`) | ||
2. fully qualified name of the BigQuery dataset (composed of the `project_id` and `dataset_name` inputs from the workflows) | ||
3. callset statistics CSV file (see step #11 above) | ||
4. TSV output from `GvsCalculatePrecisionAndSensitivity` workflow (see step #12 above) | ||
3. CSV output from `GvsCallsetStatistics` workflow |
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.
Probably a good idea to link to vds/delivery/Delivering%20a%20VDS.md since that includes details of how to get the VDS to the AoU delivery bucket, naming conventions...etc.
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.
great idea
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.
One nit to re-arrange it a bit, otherwise 👍🏻
Co-authored-by: Bec Asch <[email protected]>
We now want to fully update the AoU documentation so that a VDS can be created (no VCFs!)