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

Moving extract wdls from variantstore repo #6902

Merged
merged 4 commits into from
Oct 22, 2020
Merged

Moving extract wdls from variantstore repo #6902

merged 4 commits into from
Oct 22, 2020

Conversation

meganshand
Copy link
Contributor

I made no changes to the copied files. @kcibul @ahaessly let me know if I'm missing anything. I left some of the demo bash scripts thinking that we didn't necessarily need to keep them now that we have WDLs, but let me know if you want me to move anything else.

The readme update here assumes that #6881 will get merged. I also turned off the tests.

-R "~{reference}" \
-O "~{output_file}" \
~{probe_info_clause} \
--project-id "~{project_id}" \

Choose a reason for hiding this comment

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

FYI, my PR with the sample map change will also change this argument to be --read-project-id. Maybe we can make that change with this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Great, yeah I'll update it here.

Copy link
Contributor

@ahaessly ahaessly left a comment

Choose a reason for hiding this comment

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

👍

@meganshand meganshand merged commit 052bdf8 into ah_var_store Oct 22, 2020
@meganshand meganshand deleted the ms_copy branch October 22, 2020 19:12
kcibul pushed a commit that referenced this pull request Jan 29, 2021
* add files

* turn off testing

* updating readme

* updating parameter name
kcibul pushed a commit that referenced this pull request Feb 1, 2021
* add files

* turn off testing

* updating readme

* updating parameter name
Marianie-Simeon pushed a commit that referenced this pull request Feb 16, 2021
* add files

* turn off testing

* updating readme

* updating parameter name
kcibul pushed a commit that referenced this pull request Mar 9, 2021
* add files

* turn off testing

* updating readme

* updating parameter name
mmorgantaylor pushed a commit that referenced this pull request Apr 6, 2021
* add files

* turn off testing

* updating readme

* updating parameter name
mmorgantaylor pushed a commit that referenced this pull request Apr 6, 2021
* add files

* turn off testing

* updating readme

* updating parameter name
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.

3 participants