Removed circular dependancy of qdrant-client

sriramr244 commented 1 year ago

Pull Request Title

Related Issue

Description

Type

[X] Bug Fix
[ ] Feature Enhancement
[ ] Documentation Update
[ ] Code Refactoring
[ ] Other (please specify):

Proposed Changes

Changed the long range of version dependancy cohere
Changed the qdrant-cient version dependency to the latest version which works with urllib-2.0.3

Checklist

[X] The code compiles successfully without any errors or warnings
[ X] The changes have been tested and verified
[ ] The documentation has been updated (if applicable)
[X] The changes follow the project's coding guidelines and best practices
[X] The commit messages are descriptive and follow the project's guidelines
[X] All tests (if applicable) pass successfully
[X] This pull request has been linked to the related issue (if applicable)

srbhr commented 1 year ago

Hey @sriramr244 I think you have added the RM package in this PR. Can you remove it?

sriramr244 commented 1 year ago

Oops sorry I will remove it. Its a venv

On Wed, Aug 30, 2023 at 10:32 AM Saurabh Rai @.***> wrote:

Hey @sriramr244 https://github.com/sriramr244 I think you have added the RM package in this PR. Can you remove it?

— Reply to this email directly, view it on GitHub https://github.com/srbhr/Resume-Matcher/pull/129#issuecomment-1699301661, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMA3736EYR2VEBKZEC3BE53XX5FIHANCNFSM6AAAAAA4ETUYFA . You are receiving this because you were mentioned.Message ID: @.***>

srbhr commented 1 year ago

The thing is that adding qdrant-client in the requirements.txt installs version <1. Which doesn't contains the URL parameter. I think I need to add that into the README or some other file.

srbhr / Resume-Matcher