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

Static community assistants #319

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

pilitz
Copy link
Contributor

@pilitz pilitz commented Feb 26, 2025

Description

  • community bots are now read only
  • static bots can be configured with .env
  • createBotConfig overwrites existing id

@pilitz pilitz requested a review from Meteord February 26, 2025 08:26
Copy link
Contributor

@Meteord Meteord left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, in addition static_bots.ts can be deleted

quick_prompts= self._get_quick_prompt_list(index),
temperature= float(getenv(f"{prefix}TEMPERATURE", 0.7)),
max_output_tokens= int(getenv(f"{prefix}MAX_TOKENS", 100)),
id = str(getenv(f"{prefix}ID", uuid())),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wouldn't provide an a default id with uuid() if no id is present in the env. Should throw an error in this case.

Otherwise we generate a new random id every server restart and we will have a lot of duplicated community assistents.

@@ -76,13 +79,24 @@ export const Layout = () => {
console.error("Keine Modelle vorhanden");
}
setLLM(result.models.find(model => model.llm_name == llm_pref) || result.models[0]);
for (let bot of result.frontend.community_assistants) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this necessary?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants