Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Multi model chat #4993

Merged
merged 8 commits into from
Mar 30, 2023
Merged

Multi model chat #4993

merged 8 commits into from
Mar 30, 2023

Conversation

mojtaba-komeili
Copy link
Contributor

Patch description
An extension of the model chat which runs in non-turn-based multi-party mode. In the current form it runs a chat between one human and two other agents controlled by the model.

@@ -157,7 +157,9 @@ class BaseModelChatBlueprintArgs(ParlAIChatBlueprintArgs):
)
allowed_worker_qualification: Optional[str] = field(
default=None,
metadata="The qualification name for the workers that are exclusively allowed to do the HITs from this task.",
metadata={
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This must fix the failing crowdsourcing tests.

Copy link
Contributor

@klshuster klshuster left a comment

Choose a reason for hiding this comment

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

the python code looks good to me! deferring to other reviewers for .jsx

@@ -0,0 +1,67 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

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

do we generally open-source our onboarding ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We had it at least for the Wizard of Internet as far as I remember.

@mojtaba-komeili mojtaba-komeili merged commit af9f3a0 into main Mar 30, 2023
@mojtaba-komeili mojtaba-komeili deleted the multi-model_chat branch March 30, 2023 16:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants