Skip to content

fix(workflow): stop scheduling VEP annotation in the pipeline - #840

Merged
bencap merged 1 commit into
chore/bencap/797/export-hardeningfrom
bugfix/bencap/703/use-mapped-transcript-for-vep-consequence
Aug 12, 2026
Merged

fix(workflow): stop scheduling VEP annotation in the pipeline#840
bencap merged 1 commit into
chore/bencap/797/export-hardeningfrom
bugfix/bencap/703/use-mapped-transcript-for-vep-consequence

Conversation

@bencap

@bencap bencap commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

VEP consequences were sourced from the top-level most_severe_consequence, which reports the worst call across all overlapping transcripts instead of the transcript the variant was actually mapped to, producing misleading annotations.

The job definition is left commented out rather than deleted so it can be restored once VEP annotates Allele rows directly.

See #772

@coveralls

coveralls commented Aug 11, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 31627793881

Warning

No base build found for commit 50c3137 on chore/bencap/797/export-hardening.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 89.335%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 15106
Covered Lines: 13495
Line Coverage: 89.34%
Coverage Strength: 0.89 hits per line

💛 - Coveralls

VEP consequences were sourced from the top-level most_severe_consequence,
which reports the worst call across all overlapping transcripts instead of
the transcript the variant was actually mapped to, producing misleading
annotations.

The job definition is left commented out rather than deleted so it can be
restored once VEP annotates Allele rows directly.

Refs #772
@bencap
bencap force-pushed the bugfix/bencap/703/use-mapped-transcript-for-vep-consequence branch 2 times, most recently from 9fd3dec to c745403 Compare August 12, 2026 18:28
@bencap
bencap merged commit 7fcf9a9 into release-2026.2.7.1 Aug 12, 2026
5 checks passed
@bencap
bencap deleted the bugfix/bencap/703/use-mapped-transcript-for-vep-consequence branch August 12, 2026 19:02
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