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

feat: SpannerChatMessageHistory implementation #7

Merged
merged 10 commits into from
Feb 21, 2024
Merged

feat: SpannerChatMessageHistory implementation #7

merged 10 commits into from
Feb 21, 2024

Conversation

rahul2393
Copy link
Contributor

@rahul2393 rahul2393 commented Jan 30, 2024

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

@rahul2393 rahul2393 requested a review from a team as a code owner January 30, 2024 15:27
@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/langchain-google-spanner-python API. label Feb 6, 2024
"cell_type": "code",
"outputs": [],
"source": [
"To run this notebook, you will need a Google Cloud Project, a Cloud Spanner instance, and Google credentials."
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

{
"cell_type": "markdown",
"source": [
"%pip install langchain-google-spanner"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be a code cell

Copy link
Collaborator

@averikitsch averikitsch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doc fixes can be prioritized separately

@rahul2393 rahul2393 force-pushed the memory branch 6 times, most recently from b0af3cf to 0b1f6df Compare February 19, 2024 09:24
Copy link
Collaborator

@averikitsch averikitsch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please let me know if the user agent will be added in this PR or separately.

@rahul2393
Copy link
Contributor Author

@averikitsch Added user agent support

COLUMN_NAME = "history"


def client_with_user_agent(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keeping a note: We will create a common utility at the module layer once all the changes are checked in.

@averikitsch averikitsch merged commit f9a3b93 into main Feb 21, 2024
7 checks passed
@averikitsch averikitsch deleted the memory branch February 21, 2024 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: spanner Issues related to the googleapis/langchain-google-spanner-python API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants