-
Notifications
You must be signed in to change notification settings - Fork 2k
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
feat: re-add GenAI embedding code samples #3616
Conversation
Here is the summary of changes. You are about to add 2 region tags.
This comment is generated by snippet-bot.
|
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
Co-authored-by: Patti Shin <[email protected]>
Co-authored-by: Patti Shin <[email protected]>
Co-authored-by: Patti Shin <[email protected]>
* chore: remove .preview from generative ai methods chore: update ai package versions fix: fix broken tests fix: update dependency to use github over npm feat: change assertion to more common output added function calling examples removed double await, fix linter * feat: readd GenAI embedding code samples (#3616) Co-authored-by: Patti Shin <[email protected]> * fix: adding in type:module in package.docs.json (#3624) chore: attempt to fix linting errors chore: attempt to fix linting errors chore: attempt to lint functionCallingStreamChat chore: lint functionCallingStreamContent chore: lint functionCallingStreamContent chore: lint functionCallingStreamContent chore: lint functionCallingStreamContent chore: lint functionCallingStreamContent * fix: Attempt at pleasing the Linter (#3625) * Attempt at pleasing the Linter * fix(linter): trying to please CI --------- Co-authored-by: Patti Shin <[email protected]> * fix: lint --------- Co-authored-by: Eric Dong <[email protected]> Co-authored-by: Patti Shin <[email protected]> Co-authored-by: Guillaume Laforge <[email protected]>
Description
Fixes #3320
Re-add code samples for GenAI embeddings. These samples were published but later removed because of the launch schedule change.