Skip to content

feat(ensembl): support versioned Ensembl IDs#149

Open
davereinhart wants to merge 1 commit into
inveniosoftware:masterfrom
davereinhart:master
Open

feat(ensembl): support versioned Ensembl IDs#149
davereinhart wants to merge 1 commit into
inveniosoftware:masterfrom
davereinhart:master

Conversation

@davereinhart

@davereinhart davereinhart commented Jul 7, 2026

Copy link
Copy Markdown

Description

Accept an optional version suffix (e.g. ENSG00000012048.15) in the Ensembl regexp, matching the version handling already used by the uniprot and refseq patterns. The version is preserved by normalization (as with refseq) and stripped only when generating a landing-page URL.

Add a test case covering detection, normalization and URL generation for a versioned ID.

Checklist

Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

Frontend

Reminder

By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

  1. You license your contribution under the same terms as the current repository’s license.
  2. You agree that you have the right to license your contribution under the current repository’s license.

Accept an optional version suffix (e.g. ENSG00000012048.15) in the
Ensembl regexp, matching the version handling already used by the
uniprot and refseq patterns. The version is preserved by normalization
(as with refseq) and stripped only when generating a landing-page URL.

Add a test case covering detection, normalization and URL generation
for a versioned ID.

@tmorrell tmorrell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this contribution! This looks like a sensible improvement. I'll leave the PR open for a bit in case others have opinions.

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.

2 participants