DocsGPT icon indicating copy to clipboard operation
DocsGPT copied to clipboard

Add Oracle DB as vector store

Open IlyasOsman opened this issue 1 year ago • 6 comments

  • 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 avatar Jun 12 '24 19:06 IlyasOsman

@IlyasOsman is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Jun 12 '24 19:06 vercel[bot]

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

vercel[bot] avatar Jun 18 '24 10:06 vercel[bot]

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.

siiddhantt avatar Jun 19 '24 05:06 siiddhantt

just a few comments:

  1. please filter on source_id during search (you can get it on init) check out other vectorstores
  2. Lets lazy load our dependency

Thank you, very sorry for delays. Also where is best to test this db?

dartpain avatar Oct 05 '24 22:10 dartpain

Yeah that path comment is now source_id @IlyasOsman similar to 'path' @siiddhantt mentioned above

dartpain avatar Oct 05 '24 22:10 dartpain

Hey @IlyasOsman is there any update? thank you

dartpain avatar Dec 13 '24 11:12 dartpain