BioSequences.jl icon indicating copy to clipboard operation
BioSequences.jl copied to clipboard

WIP: Improve bit-twiddling for kmer counting

Open jakobnissen opened this issue 7 months ago • 1 comments

When implementing https://github.com/BioJulia/Kmers.jl/issues/48, I want to use BioSequences internals. Keep an open branch here for all optimisations I want to upstream.

jakobnissen avatar Jun 04 '25 12:06 jakobnissen

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.86%. Comparing base (95d9218) to head (36b00f4). Report is 21 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #332      +/-   ##
==========================================
+ Coverage   90.87%   91.86%   +0.99%     
==========================================
  Files          31       29       -2     
  Lines        2400     2804     +404     
==========================================
+ Hits         2181     2576     +395     
- Misses        219      228       +9     
Flag Coverage Δ
unittests 91.86% <100.00%> (+0.99%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

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

codecov[bot] avatar Jun 04 '25 12:06 codecov[bot]

Any of this that can be shunted off to BioGenerics.jl to avoid the need to duplicate? Or do you mean that BioSequences is a dependency of Kmers, and you want to make the change here instead of there?

kescobo avatar Jun 27 '25 12:06 kescobo

Yeah, I want to upstream some code from Kmers to BioSequences

jakobnissen avatar Jun 27 '25 12:06 jakobnissen