[ensembl-dev] empty values in variant_effect_predictor.pl

Genomeo Dev genomeodev at gmail.com
Thu Dec 4 11:42:10 GMT 2014


Personally, for now, since I am about to start running VEP, I will be happy
Will if you could point me all the places where I can patch the code to get
rid of the dashes.

G.

On 3 December 2014 at 16:49, Michael Yourshaw <myourshaw at ucla.edu> wrote:

> Hi Will—
>
> I suspect this is an issue for many others, as well. For one thing, in a
> database table the dash requires what should be a numerical field to be
> treated like text.
>
>  I deal with the dashes in post-processing, but as G says this can be a
> pain. Rather than users having to patch each new version of VEP, could you
> consider adding a new parameter so your code would look like
>
>         $output = join "\t", map {
>             (defined $line->{$_} ? $line->{$_} : (defined $extra{$_} ?
> $extra{$_} : $config->{empty_field_char}))
>         } @{$config->{fields}};
>
>
> and allow -- empty_field_char with no argument to yield ‘’.
>
>
>>
> Michael Yourshaw, PhD
> UCLA Geffen School of Medicine
> Department of Pediatrics
> 695 Charles E Young Drive S
> Gonda 5554
>
> Los Angeles CA 90095-8348 USA
>
> myourshaw at ucla.edu
>
> 970.691.8299
>
> This message (including any attachments) is intended only for the use of
> the addressee(s) and may contain information that is PRIVILEGED and
> CONFIDENTIAL, and/or may constitute ATTORNEY WORK PRODUCT. If you are not
> an intended recipient, you are hereby notified that any dissemination of
> this communication is strictly prohibited. If you have received this
> message in error, please do not read, copy, or forward this message or
> any attachments. Please permanently delete all copies of the message and
> any attachments and notify the sender immediately by sending an email to
> myourshaw at yourshaw.org. Thank you. As part of our commitment to the
> environment, this message was manufactured with 100% recycled electrons.
>
>
>
>
> On 3Dec, 2014, at 08:23, Will McLaren <wm2 at ebi.ac.uk> wrote:
>
> Hi G,
>
> The print_line() subroutine in variant_effect_predictor.pl has the
> following code:
>
>         $output = join "\t", map {
>             (defined $line->{$_} ? $line->{$_} : (defined $extra{$_} ?
> $extra{$_} : '-'))
>         } @{$config->{fields}};
>
> So to do as you intend you could change this to
>
>         $output = join "\t", map {
>             (defined $line->{$_} ? $line->{$_} : (defined $extra{$_} ?
> $extra{$_} : ''))
>         } @{$config->{fields}};
>
> Regards
>
> Will McLaren
> Ensembl Variation
>
> On 3 December 2014 at 16:17, Genomeo Dev <genomeodev at gmail.com> wrote:
>
>> Hi,
>>
>> Current VEP script puts dash in empty fields in the default output. Could
>> you please point me to the code where I can remove replace dash with an
>> empty string? Modifying the output file after it is generated is
>> non-trivial for very large files.
>>
>> --
>> G.
>>
>> _______________________________________________
>> Dev mailing list    Dev at ensembl.org
>> Posting guidelines and subscribe/unsubscribe info:
>> http://lists.ensembl.org/mailman/listinfo/dev
>> Ensembl Blog: http://www.ensembl.info/
>>
>>
> _______________________________________________
> Dev mailing list    Dev at ensembl.org
> Posting guidelines and subscribe/unsubscribe info:
> http://lists.ensembl.org/mailman/listinfo/dev
> Ensembl Blog: http://www.ensembl.info/
>
>
>
> _______________________________________________
> Dev mailing list    Dev at ensembl.org
> Posting guidelines and subscribe/unsubscribe info:
> http://lists.ensembl.org/mailman/listinfo/dev
> Ensembl Blog: http://www.ensembl.info/
>
>


-- 
G.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.ensembl.org/pipermail/dev_ensembl.org/attachments/20141204/d5da5c0e/attachment.html>


More information about the Dev mailing list