Skip to content
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] add max_containment column #76

Merged
merged 11 commits into from
Sep 1, 2023
Merged

[MRG] add max_containment column #76

merged 11 commits into from
Sep 1, 2023

Conversation

mr-eyes
Copy link
Member

@mr-eyes mr-eyes commented Aug 30, 2023

resolves #75

also: because of #85,

Fixes #83
Fixes #77
Fixes #84

@mr-eyes mr-eyes requested a review from ctb August 30, 2023 01:41
Copy link
Collaborator

@ctb ctb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add a test for the specific numbers output, over in src/python/tests/test_search.py. thanks!

@ctb
Copy link
Collaborator

ctb commented Sep 1, 2023

tests added in #85

ctb and others added 5 commits September 1, 2023 08:23
* [MRG] add mastiff interface functions (#58)

* try adding rdb index and manysearch

* init testing

* use pathlist loading for better errs; more tests

* also check intersect_hashes

* add test for index check

* add multiquery mastiff gather

* init mastiff gather testing

* remove original single-query mastiff search, gather

* more cleanup

* MRG: fix `if let` warnings (#63)

* fix threads for changes from main

* rm threshold

* [MRG] allow index from a signature zipfile (#74)

* zipfile hackaround

* fix

* fix tests

* clean up; unify search testing; pin core to commit

* upd py toml

* test index zip

* add some indexed fastmultigather testing

* add cargo lock

* more index tests

* indexed multigather tests

* revert to branch while trying upds

* better help; avoid recalc threshold

* EXP: try fix CI for rocksdb (#80)

* Add trial workflow

* ok weird removing sourmash

* try again

* do the test

* remove maturin CI for the moment

* try caching rust build stuff

* fix yaml syntax

* test actions

---------

Co-authored-by: C. Titus Brown <[email protected]>

* improve gather output

* cargo lock

* re add jaccard

* added test for max cont

* version and cite

* rm warning

* bump versions

---------

Co-authored-by: Tessa Pierce Ward <[email protected]>
@ctb ctb merged commit 0856744 into main Sep 1, 2023
1 check passed
@ctb ctb deleted the max_containment branch September 1, 2023 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants