8000 SIM-2935: Fix a bug that was causing the OpenAI model to always be overridden. by fablechris · Pull Request #56 · fablestudio/fable-saga · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

SIM-2935: Fix a bug that was causing the OpenAI model to always be overridden. #56

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

Merged
merged 1 commit into from
May 8, 2024

Conversation

fablechris
Copy link
Contributor

No description provided.

Copy link
Contributor
@frankcarey frankcarey left a comment

Choose a reason for hiding this comment

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

LGTM

@frankcarey frankcarey merged commit 0e4b76b into main May 8, 2024
1 check passed
@frankcarey frankcarey deleted the SIM-2935-model-override-bug branch May 8, 2024 15:48
frankcarey added a commit to fablestudio/thistle-gulch that referenced this pull request May 8, 2024
Requires fablestudio/fable-saga#56

- Allow up to Python 3.13
- Load bridge_config.yaml from disk to set the host, port, cors, runtime
path, and LLM models.
- Read llm config from bridge_config.yaml. 
- Dynamically load llms based on runtime configuration.
- Make sure to close the runtime if there is an exception.
- Create a config file for personas.
- Add memories to the persona config. 
- Use api to initialize all personas from config.
- Pass the world context to on_ready. 
- Remove request model overrides so the Runtime can no longer change the
model type.
- Allow multiple character properties to be updated in a single API
call.
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
0