maize-genetics / phg_v2

Practical Haplotype Graph (PHG) version 2
https://phg.maizegenetics.net/
Apache License 2.0
21 stars 2 forks source link

Haplotype graph efficiency improvements #230

Closed lynnjo closed 1 month ago

lynnjo commented 1 month ago

Description

Reworked HaplotypeGraph for efficiency, removed coroutines

Type of change

What type of changes does your code introduce? Put an x in boxes that apply.

Checklist:

Changelog entry

Please add a one-line changelog entry below. This will be copied to the changelog file during the release process.

HaplotypeGraph refactored for efficiency
codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 88.40580% with 8 lines in your changes missing coverage. Please review.

Project coverage is 81.35%. Comparing base (4ff26c1) to head (24fb209). Report is 13 commits behind head on main.

Files with missing lines Patch % Lines
...n/kotlin/net/maizegenetics/phgv2/utils/VCFUtils.kt 76.66% 0 Missing and 7 partials :warning:
...tlin/net/maizegenetics/phgv2/api/HaplotypeGraph.kt 97.43% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #230 +/- ## ========================================== - Coverage 81.41% 81.35% -0.06% ========================================== Files 81 81 Lines 5444 5466 +22 Branches 903 912 +9 ========================================== + Hits 4432 4447 +15 Misses 677 677 - Partials 335 342 +7 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.