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

Lazy load the LLM modules to relax import requirements #69

Merged
merged 2 commits into from
Dec 21, 2024

Conversation

markbackman
Copy link
Contributor

Resolve this issue: #64.

Copy link

vercel bot commented Dec 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
pipecat-flows ✅ Ready (Inspect) Visit Preview Dec 21, 2024 1:36pm

@markbackman markbackman changed the title Lazy load the LLM modules to resolve the imports issues Lazy load the LLM modules to relax import requirements Dec 20, 2024
@aconchillo
Copy link

I think we should move everything the adapters do to Pipecat. Pipecat Flows shouldn't need to know about any specific LLM service, simply the LLMService class.

@markbackman markbackman merged commit dc19d8b into main Dec 21, 2024
2 checks passed
@markbackman markbackman deleted the mb/fix-imports branch December 21, 2024 13:37
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

Successfully merging this pull request may close these issues.

2 participants