-
Notifications
You must be signed in to change notification settings - Fork 80
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
make load_one_signature
use load_file_as_signatures
API
#1062
Comments
load_file_as_signatures
APIload_file_as_signatures
API
ok, dug into this a little bit when working on #1279.
My current hot take is that
I'm not sure how to rename them tho. |
load_file_as_signatures
APIload_one_signature
use load_file_as_signatures
API
Huh. Working on this over in #3153, and I noticed that |
aaaand
so we've done a great job of removing this from the core sourmash code. (Both functions in |
This is being cleaned up a lot in #3161 which will -
|
This will be fixed when #3153 and #3161 are merged -
|
…signatures (#3153) Fix `sig overlap` and `sig 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-deprecated `sourmash.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: * #1062 - will do another PR to close this issue * #1877 * #1312 * #1060 TODO: - [x] test uncovered code - [x] do a bit more of a search and digest of related issues to see if there's other low hanging fruit --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Note: PR into #3153 Tackles some signature loading and saving cleanup throughout the codebase. Most changes are in the tests, and this is a significant cleanup of the test code! Fixes #1062. --- Goals: * deprecate external use of `sourmash.signature` load/save functions, because they are JSON-specific and inflexible. * simplify and standardize signature load/save function usage during tests; * get rid of deprecation messages during tests; In brief, * rename `sourmash.signature.load_signatures` to `load_signatures_from_json`; * rename `sourmash.signature.load_one_signature` to `load_one_signature_from_json`; * rename `sourmash.signature.save_signatures` to `save_signatures_to_json`; * deprecate `sourmash.save_signatures` and `sourmash.load_one_signature` for 5.0 (joining `load_signatures`, which was already deprecated); * reduce/eliminate deprecations by transitioning internal test code to use these three functions directly from `sourmash.signature` instead of from the top-level sourmash import. * **bonus**: eliminate zipfile UserWarning around overwriting files, which causes lots of warnings when running tests. --- Done: - [x] in sourmash.signature submodule, rename `load_signatures` to `load_signatures_from_json`, `load_one_signature` to `load_one_signatures_from_json`, and `save_signatures` to `save_signatures_to_json`; make tests pass. - [x] deprecate `sourmash.load_one_signature` and `sourmash.save_signatures`. - [x] catch zipfile UserWarning for duplicate filenames in ZipStorage.save TODO: - [x] transition internal sourmash code+tests away from deprecated functions - [ ] create issue around changing API documentation prior to 5.0; --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
right now it's based on
signature.load_signatures
which is less flexible thansourmash_args.load_file_as_signatures
.also examine
load_query_signature
and see if there's duplication.The text was updated successfully, but these errors were encountered: