Skip to content

remove self from redis_object from _add_vector in RedisStorage#89

Open
sarthakTUM wants to merge 2 commits into
pixelogik:masterfrom
sarthakTUM:sg-issue88
Open

remove self from redis_object from _add_vector in RedisStorage#89
sarthakTUM wants to merge 2 commits into
pixelogik:masterfrom
sarthakTUM:sg-issue88

Conversation

@sarthakTUM

Copy link
Copy Markdown
Collaborator

Refer to #88

@sarthakTUM sarthakTUM requested a review from pixelogik February 16, 2020 20:27
@sarthakTUM

Copy link
Copy Markdown
Collaborator Author

looks like CI is broken?

@sarthakTUM sarthakTUM requested review from pixelogik and removed request for pixelogik February 17, 2020 20:30
@amorgun

amorgun commented Feb 18, 2020

Copy link
Copy Markdown
Collaborator

LGTM

@sarthakTUM sarthakTUM changed the title removed redis_object from _add_vector remove self from redis_object from _add_vector in RedisStorage Feb 19, 2020
@sarthakTUM

Copy link
Copy Markdown
Collaborator Author

Any updates on this PR?

@sarthakTUM

Copy link
Copy Markdown
Collaborator Author

@amorgun do you have to rights to merge?

@amorgun

amorgun commented Feb 26, 2020

Copy link
Copy Markdown
Collaborator

No, I cannot merge requests. We have to wait for @pixelogik.

@pixelogik

Copy link
Copy Markdown
Owner

@amorgun @sarthakTUM Sorry for the long silence. Last year was crazy because of Covid, work and family. I do not have much time for this project anymore. Are you interested in joining the project so that you can also accept pull requests?

@sarthakTUM

Copy link
Copy Markdown
Collaborator Author

no problem, I hope you are doing fine @pixelogik. Sure, I am happy to join this project.

@amorgun

amorgun commented Jan 21, 2021

Copy link
Copy Markdown
Collaborator

@pixelogik Yes, I would like to join the project.

@pixelogik

Copy link
Copy Markdown
Owner

@amorgun @sarthakTUM I invited you both as contributors. Looking forward to working together.

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.

3 participants