-
Notifications
You must be signed in to change notification settings - Fork 596
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
A few last doc changes #7927
A few last doc changes #7927
Conversation
8ee25b2
to
21be0d7
Compare
21be0d7
to
a015a2d
Compare
Codecov Report
@@ Coverage Diff @@
## ah_var_store #7927 +/- ##
================================================
Coverage ? 86.247%
Complexity ? 35200
================================================
Files ? 2173
Lines ? 165016
Branches ? 17793
================================================
Hits ? 142321
Misses ? 16368
Partials ? 6327 |
84fbd83
to
cce8872
Compare
cce8872
to
c3b3631
Compare
060e046
to
d6042d3
Compare
@@ -2,17 +2,17 @@ | |||
|
|||
| Workflow Version | Date Updated | Documentation Authors | Questions or Feedback | | |||
| :----: | :---: | :----: | :--------------: | | |||
| [GvsJointVariantCalling](https://github.com/broadinstitute/gatk/blob/rc-vs-483-beta-user-wdl/scripts/variantstore/wdl/GvsJointVariantCalling.wdl) | June, 2022 | [Kaylee Mathews](mailto:[email protected]) and [Aurora Cremer](mailto:[email protected]) | If you have questions or feedback, contact the [Broad Variants team](mailto:[email protected]) | | |||
| [GvsJointVariantCalling](https://github.com/broadinstitute/gatk/blob/ah_var_store/scripts/variantstore/wdl/GvsJointVariantCalling.wdl) | June, 2022 | [Kaylee Mathews](mailto:[email protected]) and [Aurora Cremer](mailto:[email protected]) | If you have questions or feedback, contact the [Broad Variants team](mailto:[email protected]) | |
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.
Didn't we want this pinned to a particular release so that it doesn't change out from under users?
| 10379 | 05:38:00 | $683.33 | $0.0658 | | ||
| Number of Samples | Elapsed Time (hh:mm) | Terra Cost | BigQuery Cost | Total Cost | Approximate Cost per Sample | | ||
|-------------------|----------------------|------------|---------------|------------|-----------------------------| | ||
| 10 | 04:30 | $0.84 | $0.51 | $1.35 | $0.14 | |
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.
I thought we had run this on other sets that were larger than this to provide more cost information?
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.
Ah I see those are in a different document and this is just for the quickstart. Nvm!
adding time and cost estimates to the documentation