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.

Sign up for GitHub

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

Jump to bottom

Allow input arguments and data source initialization code to be shared across traversal types #299

Closed
droazen opened this issue Mar 12, 2015 · 1 comment
Closed

Allow input arguments and data source initialization code to be shared across traversal types #299

droazen opened this issue Mar 12, 2015 · 1 comment
Assignees
@droazen
Labels
coding Engine

Comments

@droazen
Copy link
Collaborator

droazen commented Mar 12, 2015

Currently, each Walker class defines its own input arguments, and contains its own code to initialize the appropriate data sources for the inputs present. This code should ideally be shared across traversal types (eg., in GATKTool), with the ability to override attributes such as required and optional on a per-traversal and/or per-tool basis.

Potentially requires #149

Related to: #120, #133, #141
@droazen droazen added this to the Coelacanth milestone Mar 12, 2015
@droazen droazen self-assigned this Mar 12, 2015
@droazen
Copy link
Collaborator Author

droazen commented Mar 16, 2015

This was done as part of #297 -- closing. (#149 is still an open TODO, however)

@droazen droazen closed this as completed Mar 16, 2015
@akiezun akiezun modified the milestone: Coelacanth Apr 20, 2015
kcibul added a commit that referenced this issue May 26, 2021
@kcibul
#299 - Sample list ease of use for cohort extracts (#7272)
f09214c 
* work on issue 299
* PR comments
* update dockstore/image
This was referenced Mar 17, 2023
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@droazen droazen

Labels
coding Engine
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@droazen @akiezun