-
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
UX principles around extremely large collections of signatures #1350
Comments
From an internal API perspective, we should be careful to not resolve generators and iterators to concrete lists unless we absolutely have to. Right now we do do that for the Selectors #1072 can also be introduced consistently thruout. |
Finally found the conversations on some of the underlying implementation issues around "lazy" and operating at scale -- Lazy loading LinearIndex? |
oh, interesting thought, manifest-containing Index objects could support progress bars... #1082 |
I keep on revisiting this in my head and realizing it's not a good idea, but then never writing it down. Linking all of this to #1877, which is moving away from progress bars and iteration and towards generic interfaces that are also fast. |
interestingly, the only remaining bit here is covered by #1426. Closing! 🎉 |
Building off of zip file collections specifically and the new
load_file_as_signatures
code more generally, we should be considering how to improve the user experience associated with large collections of signatures. Off the top of my head,I think the proper place to put stuff like this is mostly in
sourmash_args
which is the generic place for "utility functions for CLI stuff"The text was updated successfully, but these errors were encountered: