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

Uploaded variation name #118

Closed
wants to merge 13 commits into from
Closed

Uploaded variation name #118

wants to merge 13 commits into from

Conversation

monsanto-pinheiro
Copy link
Contributor

@monsanto-pinheiro monsanto-pinheiro commented Jan 5, 2018

To backward compatibility and to be equal with documentation is necessary to make a correction in 'modules/Bio/EnsEMBL/VEP/OutputFactory.pm' line 783.

-Uploaded_variation => $vf->variation_name,
+Uploaded_variation => $vf->variation_name ne '.' ? $vf->variation_name : ($vf->{original_chr} || $vf->{chr}).''.$vf->{start}.''.($vf->{allele_string} || $vf->{class_SO_term}),

From your help:
"""
The default output format ("VEP" format when downloading from the web interface) is a 14 column tab-delimited file. Empty values are denoted by '-'. The output columns are:
'Uploaded variation - as chromosome_start_alleles'
"""

Thomas Maurel and others added 13 commits November 22, 2017 14:20
…to be finished as they need to use the synonym file
* postreleasefix/91:
  version update
  suppress recursion warnings clogging up web logs on bad user input
  dont dump LRG slices for refseq cache
  remove break point
* postreleasefix/91:
  dump_vep_otherfeatures should wait for all the core create_dump_jobs to be finished as they need to use the synonym file
* postreleasefix/91:
  fix for missing info fields in converted variation cache
  BAM_EDIT field missing from VCF output
* postreleasefix/91:
  fix when name is 0 in BED
* postreleasefix/91:
  add --vcf_info_field to help message
  add missing cell_type flag
  Refine filter_vep handling of VCF
Transfer functionality from miRNA.pm plugin to main VEP code
* postreleasefix/91:
  add hardcoded frequency test variants
  fix var cache info error
  cat into sort prevents running out of memory
  subversion update
  check defined before sedding
  gene_symbol should be exempt from numberification in JSON output
  remove any whitespace crud from input
  freeze installed Bio-DB-HTS version, move other installed versions to top for clarity
@at7 at7 closed this Jan 5, 2018
@coveralls
Copy link

coveralls commented Jan 5, 2018

Coverage Status

Coverage increased (+0.003%) to 99.121% when pulling 1b645a5 on master into 46e1afe on release/91.

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.

None yet

4 participants