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

WDL to extract a single callset cost (BQ only, not Terra) #7940

Merged
merged 13 commits into from
Aug 1, 2022

Conversation

RoriCremer
Copy link
Contributor

@RoriCremer RoriCremer commented Jul 13, 2022

Simple WDL with the sole purpose of getting the cost for a callset based on the callset identifier

@codecov
Copy link

codecov bot commented Jul 13, 2022

Codecov Report

❗ No coverage uploaded for pull request base (ah_var_store@63108be). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             ah_var_store     #7940   +/-   ##
================================================
  Coverage                ?   86.247%           
  Complexity              ?     35200           
================================================
  Files                   ?      2173           
  Lines                   ?    165016           
  Branches                ?     17793           
================================================
  Hits                    ?    142321           
  Misses                  ?     16368           
  Partials                ?      6327           

@RoriCremer RoriCremer force-pushed the rc-wrap-the-cost branch 3 times, most recently from 6df3c5e to f7b1489 Compare July 14, 2022 17:28
@RoriCremer RoriCremer force-pushed the rc-wrap-the-cost branch 2 times, most recently from 10f4320 to 8d47881 Compare July 19, 2022 16:15
@RoriCremer RoriCremer marked this pull request as ready for review July 20, 2022 18:07
@RoriCremer RoriCremer changed the title Rc wrap the cost WDL to extract a single callset cost (BQ only, not Terra) Jul 20, 2022
Comment on lines 14 to 15
# the call_set_identifier string is used to name many different things throughout this workflow (BQ tables, vcfs etc),
# and so make sure nothing is broken by creative users, we replace spaces and underscores with hyphens
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code this describes is gone

}
File gatk_override = "gs://gvs_quickstart_storage/jars/gatk-package-4.2.0.0-552-g0f9780a-SNAPSHOT-local.jar"

## TB : GB; 1 : 1024
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TiB : GiB


## TB : GB; 1 : 1024

Float write_API_cost = 0.02684353333 ## BigQuery Storage Write API: $0.025 per 1 GB. The first 2 TB per month are free.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably good to link to doc here, Google is the authority and prices may change

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

per mobbing discussion changing to the same round numbers in the docs

@RoriCremer RoriCremer merged commit 8dd4541 into ah_var_store Aug 1, 2022
@RoriCremer RoriCremer deleted the rc-wrap-the-cost branch August 1, 2022 14:54
This was referenced Mar 17, 2023
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