Add switch_llm function to manager for runtime model switching #169
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR adds a
switch_llm
function to the manager that allows changing the model during execution.Changes
switch_llm()
function in the managerKnown Issue
However, there's a persistent problem:
When
TextFrame
objects are received byLLMUserContextAggregator
orLLMAssistantContextAggregator
, a loop is created and text gets added multiple times to the context.Example:
Hello my name is joe.
Hello Hello my Hello my name Hello Hello Hello my name is
Help Needed
If someone could help me progress on this issue, I would appreciate it. I've tried adding logs everywhere in the Aggregators but I still don't understand where this is coming from.
Testing
Note: This PR is functional but the text duplication issue in the context aggregators needs to be resolved before merging.