-
Notifications
You must be signed in to change notification settings - Fork 77
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: transition internal signature loading functions #3161
Conversation
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## latest #3161 +/- ##
=======================================
Coverage 86.66% 86.66%
=======================================
Files 136 136
Lines 15821 15830 +9
Branches 2713 2716 +3
=======================================
+ Hits 13711 13719 +8
- Misses 1800 1801 +1
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
This feels done! It changes a lot of code in dumb ways, and cleans up the warnings emitted by the test, too; probably shouldn't be combined with any more changes 😅 Ready for review, @sourmash-bio/devs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
**Note:** PR into #3161 This PR cleans up and regularizes naming output on `sig` subcommands. Specifically, it: - [x] adds `--set-name` to `sig intersect` and `sig subtract` to set names on output signatures. - [x] confirms and tests default names for output from `sig inflate`, `sig filter`, `sig downsample`, and `sig flatten` - [x] updates documentation appropriately - [x] aliases `--name` to `--set-name` on `sig merge`, and nominates `--name` for deprecation and removal on v5 - [x] nominates `--name` on `sig extract`, `sig filter`, and `sig flatten` for deprecation and removal on v5 - [x] highlights `--name` on `sketch dna`, `sketch protein`, and `sketch translate` - ref #3152 Fixes #1801 Fixes #3152 Related issues: * #3173 --------- 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:
sourmash.signature
load/save functions, because they are JSON-specific and inflexible.In brief,
sourmash.signature.load_signatures
toload_signatures_from_json
;sourmash.signature.load_one_signature
toload_one_signature_from_json
;sourmash.signature.save_signatures
tosave_signatures_to_json
;sourmash.save_signatures
andsourmash.load_one_signature
for 5.0 (joiningload_signatures
, which was already deprecated);sourmash.signature
instead of from the top-level sourmash import.Done:
load_signatures
toload_signatures_from_json
,load_one_signature
toload_one_signatures_from_json
, andsave_signatures
tosave_signatures_to_json
; make tests pass.sourmash.load_one_signature
andsourmash.save_signatures
.TODO: