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

lca index could use some upgrading #1948

Open
ctb opened this issue Apr 12, 2022 · 2 comments
Open

lca index could use some upgrading #1948

ctb opened this issue Apr 12, 2022 · 2 comments

Comments

@ctb
Copy link
Contributor

ctb commented Apr 12, 2022

First, it doesn't pick up the k-mer size from the available signatures, so you have to specify it explicitly on the command line (unlike sourmash index, where you only need to specify -k if it's not obvious).

Second, when loading from a database it doesn't make use of manifests or progress loading stuff, so it will simply say "loading signatures ... 1 of 1".

So you get

... loading signature GCA_900761165.1 Clostridia bac (1 of 1)

instead of any kind of useful indication of how many are being loaded - ref #1426, #1877

@ctb
Copy link
Contributor Author

ctb commented Apr 12, 2022

actually, after watching sourmash index I think mostly what I want is the output like that:

...loading from 'gtdb-rs207.genomic-reps.dna.k31.zip' / 22340 sigs total

which is a progress indicator rather than a progress bar ;).

@ctb
Copy link
Contributor Author

ctb commented Aug 15, 2022

could be part of #2198 refactoring.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant