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
Error in documentation regarding initialisation of RedisBloom and TopK #2415
Comments
Hey - thanks for using the library! What a find - So I get the same error when I do that from the terminal / interpreter, however when I use that syntax in a script it runs just as expected. Could you confirm for me if this syntax works for you in a script? |
Hello @sav-norem I ran the same syntax both in a shell (terminal) and as a script. It throws an error in both cases. |
@radioactive11 woops! I forgot a key part of my code that works in a script - you sort of got it in the piece that works, but you need to set the |
@sav-norem Exactly But in the documentation, the sample that is provided doesn't instantiate a |
@radioactive11 yup! I've opened an issue with the docs team and we'll leave this open until that change in the docs gets out to the readthedocs page |
* Fixes #2415, adds a connect step to bloom commands * Update redismodules.rst Apply feedback
Link to documentation
Version: redis-py version: 4.3.4 & redis-stack running on Docker (v6.2.7)
Platform: Python 3.10.6 on macOS
Description: Creating a bloom filter or TopK client according to the documentation throws an AttributeError
Sample:
Solution:
I am facing similar issue with TopK filters as well.
I can raise a PR to solve this issue as well.
The text was updated successfully, but these errors were encountered: