Closed althonos closed 2 years ago
Merging #13 (3d79ab4) into master (eb6fe7c) will decrease coverage by
0.25%
. The diff coverage is72.17%
.
@@ Coverage Diff @@
## master #13 +/- ##
==========================================
- Coverage 80.54% 80.28% -0.26%
==========================================
Files 3 3
Lines 4122 4296 +174
==========================================
+ Hits 3320 3449 +129
- Misses 802 847 +45
Flag | Coverage Δ | |
---|---|---|
CPython | 80.28% <72.17%> (-0.26%) |
:arrow_down: |
Linux | 80.28% <72.17%> (-0.26%) |
:arrow_down: |
OSX | 80.28% <72.17%> (-0.26%) |
:arrow_down: |
v3.6 | 80.23% <72.17%> (-0.26%) |
:arrow_down: |
v3.7 | 80.23% <72.17%> (-0.26%) |
:arrow_down: |
v3.8 | 80.28% <72.17%> (-0.26%) |
:arrow_down: |
v3.9 | 80.28% <72.17%> (-0.26%) |
:arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
pyhmmer/plan7.pyx | 73.68% <70.63%> (+0.07%) |
:arrow_up: |
pyhmmer/easel.pyx | 85.64% <100.00%> (-0.10%) |
:arrow_down: |
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 eb6fe7c...3d79ab4. Read the comment docs.
Based on tests made after issue #12 I realised the implementation of
nhmmer
was inconsistent with the binary provided with HMMER3. This PR does the following:LongTargetsPipeline
to use when searching long nucleotide sequences, using a sliding window over the target instead of loading everything.nhmmer
to useLongTargetsPipeline
instead ofPipeline
Pipeline.search_hmm
also accept a queryProfile
orOptimizedProfile
Pipeline.search_seq
andPipeline.search_msa
to usePipeline.search_hmm
instead of duplicating code.