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

Reduced dependency size #654

Closed
alita-moore opened this issue Oct 18, 2023 · 4 comments
Closed

Reduced dependency size #654

alita-moore opened this issue Oct 18, 2023 · 4 comments
Labels
Enhancement New feature or request

Comments

@alita-moore
Copy link

Is your feature request related to a problem? Please describe.

I want to host this on a serverless function, but the dependency are too large. It seems that the biggest offenders are torch and semgrep. But are these actually required for this package to operate?

Describe the solution you'd like

I would like a much smaller package size. Something that focuses on brevity and isolates the core functionality. Utilizing a lightweight proxy for llm requests. This is to avoid dependency on the heavy libraries like openai.

Describe alternatives you've considered

utilizing serverless-python and unzipping. But this is extremely slow and has other problems like sqlite incompatiblity with chroma.

Additional context

No response

@alita-moore alita-moore added the Enhancement New feature or request label Oct 18, 2023
@KillianLucas
Copy link
Collaborator

KillianLucas commented Oct 18, 2023 via email

@alita-moore
Copy link
Author

well the dependency weight depends on what packages those packages install and so on. So I'm not sure exactly which packages depend on torch, but whichever do would be what's causing problems. Also making the science packages optional (pandas and numpy are installed) would be interesting. Note that I think #655 is a better solution. Perhaps this package should be focused on handling the response from the llm / integrating with multiple llms and then offloading the execution to another project that runs it securely.

@KillianLucas
Copy link
Collaborator

KillianLucas commented Oct 18, 2023 via email

@MikeBirdTech
Copy link
Collaborator

Closing this stale issue. Please create a new issue if the problem is not resolved or explained in the documentation. Thanks!

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

No branches or pull requests

3 participants