-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Issue with Milvus #2081
Comments
I am facing a similar issue. Environment: Error Log: System: |
Same goes to me for deepset-ai/rasa-haystack by using docker-compose. 2022-01-28 07:27:02 INFO rasa_sdk.endpoint - Starting action endpoint server... System: OS: Ubuntu 18.04 |
Having the same issue now after re-installing farm-haystack==1.0.0. Setting this environment variable MILVUS2_ENABLED=False solved it.
Why were older releases affected by this? |
PyMilvus updated to version 2.0.0 on January 25th and that is now what gets installed from PyPI by default so recent recent farm-haystack installations have been breaking. I fixed this issue by installing |
For those who have tried Or for short:
|
Hey @baregawi, thank you so much for the comments above! The issue was really the latest release of Milvus and pinning its version to the latest 1.x release (in this case, 1.1.2) should fix it. @Krak91 Haystack has partial Milvus2 support since a few releases, it was just hidden behind that environment variable. Setting it to any value should enable Milvus 2.0 support, which on one hand fixes the bug (because this way, the imports will be working with pymilvus==2.0.0), but the code paths that it enables are a bit buggy, so I generally wouldn't recommend to do it unless you have no way to downgrade pymilvus. As a general reminder, make sure that also your Milvus containers or local installation also are fixed to v1.1.2 to avoid incompatibilities between Milvus and pymilvus. Sorry for this issue! |
@ZanSara thanks for the clarification. If I don't intent to use milvus at all, is the env variable sufficient or is it still recommended to downgrade the version? |
@Krak91 I recommend to downgrade. The code that is enabled by setting the env var is experimental and contains some bugs, so it might always cause unforeseen issues later. Downgrading pymilvus is safer and way better tested 🙂 |
@ZanSara @MichelBartels is that problem resolved now? |
I'd say Milvus2 support is not ready for use yet, but a lot of progress has been done. See this PR: #2126 |
Hey. I've been trying to list Haystack in the requirements.txt file with the constraint like proposed above but it doesn't work. Any idea how to list haystack in requirements.txt and bypass this issue? |
Hey @alankashkash, could you give us more details? For example, which version of Haystack, pip and Python are you using? The answer is likely to differ depending on whether you installed Haystack from PyPI or from GitHub. |
Hey @ZanSara absolutely. Thank you for the response! I'm using AWS Elasticbeanstalk to deploy API and did a little PoC endpoint using Haystack. However, since AWS EB installs all packages using requirements.txt I'm not able to provide additional commands for setting up a constraint. Appreciate any suggestions :) |
@alankashkash ok, you can probably upgrade then! We release version 1.2.0 just yesterday, and it comes with improved dependency management. Two ways to go with it:
Let me know if this helps! |
In addition, we recently merged full support for Milvus2! 🎉 #2126
|
Thanks for the hardworking to support milvus 2.0 @ZanSara |
Milvus will soon support embedded model so user could play with a simply a pip install~ |
Hello @xiaofan-luan, sure, feel free to add us to the list! 🙂 Many thanks for the heads up about the embedded model feature! We'll look into it 👍 |
Since release 1.3.0 includes all Milvus fixes, I will close this issue. If you still experience similar problems with Milvus, please open a new issue 🙂 |
I am building docker image of my project with Python3.8 as base image. And installing pymilvus in it. But when I run container, I am getting no module name pymilvus error. I tried to downgrade the version of pymilvus, still getting same issue. Any suggestions? |
Sounds weird. Could you open a new issue about this, with more details about your image? PS: Sorry for the late reply, but we don't monitor old and closed threads like this one. I noticed your reply mostly by chance. Next time please open a new issue instead. |
pymilvus==1.1.2 !!!! |
Describe the bug
We installed farm_reader.
Error message
1)we installled farm-haystack==0.9.0
2) We installed pip3 install pymilvus==1.1.2
File "/app/main.py", line 6, in
from haystack import document_store
File "/usr/local/lib/python3.7/site-packages/haystack/init.py", line 5, in
from haystack.finder import Finder
File "/usr/local/lib/python3.7/site-packages/haystack/finder.py", line 8, in
from haystack.reader.base import BaseReader
File "/usr/local/lib/python3.7/site-packages/haystack/reader/init.py", line 1, in
from haystack.reader.farm import FARMReader
File "/usr/local/lib/python3.7/site-packages/haystack/reader/farm.py", line 21, in
from haystack.document_store.base import BaseDocumentStore
File "/usr/local/lib/python3.7/site-packages/haystack/document_store/init.py", line 4, in
from haystack.document_store.milvus import MilvusDocumentStore
File "/usr/local/lib/python3.7/site-packages/haystack/document_store/milvus.py", line 7, in
from milvus import IndexType, MetricType, Milvus, Status
ModuleNotFoundError: No module named 'milvus'
Expected behavior
It was working until 25th Jan and it stopped working.
Additional context
Add any other context about the problem here, like document types / preprocessing steps / settings of reader etc.
To Reproduce
Steps to reproduce the behavior
FAQ Check
System:
The text was updated successfully, but these errors were encountered: