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

Remove hard-coded default list of model aliases in magic commands #795

Open
JasonWeill opened this issue May 20, 2024 · 2 comments
Open
Labels
enhancement New feature or request @jupyter-ai/magics

Comments

@JasonWeill
Copy link
Collaborator

Problem

A fresh installation of Jupyter AI has the following aliases predefined. Several of these are obsolete or little-used, and by pre-defining these aliases, we developers seem to be endorsing these models.

Name Target
gpt2 huggingface_hub:gpt2
gpt3 openai:davinci-002
chatgpt openai-chat:gpt-3.5-turbo
gpt4 openai-chat:gpt-4
ernie-bot qianfan:ERNIE-Bot
ernie-bot-4 qianfan:ERNIE-Bot-4
titan bedrock:amazon.titan-tg1-large

Proposed Solution

Remove these aliases from the default Jupyter AI configuration. This may be a breaking change, if users rely on these aliases.

Additional context

Users could not add custom model aliases when Jupyter AI first launched (#45) but this was added with #136, in June 2023.

@JasonWeill JasonWeill added enhancement New feature or request @jupyter-ai/magics labels May 20, 2024
@aroffe99
Copy link
Contributor

To avoid breaking changes with deleting these aliases, I think a nice feature would be for users to specify aliases in the startup configuration.

This would allow users to specify these aliases (amongst others they might always use) in the configuration instead of running the %ai register command in their notebooks. And those aliases will be available without the need for re-registering them in a notebook.

@krassowski
Copy link
Member

It is possible to add traitlet configuration to magics, in IPython core this is done for %store magic here and then user just sets c.StoreMagics.autorestore = True in their config file. I think these aliases should be moved to a Dict traitlet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request @jupyter-ai/magics
Projects
None yet
Development

No branches or pull requests

3 participants