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

Adds Perplexity AI #261

Merged
merged 9 commits into from
Mar 6, 2025
Merged

Adds Perplexity AI #261

merged 9 commits into from
Mar 6, 2025

Conversation

fbettag
Copy link
Contributor

@fbettag fbettag commented Feb 23, 2025

Also adds Tool Calls by using a workaround with perplexity's structured output.

Tool Calls in this are by far not perfect yet, but it's a start.
Perplexity is really iffy in a) returning only structured data and b) liking what we send for some reason.

@reetou
Copy link

reetou commented Mar 6, 2025

Hey @brainlid , are there any plans to merge this? Or maybe help needed?

brainlid added 2 commits March 6, 2025 10:44
- fix failing test (is live)
- add ENV template for perplexity
- fixed for CI
@brainlid brainlid merged commit 784bdb8 into brainlid:main Mar 6, 2025
1 check passed
@brainlid
Copy link
Owner

brainlid commented Mar 6, 2025

Thanks @fbettag for the contribution! Merged!
❤️💛💙💜

Thanks @reetou for the nudge to get it prepped.

@fbettag
Copy link
Contributor Author

fbettag commented Mar 6, 2025

no worries dude, thanks for the library!

@reetou tool calls are still iffy, though the tests pass. Patches are welcome :) also keep in mind you need to still add the prompt as outlined in the readme regardless of the tool calls you pass in. (it's not ideal from the perplexity side)

@fbettag fbettag deleted the pr/adds-perplexity branch March 6, 2025 17:50
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.

3 participants