-
Notifications
You must be signed in to change notification settings - Fork 3
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 a dedicated settings panel #35
Comments
Maybe it could be worth having a UI similar to what Not sure how reusable the |
Also a dedicated panel would help have better control on the persistence of secrets such as API keys and allow for temporary secrets valid until the next page refresh. Instead of relying on the settings which get stored in the browser in the case of JupyterLite, on the server for JupyterLab. |
Also opened #37, which could be done after the move to a proper panel. The plugin providing the settings panel could then depend on the plugin providing the secrets manager. |
Also maybe it could be useful to provide instructions for using the model in the settings panel directly. For now the README has some information about some of the models, but it could be more practical to put that information somewhere in the settings panel directly (or hidden by default, but can be displayed easily). For example in the case of ChromeAI that would allow putting links to access and enable the required flags. |
Right, this helper could be updated when changing the provider. I'll try to start that settings panel soon. |
Problem
Each provider has its own settings in
@langchainjs
.Therefore, we need to update manually the settings when changing the provider.
Proposed Solution
Have a dedicated AI settings panel that does not use the Jupyterlab settings system, to give more flexibility.
Additional context
Discussion started at #27 (comment)
The text was updated successfully, but these errors were encountered: