Skip to content

Commit

Permalink
fix assert bug that crashes on cycle
Browse files Browse the repository at this point in the history
  • Loading branch information
glennhickey committed Jun 4, 2024
1 parent 1fca800 commit 2f80767
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/traversal_clusters.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -236,14 +236,14 @@ vector<vector<int>> assign_child_snarls_to_traversals(const PathHandleGraph* gra
}
vector<int> contained_snarls;
for (const auto& cs_count : fw_count) {
assert(cs_count.second == 1 || cs_count.second == 2);
if (cs_count.second == 2 || (!fully_contained && cs_count.second == 1)) {
assert(cs_count.second == 1 || cs_count.second >= 2);
if (cs_count.second >= 2 || (!fully_contained && cs_count.second == 1)) {
contained_snarls.push_back(cs_count.first);
}
}
for (const auto& cs_count : rv_count) {
assert(cs_count.second == 1 || cs_count.second == 2);
if (cs_count.second == 2 || (!fully_contained && cs_count.second == 1)) {
assert(cs_count.second == 1 || cs_count.second >= 2);
if (cs_count.second >= 2 || (!fully_contained && cs_count.second == 1)) {
contained_snarls.push_back(cs_count.first);
}
}
Expand Down

1 comment on commit 2f80767

@adamnovak
Copy link
Member

Choose a reason for hiding this comment

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

vg CI tests complete for branch deconstruct. View the full report here.

16 tests passed, 0 tests failed and 0 tests skipped in 17542 seconds

Please sign in to comment.