Add OpenAI provider #19
Conversation
|
Just pushed to resolve the |
brichet
force-pushed
the
openAI
branch
2 times, most recently
from
dee9fb2 to
ee784b0
Compare
|
Seems to be working fine locally for both the chat and completer: jupyterlite-ai-openai.mp4For the inline completer, there seems to be some issues similar to the other providers. Maybe there is something to be done to improve the quality of the suggestions in general, by checking the response against the input used when the completer was triggered. |
| } | ||
| return 'Unknown provider'; | ||
| } | ||
|
|
||
| /* | ||
| * Get an LLM completer from the name. | ||
| */ | ||
| export function getSettings(name: string): JSONObject | null { | ||
| export function getSettings(name: string): any { |
There was a problem hiding this comment.
Maybe this any is fine for now, but we should maybe check if we can put JSONObject | null back as the return type.
|
Marking as ready for now since it seems to be functional. |
I'm not clear about the usage of the |
|
LGTM, I can't approve it |
No description provided.