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.

Sign up for GitHub

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

Jump to bottom

feat: add cadd snv annotation #1150

Merged
merged 15 commits into from May 22, 2023
Merged

feat: add cadd snv annotation #1150

merged 15 commits into from May 22, 2023

Conversation

khurrammaqbool
Copy link
Contributor

This PR:

Added: CADD Annotation for somatic SNVs.

Review and tests:

  • Tests pass
  • Code review
  • New code is executed and covered by tests, and test approve

@khurrammaqbool khurrammaqbool requested a review from a team April 27, 2023 11:09
@khurrammaqbool khurrammaqbool self-assigned this Apr 27, 2023
@khurrammaqbool
Copy link
Contributor Author

Screenshot 2023-04-26 at 16 15 20

@khurrammaqbool khurrammaqbool linked an issue Apr 27, 2023 that may be closed by this pull request
CADD scores for SNVs and INDELs #1106
Closed
4 tasks
@codecov
Copy link

codecov bot commented Apr 27, 2023 •
edited

Codecov Report

Patch and project coverage have no change.

Comparison is base (b5cc183) 99.31% compared to head (0fd8db9) 99.31%.

Additional details and impacted files 
@@           Coverage Diff            @@
##           develop    #1150   +/-   ##
========================================
  Coverage    99.31%   99.31%           
========================================
  Files           29       29           
  Lines         1758     1758           
========================================
  Hits          1746     1746           
  Misses          12       12
Flag Coverage Δ
unittests 99.31% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

ivadym
ivadym approved these changes May 16, 2023
View reviewed changes
Copy link
Contributor

@ivadym ivadym left a comment

Choose a reason for hiding this comment

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

Great work @khurrammaqbool ⭐

CHANGELOG.rst Outdated Show resolved Hide resolved
docs/balsamic_annotation.rst Show resolved Hide resolved
Base automatically changed from feat/add_cadd_snv_ref to develop May 22, 2023 09:25
@sonarcloud
Copy link

sonarcloud bot commented May 22, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@khurrammaqbool khurrammaqbool merged commit 3f1cbce into develop May 22, 2023
22 checks passed
@khurrammaqbool khurrammaqbool deleted the feat/add_cadd_snv_annotation branch May 22, 2023 10:48
@khurrammaqbool khurrammaqbool mentioned this pull request May 23, 2023
Closed
4 tasks
@ivadym ivadym mentioned this pull request Nov 6, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ivadym ivadym ivadym approved these changes

Assignees

@khurrammaqbool khurrammaqbool

Labels
None yet
Projects
BALSAMIC
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CADD scores for SNVs and INDELs
2 participants
@khurrammaqbool @ivadym