Skip to content

Commit

Permalink
Merge pull request #153 from Kulivox/mergeSTR_harmonization_update
Browse files Browse the repository at this point in the history
Merge str harmonization update
  • Loading branch information
LiterallyUniqueLogin authored Mar 24, 2022
2 parents 8dd9abf + 7aba69c commit 261265f
Show file tree
Hide file tree
Showing 8 changed files with 325 additions and 100 deletions.
9 changes: 8 additions & 1 deletion RELEASE_NOTES.rst
Original file line number Diff line number Diff line change
@@ -1,11 +1,18 @@
Unreleased changes
-----

CompareSTR and mergeutils changes:
Functionality Changes:

* MergeSTR: Decision on which records are able to be merged is now based on data from harmonized TRRecords,
and not from the records directly from VCF readers. Thanks to this, HipSTR records which have different starting positions,
but position of their repeat is at the same position are merged correctly. This difference is caused by flanking BP that are
included in the record. These flanking BPs are also removed during the merge, and merged HipSTR records no longer contain them

* CompareSTR: the tool now only compares records that start and end at the same position. If overlap in records is detected,
the program will output a warning to the user. This warning contains IDs of the records and their positions.

Misc:

* mergeutils: function GetMinHarmonizedRecords was transformed into GetRecordComparabilityAndIncrement, which allows the caller
to define custom predicate that decides whether records are comparable.

Expand Down
Loading

0 comments on commit 261265f

Please sign in to comment.