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.

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

[FIX] Readd removed if case #223

Merged
merged 3 commits into from
Jul 26, 2022
Merged

Conversation

Irallia
Copy link
Collaborator

@Irallia Irallia commented Jul 19, 2022

I made a mistake when calling INVs. I accidentally deleted one if case too many.
With the new benchmarks, I then noticed on the plot that something is not right.
It was:
results no_DUP_and_INV-INS all
and now it is:
results no_DUP_and_INV all

@Irallia Irallia self-assigned this Jul 19, 2022
@codecov
Copy link

codecov bot commented Jul 19, 2022

Codecov Report

Merging #223 (3cd2375) into master (affb214) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #223   +/-   ##
=======================================
  Coverage   98.39%   98.40%           
=======================================
  Files          18       18           
  Lines         874      875    +1     
=======================================
+ Hits          860      861    +1     
  Misses         14       14           
Impacted Files Coverage Δ
src/variant_detection/variant_output.cpp 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update affb214...3cd2375. Read the comment docs.

@Irallia Irallia requested review from joergi-w and joshuak94 July 19, 2022 17:30
src/variant_detection/variant_output.cpp Outdated Show resolved Hide resolved
src/variant_detection/variant_output.cpp Show resolved Hide resolved
Comment on lines -29 to -31
chr1 267 . N <DEL> 3 PASS END=285;SVLEN=-18;iGenVar_SVLEN=-18;SVTYPE=DEL GT ./.
chr1 283 . N <DEL> 3 PASS END=298;SVLEN=-15;iGenVar_SVLEN=-15;SVTYPE=DEL GT ./.
chr1 509 . N <DEL> 1 PASS END=528;SVLEN=-19;iGenVar_SVLEN=-19;SVTYPE=DEL GT ./.
Copy link
Member

Choose a reason for hiding this comment

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

Were these deletions incorrect?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, they came from other SV types:
Double Translocation ref: (266, 282] translocated after 286
Duplication in reference and read covered by SA tag ref: (509, 527] with copynumber=5

@Irallia Irallia force-pushed the FIX/readd_removed_if branch from 0dde7db to 3cd2375 Compare July 20, 2022 12:00
@Irallia Irallia mentioned this pull request Jul 20, 2022
@Irallia Irallia merged commit 344ccb4 into seqan:master Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants