Skip to content
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

release-to-lemonade-hq #1

Open
wants to merge 3 commits into
base: opensearch
Choose a base branch
from
Open

Conversation

igorshapiro
Copy link
Collaborator

No description provided.

@igorshapiro igorshapiro force-pushed the opensearch branch 2 times, most recently from a6f0006 to 37f1b8f Compare April 13, 2023 16:38
igorshapiro pushed a commit that referenced this pull request May 14, 2023
feat: add MyScale as vectorStore provider
igorshapiro pushed a commit that referenced this pull request May 14, 2023
* Merge post upstream sync (#1)

* Added RedisMemory with tests and docs

* fixed Redis test file name

* TODOs for quick refactor

* updated memory to accept a client

* Added TODOS

* ongoing testing

* Updated tests

* Updated tests for memory return option

* finalized tests + updated docs

* Adding docs

* Readded init and cleaned up func

* fixed test typing

* redo yarn lock

* remove yarn.lock

* updated yarn lock and namespaced redis

* fix merge conflict

* updated BaseChatMemoryInput

* Updated with lint fixes

* Fixed docs to match memory instantiation

* yarn format docs

* Merging fixes to address ForkPR comments (langchain-ai#2)

* updating for pr

* Removed redis_memory in favor of chat memory

* Fixed tests and updated docs

* Bump docs

* lint results

* fixes from lint and format

---------

Co-authored-by: Chris Toomey <[email protected]>

* Update RedisChatMessageHistory class to initialize a client internally

* Update Redis chat message history docs to reflect the fact that Redis is a peer dependency, allow direct passing of Redis client

* Patch typing issues with passing node-redis client directly into a RedisChatMessageHistory instance

---------

Co-authored-by: Chris Toomey <[email protected]>
Co-authored-by: Jacob Lee <[email protected]>
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.

1 participant