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: Adapt ACMG Rating for multiple sources of info (#65) #66

Merged
merged 15 commits into from
Sep 25, 2023
Merged

feat: Adapt ACMG Rating for multiple sources of info (#65) #66

merged 15 commits into from
Sep 25, 2023

Conversation

gromdimon
Copy link
Contributor

@gromdimon gromdimon commented Sep 19, 2023 •
edited
Loading

Tasks:

  • Implement new structure with types/interfaces
  • Refactor existing code
  • Change implementation of acmg class calculation
  • Test

@gromdimon gromdimon linked an issue Sep 19, 2023 that may be closed by this pull request
Adapt ACMG rating for multiple sources of information #65
Closed
@codecov
Copy link

codecov bot commented Sep 19, 2023 •
edited
Loading

Codecov Report

Merging #66 (3946acc) into main (507ad2c) will increase coverage by 9.87%.
The diff coverage is n/a.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main      #66      +/-   ##
==========================================
+ Coverage   87.71%   97.59%   +9.87%     
==========================================
  Files          39        2      -37     
  Lines        1424       83    -1341     
  Branches      409        0     -409     
==========================================
- Hits         1249       81    -1168     
+ Misses        170        2     -168     
+ Partials        5        0       -5

see 37 files with indirect coverage changes

@gromdimon gromdimon marked this pull request as ready for review September 22, 2023 13:54
@gromdimon gromdimon enabled auto-merge (squash) September 25, 2023 09:31
@gromdimon gromdimon merged commit 54a2d88 into main Sep 25, 2023
9 checks passed
@gromdimon gromdimon deleted the 65-adapt-acmg-rating-for-multiple-sources-of-information branch September 25, 2023 09:31
@varfish-bot varfish-bot mentioned this pull request Sep 18, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Adapt ACMG rating for multiple sources of information
1 participant
@gromdimon