New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MRG: upgrade sig overlap
and sig subtract
to load more than JSON signatures
#3153
base: latest
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## latest #3153 +/- ##
==========================================
+ Coverage 86.66% 86.67% +0.01%
==========================================
Files 136 136
Lines 15814 15828 +14
Branches 2713 2713
==========================================
+ Hits 13705 13719 +14
Misses 1799 1799
Partials 310 310
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
for more information, see https://pre-commit.ci
sig overlap
and sig subtract
sig overlap
and sig subtract
Ready for review @sourmash-bio/devs |
sig overlap
and sig subtract
sig overlap
and sig subtract
to load more than JSON signatures
@ccbaumler @AnneliektH would either of you be able to look at this and (potentially) approve it? lmk if you don't have review privileges. |
Fix
sig overlap
andsig subtract
to take more than just JSON signatures.Also, adds a function
sourmash_args.load_one_signature
that I think should (eventually) replace the now-deprecatedsourmash.signature.load_one_signature
. This will be the topic of a new PR - for now, I think it's a nice quick fix!Fixes #3136
Related issues:
load_one_signature
useload_file_as_signatures
API #1062 - will do another PR to close this issueload_*
sig/db functions insourmash_args.py
#1877TODO: