feat: l2g feature to indicate if gene is protein-coding or not

[xyg123]

✨ Context

Adding a feature to tell l2g when a gene is protein coding or not
opentargets/issues#3509

🛠 What does this PR implement

A new feature that is either 0 or 1, for each studylocus-to-gene mapping, 1 when the gene is protein-coding, 0 otherwise.

🚦 Before submitting

• Do these changes cover one single feature (one change at a time)?
• Did you read the contributor guideline?
• Did you make sure to update the documentation with your changes?
• Did you make sure there is no commented out code in this PR?
• Did you follow conventional commits standards in PR title and commit messages?
• Did you make sure the branch is up-to-date with the dev branch?
• Did you write any new necessary tests?
• Did you make sure the changes pass local tests (make test)?
• Did you make sure the changes pass pre-commit rules (e.g poetry run pre-commit run --all-files)?

[ireneisdoomed]

Since this is only going to be one feature, I don't see the need in having a common function.

[addramir]

Looks good to me. @ireneisdoomed please have a look.
BTW why is it named isProteinCoding1mb? Why 1mb?

[ireneisdoomed]

LGTM

[xyg123]

Looks good to me. @ireneisdoomed please have a look. BTW why is it named isProteinCoding1mb? Why 1mb?

The feature is generated for each study locus, and it’s set to 1mb because, by default, VEP and other related annotation tools use this range to define potential effector genes.

[ireneisdoomed]

@xyg123 I agree with @addramir in that the window only matters to build the list of variant/gene pairs. If the gene is protein coding or not is not dependent of the window.