-
Notifications
You must be signed in to change notification settings - Fork 2
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
chore: Docs add model version and api version #188
Conversation
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.
LGTM.
const response = await orchestrationClient.chatCompletion( | ||
{ | ||
inputParams: { country: 'France' } | ||
}, | ||
{ | ||
headers: { | ||
// Add more headers here | ||
}, | ||
params: { | ||
// Add more parameters here | ||
} | ||
// Add more request configuration here | ||
} | ||
); |
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.
Could you add this under a section Customizing the Request Configuration
?
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.
Context
AI/gen-ai-hub-sdk-js-backlog#139.
Add documentation for
model_version
in orchestrationrequestConfig
inDefinition of Done
[ ] Code is tested (Unit, E2E)[ ] Error handling created / updated & covered by the tests above[ ] (Optional) Aligned changes with the Java SDK[ ] (Optional) Release notes updated