redis-developer / redisbloom-moby-dick

Experiments with RedisBloom and the text from Moby Dick
MIT License
4 stars 3 forks source link

Added example project in Python #2

Closed radioactive11 closed 2 years ago

radioactive11 commented 2 years ago

PR to solve this issue Here is a list of all the changes:

SuzeShardlow commented 2 years ago

Closes #1 once merged.

SuzeShardlow commented 2 years ago

Hi @radioactive11, thanks for your PR! We will review it.

In the meantime, if you are a member of our Discord server (https://discord.gg/redis), please let us know your handle. Thanks!

radioactive11 commented 2 years ago

@SuzeShardlow My Discord handle is radioactive11#3360

radioactive11 commented 2 years ago

Greetings @simonprickett This error was caused due a mistake on my part. I used split instead of strip. The code should be working fine after my last commit.

Thank you and sorry for the hassle 😅

radioactive11 commented 2 years ago

Greetings @simonprickett I have made the necessary changes as suggested by you. Adding the pipeline has improved the execution time by a huge factor. Do let me know if any other changes are required.

Thank You

SuzeShardlow commented 2 years ago

Hi @radioactive11, thanks for your contribution :) I have given you the Hacktoberfest 2022 role in our Discord server.