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

Refactor in several libs to make this smaller and easier to reason about #144

Open
1 of 4 tasks
rom1504 opened this issue May 12, 2022 · 2 comments
Open
1 of 4 tasks

Comments

@rom1504
Copy link
Owner

rom1504 commented May 12, 2022

Initially this repo was about integrating multiple small components into a simple retrieval system
it now growed a lot and got a lot more features
and it's becoming too big

to grow in a reasonable way, let's extract big components in smaller libs and use them here:

  • clip inference
  • clip back
  • clip filter
  • all-clip

should be independent libs

img2dataset and autofaiss are already independent

@rom1504 rom1504 changed the title split in several packages Refactor in several libs to make this smaller and easier to reason about May 21, 2022
@rom1504 rom1504 pinned this issue May 21, 2022
@rom1504
Copy link
Owner Author

rom1504 commented Jun 15, 2022

Now even more important with clip client has it wouldn't need much dependencies at all if it was its own module

@dmarx
Copy link

dmarx commented Jul 19, 2022

i might tinker with this refactor

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

No branches or pull requests

2 participants