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.

Sign up for GitHub

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

Jump to bottom

Update PersonasProvider to use default personas when none are configured #15

Merged
merged 1 commit into from
Jan 17, 2025
Merged

Update PersonasProvider to use default personas when none are configured #15

merged 1 commit into from
Jan 17, 2025

Conversation

NeonDaniel
Copy link
Member

Description

Updates persona init to prevent loading an LLM with no personas, making the LLM inaccessible in Klat

Issues

Other Notes

This is an imperfect patch; persona management in general needs a better spec for how to manage:

  • user-defined personas
  • deployment-defined (default?) personas
  • llm-defined specific personas (i.e. BrainForge LLMs)

@NeonDaniel
Copy link
Member Author

Deployed with chatgpt in the Alpha namespace

@NeonDaniel NeonDaniel marked this pull request as ready for review January 17, 2025 00:43
@NeonDaniel NeonDaniel requested a review from NeonKirill January 17, 2025 00:43
@NeonDaniel NeonDaniel mentioned this pull request Jan 17, 2025
Merged
@NeonDaniel NeonDaniel merged commit 06ca6a4 into dev Jan 17, 2025
7 checks passed
@NeonDaniel NeonDaniel deleted the REF_UseConfiguredPersonasByDefault branch January 17, 2025 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NeonKirill NeonKirill NeonKirill approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@NeonDaniel @NeonKirill