Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Data Loaders] - Should not ask for filePath when providing docs or splitDocs #33

Closed
2 tasks done
pranav-kural opened this issue Jul 14, 2024 · 0 comments · Fixed by #34
Closed
2 tasks done

[Data Loaders] - Should not ask for filePath when providing docs or splitDocs #33

pranav-kural opened this issue Jul 14, 2024 · 0 comments · Fixed by #34
Assignees
@pranav-kural
Labels
bug Something isn't working

Comments

@pranav-kural
Copy link
Collaborator

Affected components(s)

  • [] API Key Authentication
  • [] Chat Agents
  • Chat Endpoints
  • [] Chat History
  • [] Embedding Models
  • [] Endpoint Deployment
  • [] LLM Models
  • RAG
  • [] Response Caching
  • [] Vector Stores
  • [] Other

Describe the bug
When configuring retriever for endpoint, it should not ask for filePath when providing docs or splitDocs.
@pranav-kural pranav-kural added the bug Something isn't working label Jul 14, 2024
@pranav-kural pranav-kural self-assigned this Jul 14, 2024
pranav-kural added a commit that referenced this issue Jul 15, 2024
@pranav-kural
fix for #33 - refactored code for data retriever
5efbe78
pranav-kural added a commit that referenced this issue Jul 15, 2024
@pranav-kural
Bumped package version #33
b85961d
@pranav-kural pranav-kural mentioned this issue Jul 15, 2024
Merged
@pranav-kural pranav-kural linked a pull request Jul 15, 2024 that will close this issue
Fix for the issue where filePath was still required when providing docs in retriever config #33 #34
Merged
pranav-kural added a commit that referenced this issue Jul 15, 2024
@pranav-kural
Fix for the issue where filePath was still required when providing …
987cbdb 
…`docs` in retriever config #33 (#34)

* fix for #33 - refactored code for data retriever

* Bumped package version #33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pranav-kural pranav-kural

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
1 participant
@pranav-kural