Skip to content

Commit

Permalink
Fix settings update and vertical scroll (#249)
Browse files Browse the repository at this point in the history
* Fix for propagating provider param updates

* Adds vertical scroll for settings panel

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
  • Loading branch information
3coins and pre-commit-ci[bot] committed Jul 5, 2023
1 parent f84e018 commit f4389f6
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 4 deletions.
13 changes: 10 additions & 3 deletions packages/jupyter-ai/jupyter_ai/actors/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -88,14 +88,18 @@ def get_llm_chain(self):
else None
)

if not lm_provider:
if not lm_provider or not lm_provider_params:
return None

if curr_lm_id != next_lm_id:
self.log.info(
f"Switching chat language model from {curr_lm_id} to {next_lm_id}."
)
self.create_llm_chain(lm_provider, lm_provider_params)
elif self.llm_params != lm_provider_params:
self.log.info("Chat model params changed, updating the llm chain.")
self.create_llm_chain(lm_provider, lm_provider_params)

return self.llm_chain

def get_embeddings(self):
Expand All @@ -104,10 +108,13 @@ def get_embeddings(self):
embedding_params = ray.get(actor.get_provider_params.remote())
embedding_model_id = ray.get(actor.get_model_id.remote())

if not provider:
if not provider or not embedding_params:
return None

if embedding_model_id != self.embedding_model_id:
if (
embedding_model_id != self.embedding_model_id
or self.embeddings_params != embedding_params
):
self.embeddings = provider(**embedding_params)

return self.embeddings
Expand Down
3 changes: 2 additions & 1 deletion packages/jupyter-ai/src/components/chat-settings.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -289,7 +289,8 @@ export function ChatSettings(): JSX.Element {
sx={{
padding: 4,
boxSizing: 'border-box',
'& > .MuiAlert-root': { marginBottom: 2 }
'& > .MuiAlert-root': { marginBottom: 2 },
overflowY: 'auto'
}}
>
{state === ChatSettingsState.SubmitError && (
Expand Down

0 comments on commit f4389f6

Please sign in to comment.