Skip to content

[TEST] Increase kmers in execute_layout_test for minHashes later. - #338

Merged
eseiler merged 4 commits into
seqan:mainfrom
smehringer:fix
Sep 8, 2026
Merged

eseiler merged 4 commits into
seqan:mainfrom
smehringer:fix

Conversation

@smehringer

Copy link
Copy Markdown
Member

When I use minHashes in LSH, the normal tests fail because there are not enough kmers to create the needed minHashes. Needs at least 640 hashes.

@smehringer
smehringer requested a review from eseiler September 8, 2026 08:33
@seqan-actions seqan-actions added lint and removed lint labels Sep 8, 2026
@seqan-actions

Copy link
Copy Markdown
Member

Documentation preview available at https://docs.seqan.de/preview/seqan/chopper/338

@codecov

codecov Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.35%. Comparing base (82c0939) to head (310a7ca).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #338   +/-   ##
=======================================
  Coverage   93.35%   93.35%           
=======================================
  Files          20       20           
  Lines         753      753           
  Branches       18       18           
=======================================
  Hits          703      703           
  Misses         50       50           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@seqan-actions seqan-actions added lint and removed lint labels Sep 8, 2026
@seqan-actions seqan-actions added lint and removed lint labels Sep 8, 2026
Comment thread test/api/layout/execute_layout_test.cpp Outdated
@seqan-actions seqan-actions added lint and removed lint labels Sep 8, 2026
@eseiler
eseiler enabled auto-merge (squash) September 8, 2026 12:33
@eseiler
eseiler disabled auto-merge September 8, 2026 12:39
@eseiler
eseiler merged commit 8dbd15a into seqan:main Sep 8, 2026
28 checks passed
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.

3 participants