Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

filter_vep_transcript_csqs does not work for passing list of CSQ terms #707

Open
mike-w-wilson opened this issue May 31, 2024 · 1 comment

Comments

@mike-w-wilson
Copy link
Contributor

mike-w-wilson commented May 31, 2024

There is a bug here:

criteria.append(lambda csq: csqs.contains(csq.most_severe_consequence))

The most_severe_consequence field is a top level vep field, at the same level as transcript_consequences, so the way this is currently built does not work as the function looks for most_severe_consequence inside of transcript_consequences here. All other vep fields that get worked on in this function are within transcript_consequences so they work.

@mike-w-wilson
Copy link
Contributor Author

I missed that

def add_most_severe_consequence_to_consequence(
needs to be run first -- should add a note this function wont run without it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant