-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Cleanup Vertex + Chat #2553
base: main
Are you sure you want to change the base?
Cleanup Vertex + Chat #2553
Conversation
pub(crate) struct VertexChat { | ||
messages: Vec<Message>, | ||
// Messages is ignored there. | ||
parameters: VertexParameters, |
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.
Shouldn't this be Option<VertexParameters>
, how is the following payload handled?
{"intances": [{"messages": [{"role": "user", "content": "What's Deep Learning?"}]}]}
Thanks in advance!
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.
Good point ! Fixed it and added the test
use crate::{Message, MessageContent}; | ||
|
||
#[test] | ||
fn vertex_deserialization() { |
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.
Can we add a test when parameters
is not provided too?
What does this PR do?
Fixes # (issue)
Before submitting
Pull Request section?
to it if that's the case.
documentation guidelines, and
here are tips on formatting docstrings.
Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.