Harrison/GitHub setup #2
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jacoblee93
pushed a commit
to jacoblee93/langchainjs
that referenced
this pull request
May 3, 2023
jacoblee93
pushed a commit
to jacoblee93/langchainjs
that referenced
this pull request
May 10, 2023
* 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 <ctoomey@fb.com>
nfcampos
pushed a commit
that referenced
this pull request
May 11, 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 (#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 <ctoomey@fb.com> * 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 <ctoomey@fb.com> Co-authored-by: Jacob Lee <jacoblee93@gmail.com>
nfcampos
added a commit
that referenced
this pull request
Jun 27, 2023
…odelStart always in the same order as the prompts passed in What was happening was that the runManager was only added to the managers array after the callback handlers finished executing, so eg if the callback handlers for the second run finished first the runManager for prompt #2 would actually be in position #1 of the returned array.
agola11
pushed a commit
that referenced
this pull request
Jun 27, 2023
…odelStart always in the same order as the prompts passed in (#1779) What was happening was that the runManager was only added to the managers array after the callback handlers finished executing, so eg if the callback handlers for the second run finished first the runManager for prompt #2 would actually be in position #1 of the returned array.
jacoblee93
pushed a commit
that referenced
this pull request
Aug 25, 2023
1 task
1 task
This was referenced Sep 29, 2023
jacoblee93
pushed a commit
that referenced
this pull request
Oct 9, 2024
* fix test * lint * cr * cr * cr * cr * cr * more tests * cr
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.