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

Funcotator - Need to have a funcotation output list #4359

Closed
jonn-smith opened this issue Feb 6, 2018 · 3 comments
Closed

Funcotator - Need to have a funcotation output list #4359

jonn-smith opened this issue Feb 6, 2018 · 3 comments
Assignees
Milestone

Comments

@jonn-smith
Copy link
Collaborator

Must allow support for an output funcotation list that, if specified, will cause Funcotator only output funcotations from the given list and suppress all other funcotations.

@jonn-smith jonn-smith added this to the Engine-4.1 milestone Feb 6, 2018
@jonn-smith jonn-smith self-assigned this Feb 6, 2018
@jonn-smith jonn-smith modified the milestones: Engine-4.1, Engine-1Q2018 Feb 6, 2018
@droazen droazen modified the milestones: Engine-1Q2018, Engine-2Q2018 Apr 6, 2018
@jonn-smith jonn-smith assigned LeeTL1220 and unassigned jonn-smith May 15, 2018
@jonn-smith
Copy link
Collaborator Author

This is related to #4583 - that issue must be done before this one is worked on.

@LeeTL1220
Copy link
Contributor

LeeTL1220 commented Jun 26, 2018

  • Add a CLI flag --exclude-field-name which can be specified multiple times.
    For example ... --exclude-field-name Clinvar_bar --exclude-field-name Clinvar_foo ...

We will specify the column names with the full field name: <datasourcename>_<fieldname>

For now, we specify via an exclude. If nothing is specified, then show everything.

@jonn-smith
Copy link
Collaborator Author

Put exclusion list into OutputRenderer at construction time, then check on write.

LeeTL1220 added a commit that referenced this issue Sep 26, 2018
…. Added some developer utilities for ease of testing and creating new Funcotations.
LeeTL1220 added a commit that referenced this issue Sep 29, 2018
…rom output. (#5226)

- Added support for `--exclusion-list` parameter, which can be specified multiple times.  This will remove some rendered fields from the output.  The field should appear exactly as written in the output MAF or VCF (e.g. ClinVar_ALLELEID). Closes #4359. 
- Does basic cleaning of field values for MAF.  I.e. cleaning tabs and \n.  Fixes #4693
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants