-
Notifications
You must be signed in to change notification settings - Fork 354
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
[FEATURE] [SDK] chat field support sdk #5378
[FEATURE] [SDK] chat field support sdk #5378
Conversation
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
@@ -31,11 +31,12 @@ class ImageFieldSettings(BaseModel): | |||
type: Literal["image"] = "image" | |||
|
|||
|
|||
class ChatFieldSettings(BaseModel): | |||
type: Literal["chat"] = "chat" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we want to include something like user_role
, assitant_role
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think that's necessary for the first iteration. But it would be nice for a later iteration where we allow the user to define the 'chat schema' based on their own data.
Co-authored-by: David Berenstein <david.m.berenstein@gmail.com>
…rgilla-io/argilla into feat/chat-field-support-server
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @burtenshaw, looking good. I think we might want to add this to the from/to/disk/hub methods too.
Sorry @davidberenstein1957 . I don't get this one. How would we add this to those methods? |
Description
This PR implements chat field support on the SDK.
Type of change
How Has This Been Tested
Checklist