Skip to content

Conversation

@averikitsch
Copy link
Collaborator

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> 🦕

@averikitsch averikitsch requested review from a team as code owners December 11, 2025 23:52
@product-auto-label product-auto-label bot added the api: alloydb Issues related to the googleapis/langchain-google-alloydb-pg-python API. label Dec 11, 2025
@anubhav756
Copy link
Contributor

Errors from the failing integration tests:

E           ValueError: Model text-embedding-00541f520aa_eb20_411c_9b89_9146e21a0c3e does not exist.

Do you think we might need some changes in the model manager, or adjust its test script?

Full logs

@dishaprakash
Copy link
Contributor

dishaprakash commented Dec 15, 2025

@anubhav756 there was a change in the API results from AlloyDB list models causing this error, which has now been fixed.

@averikitsch should we close this PR in favor of #523?

@averikitsch
Copy link
Collaborator Author

@anubhav756 there was a change in the API results from AlloyDB list models causing this error, which has now been fixed.

@averikitsch should we close this PR in favor of #523?

yes, thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: alloydb Issues related to the googleapis/langchain-google-alloydb-pg-python API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants