-
-
Notifications
You must be signed in to change notification settings - Fork 102
[Platform] Add Perplexity #388
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
Conversation
2c703eb
to
c3452f3
Compare
Looks a bit like you started off with an old base branch? |
c3452f3
to
9a26b07
Compare
9a26b07
to
0cc04c4
Compare
While working on Perplexity specific features integration, I'm wondering how could we handle Related Perplexity API reference: https://docs.perplexity.ai/api-reference/chat-completions-post#response-search-results It's very specific to Perplexity platform, as they perform real web search internally (without leveraging MCP tools). I see 2 possible solutions:
I'd prefer the 1st one. WDYT @chr-hertel @OskarStark |
80fae92
to
6e9b56d
Compare
Let's go with Metadata first and see if there are similar things in other platforms before creating a new kind of result. It is still text right, but they add the search_results and citations to provide some extra information to the text that was generated, right? |
Yes I would prefer the metadata option too in the first version |
It looks like this:
|
c4af78a
to
fe54779
Compare
bac56dc
to
0595bc0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but we are lacking tests
From a functional point of view this looks good: After addressing Oskar's comments we're good to merge here - finally Perplexity - thanks already @welcoMattic! |
0595bc0
to
b33bedc
Compare
@OskarStark @chr-hertel comments addressed, tests added, ready for a final review and merge! |
Just some minor, afterwards good to merge |
b33bedc
to
b3bd2b2
Compare
cc7f142
to
f431dcc
Compare
f431dcc
to
5e7f2f2
Compare
Thanks Mathieu for working on this feature, this is much appreciated. |
This PR adds Perplexity as a supported Platform.
Models are: sonar, sonar-pro, sonar-reasoning, sonar-reasoning-pro, and sonar-deep-research
API is similar to OpenAI API, with additional features I will take care of in coming days.
Perplexity docs: