DocsGPT
DocsGPT copied to clipboard
Add Oracle DB as vector store
- What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) feature
- Why was this change needed? (You can also link to an open issue here) closes #981
@IlyasOsman is attempting to deploy a commit to the Arc53 Team on Vercel.
A member of the Team first needs to authorize it.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| docs-gpt | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jun 18, 2024 10:28am |
Hi @IlyasOsman , I'm getting an error :
TypeError: OracleDBVectorStore.__init__() got an unexpected keyword argument 'path'
whenever trying to upload any doc. This is because the path field hasn't been accounted for in your class.
just a few comments:
- please filter on source_id during search (you can get it on init) check out other vectorstores
- Lets lazy load our dependency
Thank you, very sorry for delays. Also where is best to test this db?
Yeah that path comment is now source_id @IlyasOsman similar to 'path' @siiddhantt mentioned above
Hey @IlyasOsman is there any update? thank you