bacpop / ska.rust

Split k-mer analysis – version 2
https://docs.rs/ska/latest/ska/
Apache License 2.0
56 stars 4 forks source link

Testing on a subsample of reads #72

Closed apollis44 closed 2 months ago

apollis44 commented 2 months ago

I introduced a new optional parameter --max_reads to the build function (and all associated functions). This allows you to specify the maximum number of reads to be used, and for now, it will process reads in the order they are provided.

codecov-commenter commented 2 months ago

Codecov Report

Attention: Patch coverage is 89.65517% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 93.36%. Comparing base (c2d279a) to head (3b4baf9).

:exclamation: Current head 3b4baf9 differs from pull request most recent head b724f92. Consider uploading reports for the commit b724f92 to get more accurate results

Files Patch % Lines
src/cli.rs 50.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #72 +/- ## ========================================== + Coverage 92.43% 93.36% +0.93% ========================================== Files 16 16 Lines 2273 2261 -12 ========================================== + Hits 2101 2111 +10 + Misses 172 150 -22 ```

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