Skip to content

Conversation

@eligotts
Copy link
Contributor

Description

State['client'] now becomes a dictionary mapping from str -> client object
we then need to add a new field to State to track what the current client is
in init_state, we handle whether the user passed in a dictionary for the llm client mapping or a single client
add a get_next_client function to multi_turn_env to implement client switching logic
decided that get_prompt_messages should own this model switching logic, but not sure if that is best option
for more robust solution, would need prompt message rewrite logic according to what client we are on

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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