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.

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

fix: RAG service Dockerfile path patch #767

Merged
merged 3 commits into from
Dec 9, 2024

Conversation

bangqipropel
Copy link
Collaborator

@bangqipropel bangqipropel commented Dec 9, 2024

Reason for Change:
Docker file path fixed

Requirements

  • added unit tests and e2e tests (if applicable).

Issue Fixed:
if we don't name the path as ragengine, it will report the following error:

Traceback (most recent call last):
  File "/app/services/main.py", line 5, in <module>
    from vector_store_manager.manager import VectorStoreManager
  File "/app/services/vector_store_manager/manager.py", line 6, in <module>
    from ragengine.models import Document
ModuleNotFoundError: No module named 'ragengine'

Notes for Reviewers:

Signed-off-by: Bangqi Zhu <bangqizhu@microsoft.com>
@ishaansehgal99 ishaansehgal99 merged commit 857c373 into main Dec 9, 2024
3 of 6 checks passed
@ishaansehgal99 ishaansehgal99 deleted the bangqipropel/rag_docker_patch branch December 9, 2024 22:20
@ishaansehgal99 ishaansehgal99 changed the title fix: RAG service Dockerfile path patch fix: RAG service Dockerfile path Dec 9, 2024
@ishaansehgal99 ishaansehgal99 changed the title fix: RAG service Dockerfile path fix: RAG service Dockerfile path patch Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants