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

Add support for Perplexity-style citations #262

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SplittyDev
Copy link
Contributor

@SplittyDev SplittyDev commented Feb 7, 2025

What

This PR adds support for Perplexity's citations list.

See https://docs.perplexity.ai/api-reference/chat-completions for the docs.
Note: While Perplexity declares the type as any[], in practice these are strings.

Why

In order to extend support to other compatible providers, as discussed in #259 (comment).

Affected Areas

Minimal non-breaking changes to ChatResult and ChatStreamResult.

Copy link

sonarqubecloud bot commented Feb 7, 2025

@SplittyDev
Copy link
Contributor Author

I'll update the tests shortly.

@nezhyborets
Copy link
Collaborator

nezhyborets commented Feb 7, 2025

Great! Let's also add a section to doc about supporting different providers. No need to sweat about it, just introduce the section and write a short note about the change in this PR. I'm going to put more general info there later, like "to use a different provider you can set host, specify model" etc

@nezhyborets
Copy link
Collaborator

@SplittyDev would you be able to do a fix so the check passes?

@SplittyDev
Copy link
Contributor Author

@SplittyDev would you be able to do a fix so the check passes?

Sure, but it might take a few days. I don't have much time at the moment, but I'll get back to this as soon as I can

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