Enable Base API URL for /learn

jupyterlab / jupyter-ai

A generative AI extension for JupyterLab

https://jupyter-ai.readthedocs.io/

BSD 3-Clause "New" or "Revised" License

3.26k stars 337 forks source link

Enable Base API URL for /learn #977

Open saiwing-yeung opened 3 months ago

saiwing-yeung commented 3 months ago

Please enable Base API URL for /learn.

Problem

I am using a Base API URL that points to an internal OpenAI endpoint. Features like /ask works great, but /learn doesn't seem to use the customized API URL (I'd get an authentication error).

Proposed Solution

/learn would use the Base API URL.

Additional context

Using 2.22.0

dlqqq commented 6 days ago

Thank you for reporting this issue, and sorry this went undiscovered for so long! I'll assign this issue to another contributor and add it to a milestone for tracking.

For contributors: We need a Base API URL optional field for all the OpenAI embedding providers defined in openai.py.

dlqqq commented 6 days ago

BTW, this is a duplicate of #587.