- Issue created by @Vivek Panicker
- Merge request !51Issue #3470417: Added integration with Pinecone. → (Merged) created by Vivek Panicker
- 🇬🇧United Kingdom scott_euser
I still need to do a further review sorry. In any case looks like a good start! Will continue reviewing and testing asap.
- Status changed to Needs work
4 months ago 4:26am 4 September 2024 - 🇬🇧United Kingdom scott_euser
Added some more comments after further testing. Thank you!
- 🇬🇧United Kingdom scott_euser
@vivekp27 I think we should consider using https://github.com/probots-io/pinecone-php - I haven't tried it myself but it looks promising. Would allow us to remove the Pinecone service + seems to be handling oddities in the Pinecone API. Would need to be added to composer.json of the ai project root if so.
I know this is quite a big issue already so feel free to pass that bit of scope to me for example if it's getting too much for you (same applies to any of this really).
- Assigned to scott_euser
- 🇬🇧United Kingdom scott_euser
Still a WIP, but pushed a significant update. Will continue next week.
-
scott_euser →
committed d00ce0f8 on 1.0.x authored by
vivek panicker →
Issue #3470417 by scott_euser, vivek panicker: Provide integration with...
-
scott_euser →
committed d00ce0f8 on 1.0.x authored by
vivek panicker →
-
scott_euser →
committed d00ce0f8 on mock-stream-response authored by
vivek panicker →
Issue #3470417 by scott_euser, vivek panicker: Provide integration with...
-
scott_euser →
committed d00ce0f8 on mock-stream-response authored by
vivek panicker →
Automatically closed - issue fixed for 2 weeks with no activity.