Skip to content

Master

Marek Szuba requested to merge github/fork/aparton/master into master

Created by: aparton

Requirements

  • Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
  • Review the contributing guidelines for this repository; remember in particular:
    • do not modify code without testing for regression
    • provide simple unit tests to test the changes
    • the PR must not fail unit testing
    • if you're adding/updating documentation of an endpoint, make sure you add/update the necessary parameters to the (template) configuration files in the ensembl-rest_private repo

Description

One or more sentences describing in detail the proposed changes.

Added in a transcript_id flag to allow for filtering by transcript

Use case

Requested - allows for filtering by transcript on the REST side rather than having to be performed by the user

Benefits

If applicable, describe the advantages the changes will have.

Possible Drawbacks

If applicable, describe any possible undesirable consequence of the changes.

Flag is only available on rest - slight discord between REST and CL VEP

Testing

Have you added/modified unit tests to test the changes?

Yes.

If so, do the tests pass/fail?

Pass, of course.

Have you run the entire test suite and no regression was detected?

Yes.

Changelog

Added the ability to filter VEP output by transcript

Merge request reports