-
Notifications
You must be signed in to change notification settings - Fork 183
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
Remove voyageai from anthropic drivers config #1277
Conversation
ccc87fe
to
637f71c
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
MIGRATION.md
Outdated
@@ -21,7 +55,7 @@ def handler_fn_stream_text(event: CompletionChunkEvent) -> None: | |||
|
|||
EventListener(handler=handler_fn_stream, event_types=[CompletionChunkEvent]) | |||
EventListener(handler=handler_fn_stream_text, event_types=[CompletionChunkEvent]) | |||
``` | |||
```` |
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.
intentional 4th tick?
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.
oops nope, updated
14b7f25
to
d7a7ad9
Compare
vector_store_driver=LocalVectorStoreDriver( | ||
embedding_driver=VoyageAiEmbeddingDriver() | ||
) | ||
) |
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 add the agent line here, not really critical
Describe your changes
Changed
AnthropicDriversConfig
no longer bundlesVoyageAiEmbeddingDriver
.This default configuration has been a source of confusion for users.
Issue ticket number and link
https://discord.com/channels/1096466116672487547/1250191293439676526/1298130335711891476
📚 Documentation preview 📚: https://griptape--1277.org.readthedocs.build//1277/