Skip to content

Fix EPO-Extended low-coverage species classification#1164

Open
twalsh-ebi wants to merge 1 commit into
Ensembl:postreleasefix/116from
twalsh-ebi:tweak/epo_ext_low_cov
Open

Fix EPO-Extended low-coverage species classification#1164
twalsh-ebi wants to merge 1 commit into
Ensembl:postreleasefix/116from
twalsh-ebi:tweak/epo_ext_low_cov

Conversation

@twalsh-ebi
Copy link
Copy Markdown
Contributor

Requirements

  • Filling out the template is required. Any pull request that does not include enough information to be efficiently reviewed may be rejected.
  • Please consider which branch this is to be submitted against. This is usually obvious, however if it is to be applied to both a release branch nn and master then please submit it against postreleasefix/nn and let us merge it into to the two branches.

Description

ensembl-webcode PR 1158 updated the web code to support EPO_EXTENDED alignments.

However, the code setting $is_low_coverage_species was in need of an update to reflect changes in the web code since the switch from EPO_LOW_COVERAGE to EPO_EXTENDED, in order to ensure that the species production name is used when accessing packed ALIGNMENTS metadata.

This PR would implement that update, fetching the production name using the species URL accessed via $object->species and using that to access relevant ALIGNMENTS metadata.

Views affected

This affects EPO-Extended alignment views, particularly those with a text alignment slice table.

Example:

Possible complications

None expected, as this change is in a section of code specific to EPO_EXTENDED and EPO_LOW_COVERAGE alignments.

Merge conflicts

None detected.

Related JIRA Issues (EBI developers only)

  • ENSCOMPARASW-8799

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.

1 participant