Skip to content

fix Update qdrant_impl.py #1253

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

fix Update qdrant_impl.py #1253

wants to merge 3 commits into from

Conversation

towpi13
Copy link

@towpi13 towpi13 commented Apr 2, 2025

Description

this solve the issue #909 related to get_func_id

Related Issues

this should also solve this issue #1144

Changes Made

I just added the missing functions in the QdrantVectorDBStorage class

Checklist

  • Changes tested locally
  • Code reviewed
  • Documentation updated (if necessary)
  • Unit tests added (if applicable)

Additional Notes

[Add any additional notes or context for the reviewer(s).]

Copy link
Collaborator

@danielaskdd danielaskdd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for sharing this. There are a few conflicts that need to be resolved before we can proceed with the review.

@@ -998,7 +998,7 @@ async def get_kg_context():

async def get_vector_context():
# Consider conversation history in vector search
augmented_query = query
augmented_query = f"Instructions: Based on the user query, provide a list of relevant documents that are most likely to contain the answer to the user's query.\nQuery: {query}"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@LarFii Please review whether the changes here are reasonable.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh I forgot to exclude that part . that doesn't relate to the main issue. i'm using an instruct version of the e5 model and that format is required for it to function properly. I hope there's a higher level config that allows user to easily do that without modifying the internal module.

@danielaskdd
Copy link
Collaborator

Thanks for sharing. Kindly resolve the conflicts and exclude changes to operate.py.

Copy link
Author

@towpi13 towpi13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reverted the operate.py to its original vresion

@danielaskdd
Copy link
Collaborator

please merge main branch and resolve the conflicts.

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