refactor(vep_parser): store consequence to impact score as a project config #811
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
✨ Context
Removes the
src/gentropy/assets/data/variant_consequence_to_score.tsv
file. This mapping between a variant conseauence and the impact score is now a config parameter ofVariantIndexStep
.This feature was implemented in #805. Because I didn't know the file was a dependency of the VEP parser, this PR isolates the feature. The VEP parser now reads this property directly from the dataset.
I have run the variant index generation from all the VEP outputs produced in the last run and the mapping works properly.
🛠 What does this PR implement
VariantIndexStep.consequence_to_pathogenicity_score
VariantIndex.get_most_severe_gene_consequence
, as this is already accessible in the VariantIndex dataset directly (transcriptConsequences.consequenceScore
)🙈 Missing
🚦 Before submitting
dev
branch?make test
)?poetry run pre-commit run --all-files
)?